| 84778b8d | 26-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: remove phosphor-ipmi-blobs dependency
The host tool depended on the BMC header: firmware_handler.hpp because
it defined the flags. This header depends on phosphor-ipmi-blobs,
therefore the h
tools: remove phosphor-ipmi-blobs dependency
The host tool depended on the BMC header: firmware_handler.hpp because
it defined the flags. This header depends on phosphor-ipmi-blobs,
therefore the host-tool depends on this. Move the flags into a separate
common header file and snip this dependency.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id2ad33a1611c02605a1ed5c695429d0451eb98d4
show more ...
|
| f0c71df2 | 26-Jun-2019 |
Patrick Venture <venture@google.com> |
move bmc code into bmc folder
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I2407359461a1a2f0e733ff8cd0cb1744b6304c32 |
| 7c79b252 | 24-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: implement lpc support
Tested: Verified this works for a Nuvoton BMC.
EXTRA_OECONF_append_xxx = " \
--enable-static-layout \
--enable-lpc-bridge \
--enable-nuvoton-lpc \
--enable-rebo
tools: implement lpc support
Tested: Verified this works for a Nuvoton BMC.
EXTRA_OECONF_append_xxx = " \
--enable-static-layout \
--enable-lpc-bridge \
--enable-nuvoton-lpc \
--enable-reboot-update \
MAPPED_ADDRESS=0xc0008000 \
"
/tmp/phosphor_ipmi_flash_tool \
--command update \
--interface ipmilpc \
--image image-bmc \
--sig image-bmc.sig \
--type static \
--address 0xfedc1000 \
--length 0x1000
Sending over the firmware image.
trying to open blob
sending writeMeta
caught exception
EFBIG returned!
sending writeMeta
writemeta sent
Sending over the hash file.
trying to open blob
sending writeMeta
caught exception
EFBIG returned!
sending writeMeta
writemeta sent
Opening the verification file
Committing to /flash/verify to trigger service
Calling stat on /flash/verify session to check status
other
success
Returned success
succeeded
Opening the update file
Committing to /flash/update to trigger service
Calling stat on /flash/update session to check status
running
Opening the cleanup blob
Exception received: blob exception received: Received IPMI_CC: 255
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: If866303e95e9b6a19dc8b20a99bb89fd66f95eeb
show more ...
|
| 9f937c45 | 21-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: add support for ubi tarball from host
Add a parameter that specifies the layout type, static or ubitar.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I86c09d8ffeb55ba789279d
tools: add support for ubi tarball from host
Add a parameter that specifies the layout type, static or ubitar.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I86c09d8ffeb55ba789279d2568b579a44b3923d7
show more ...
|
| de73c3b7 | 20-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: bugfix: add missing close after failure
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id873c41997e2617d1fa3db5767112c23d77c487c |
| 01123b2a | 20-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: split out the update handler
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I367ea961c98ec822d3200d101c4f5977d77e7402 |
| 03db87e4 | 20-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: specify parameter size limits for LPC
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ic7049ffc2a3c4e7dad3356e11f48f6e5ed8eeca1 |
| 5f2fcc4e | 20-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: add cleanup blob on failure
If the update process fails, call the cleanup blob and try to cleanup
artifacts automatically.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ib57
tools: add cleanup blob on failure
If the update process fails, call the cleanup blob and try to cleanup
artifacts automatically.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ib57e5cc7f13b73e2d7371116058469a4d4bfd0c4
show more ...
|
| 8104a525 | 19-Jun-2019 |
Patrick Venture <venture@google.com> |
tool: LPC data handler needs an address and length
The host needs to have memory set aside for this purpose, and this
enables the caller to provide those values.
Signed-off-by: Patrick Venture <ven
tool: LPC data handler needs an address and length
The host needs to have memory set aside for this purpose, and this
enables the caller to provide those values.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: If40ec3b08bd1932cba91d3f770c7fde6f280463d
show more ...
|
| 1f09d414 | 19-Jun-2019 |
Patrick Venture <venture@google.com> |
tool: continue the unit-tests of the host tool
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I51349fc103af2a55f1b83830bf8e74407fc15b9e |
| 380832cc | 19-Jun-2019 |
Patrick Venture <venture@google.com> |
test: cleanup by moving tool tests into subfolder
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I15b3514d095d9fc770a1b0fe125eb48d96ada2c8 |
| f6ea2e83 | 12-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: update status debug message
Update the verification commit status debug message to not specify that
it's the verification blob specifically as this method is used for both
verification and up
tools: update status debug message
Update the verification commit status debug message to not specify that
it's the verification blob specifically as this method is used for both
verification and update.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I0ee89a5e8f7eae2533ab89db12032b33d1c25ec1
show more ...
|
| 14713bec | 05-Jun-2019 |
Patrick Venture <venture@google.com> |
tools: open and commit() on updateBlobId
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Id4d531e2ea2d755aa310261966254419d78dccdb |
| da66fd84 | 03-Jun-2019 |
Patrick Venture <venture@google.com> |
merge Verification and Update statuses
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ibb9c1c9f9cb7e9cda18f4e8dddb31d57a12f0d90 |
| c674474c | 28-May-2019 |
Patrick Venture <venture@google.com> |
tools: updater: cppcheck fixups
The scope of the variable 'commandAttempts' can be reduced.
The scope of the variable 'attempts' can be reduced.
The scope of the variable 'exitLoop' can be reduced.
tools: updater: cppcheck fixups
The scope of the variable 'commandAttempts' can be reduced.
The scope of the variable 'attempts' can be reduced.
The scope of the variable 'exitLoop' can be reduced.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ib2e258c1af7110ff4568695b8f1ea877a9ffef99
show more ...
|
| 63528046 | 20-May-2019 |
Patrick Venture <venture@google.com> |
tools: scope of the variable 'bytesRead' can be reduced.
(style) The scope of the variable 'bytesRead' can be reduced.
[tools/bt.cpp:21]
[tools/p2a.cpp:111]
Signed-off-by: Patrick Venture <venture@
tools: scope of the variable 'bytesRead' can be reduced.
(style) The scope of the variable 'bytesRead' can be reduced.
[tools/bt.cpp:21]
[tools/p2a.cpp:111]
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I6b96247a02cd35d073b424501bb161e80aedf3b4
show more ...
|
| 1d5a31c9 | 20-May-2019 |
Patrick Venture <venture@google.com> |
bmc: move away from blobs namespace
The blobs namespace is really owned by phosphor-ipmi-blobs. Move all
phosphor-ipmi-flash objects into ipmi_flash namespace.
Signed-off-by: Patrick Venture <vent
bmc: move away from blobs namespace
The blobs namespace is really owned by phosphor-ipmi-blobs. Move all
phosphor-ipmi-flash objects into ipmi_flash namespace.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I51b721ad4f3cdc4cc43846c942967ed2e5f16589
show more ...
|
| 3ecb3503 | 17-May-2019 |
Patrick Venture <venture@google.com> |
bmc: enable configuration of all verification
Enable verification configuration beyond the result file check. This
patchset allows a developer to provide their own verification mechanisms
by implem
bmc: enable configuration of all verification
Enable verification configuration beyond the result file check. This
patchset allows a developer to provide their own verification mechanisms
by implementing an interface and adding configuration to enable using
their custom version.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Iba5d4be75bb49d9c4ab84be8578f0306c15b5be9
show more ...
|
| 7dad86fd | 17-May-2019 |
Patrick Venture <venture@google.com> |
util: create util object for bmc and tool
Create a common util object for the BMC library and host tool. Place in
blobs namespace favoring BMC.
Signed-off-by: Patrick Venture <venture@google.com>
util: create util object for bmc and tool
Create a common util object for the BMC library and host tool. Place in
blobs namespace favoring BMC.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I036fd65d924c65fcfa71aba5cd76275a2941ff14
show more ...
|
| 55646dec | 16-May-2019 |
Patrick Venture <venture@google.com> |
tools: add update handler
Add an UpdateHandler to handle each step of the update process.
This code was already in place but is now handled via an UpdateHandler
object.
Tested: Not yet tested.
Sig
tools: add update handler
Add an UpdateHandler to handle each step of the update process.
This code was already in place but is now handled via an UpdateHandler
object.
Tested: Not yet tested.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I36cd1b94d8e2c0788d09805d935738d86e6e33de
show more ...
|
| bc841be9 | 16-May-2019 |
Patrick Venture <venture@google.com> |
tools: drop extraneous return statemen
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I2a12b61b63ec1e0d54fffae07431c145b988ad7f |
| d61b0ff8 | 15-May-2019 |
Patrick Venture <venture@google.com> |
tools: implement firmware verification polling
The verification process is asynchronous, therefore poll the BMC
firmware state.
Tested: Verified it behaves as intended, before moving it into a
sub-
tools: implement firmware verification polling
The verification process is asynchronous, therefore poll the BMC
firmware state.
Tested: Verified it behaves as intended, before moving it into a
sub-routine:
Opening the verification file
Committing to verification file to trigger verification service
Calling stat on verification session to check status
stat received:
blob_state: 0x206
size: 0x0
metadata(1): 0x3
other
stat received:
blob_state: 0x206
size: 0x0
metadata(1): 0x0
running
stat received:
blob_state: 0x206
size: 0x0
metadata(1): 0x0
running
stat received:
blob_state: 0x206
size: 0x0
metadata(1): 0x1
success
stat received:
blob_state: 0x206
size: 0x0
metadata(1): 0x1
success
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I3cb13d1a966c2c833fd4fee5654332f34e80355a
show more ...
|
| 7dcca5dd | 15-May-2019 |
Patrick Venture <venture@google.com> |
tools: trigger verification
Trigger verification via committing to the verify blob id.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ia3722ff144ba268cd554bac0fda06f0c426edd88 |
| 213f2db6 | 15-May-2019 |
Patrick Venture <venture@google.com> |
tools: io: fix printf portability
The printfs were using actual sizes instead of leveraging %z to deal
with std::size_t.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I7ac731b0458d
tools: io: fix printf portability
The printfs were using actual sizes instead of leveraging %z to deal
with std::size_t.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I7ac731b0458d9501762651bdc9d796f7a6ada620
show more ...
|
| 46e69491 | 15-May-2019 |
Patrick Venture <venture@google.com> |
bugfix: tools: use pci member
Tool worked by declaring its own pci utility object instead of using the
one provided. There are not yet unit-tests associated with the
P2aDataHandler that would have
bugfix: tools: use pci member
Tool worked by declaring its own pci utility object instead of using the
one provided. There are not yet unit-tests associated with the
P2aDataHandler that would have caught this.
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I62cdeb17d298f1db51d8f513c59bfd299fc2884a
show more ...
|