clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

Change-Id: I09e19a126d456dc184fd70e82d4e4e4217fa8ee7
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

Change-Id: I2ee8cbc4581d0c3ccdc130b646357c45fa0373db
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

tools: Support empty image file and ignore the fileSize check

Removed the unit test for checking file size of zero.

Tested:

Sending `/dev/null` and still works fine.

tools: Support empty image file and ignore the fileSize check

Removed the unit test for checking file size of zero.

Tested:

Sending `/dev/null` and still works fine.

Change-Id: Ibbde4fd430d9550fb7b75b6a583a53d4425d8075
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

tools: Fix missing FD close()

Change-Id: Ia244931d2bf634d037e2319e11b477a77898826a
Signed-off-by: William A. Kennington III <wak@google.com>

tools: Make progress bar starting more consistent

Change-Id: I0bfa967efa1c350551dd7b68f9fd72695db638c7
Signed-off-by: William A. Kennington III <wak@google.com>

tools/progress: Add finish and abort functions

We have external progress handlers which would like to implement these
functions, in order to update the screen upon completion or error.

tools/progress: Add finish and abort functions

We have external progress handlers which would like to implement these
functions, in order to update the screen upon completion or error.

Change-Id: I2df4654d5e2092407a83d2c8bbb020dcce84ac50
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

update gcc10 and clang-format10

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I72239b884615d26b2c4311c49a26e2eced0a1fb0

tools: add progress implementation

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I9da1674d6cbc688efc7bab0e033788d6ee4694f7

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,

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 ...

tools: implement lpc support

Tested: Verified this works for a Nuvoton BMC.

EXTRA_OECONF_append_xxx = " \
--enable-static-layout \
--enable-lpc-bridge \
--enable-n

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 ...

tools: specify parameter size limits for LPC

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: Ic7049ffc2a3c4e7dad3356e11f48f6e5ed8eeca1

tools: lpc: start implementation, send metadata

With the LPC interface, send the window information to the BMC.

Tested: Ran on test system and verified window mapped via ioctl to

tools: lpc: start implementation, send metadata

With the LPC interface, send the window information to the BMC.

Tested: Ran on test system and verified window mapped via ioctl to
driver as expected.

Change-Id: I7ffac6b52205b215fd698a574d098b505091c3d4
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

tools: put all host-side code into host_tool namespace

To avoid confusion and organize the code itself more cleanly, place the
host-side code into its own namespace.

Change-Id:

tools: put all host-side code into host_tool namespace

To avoid confusion and organize the code itself more cleanly, place the
host-side code into its own namespace.

Change-Id: I573eb0494eb59874adb1d1eadc502499e928e396
Signed-off-by: Patrick Venture <venture@google.com>

show more ...

tools: start implementing data interface

The host must provide multiple data interfaces. Each data interface is
responsible for configuring its data pathway and sending the contents

tools: start implementing data interface

The host must provide multiple data interfaces. Each data interface is
responsible for configuring its data pathway and sending the contents
via this pathway, as well as any flow control.

Change-Id: If856519536d01043e69c45044fcd0ad525592486
Signed-off-by: Patrick Venture <venture@google.com>

show more ...