print: switch to stdplus print

The existing print.hpp was reported as having issues with the
latest clang. Switch to use the stdplus version.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Ch

print: switch to stdplus print

The existing print.hpp was reported as having issues with the
latest clang. Switch to use the stdplus version.

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

show more ...

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: Ib09d42d20372d1c39a81598c207e2756160b7f66
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

remove use of fmt

All uses of fmt can be replaced with either std::format directly
or a tentative implementation of std::print (which we can use
until GCC supports std::print). Remove the extra dep

remove use of fmt

All uses of fmt can be replaced with either std::format directly
or a tentative implementation of std::print (which we can use
until GCC supports std::print). Remove the extra dependency.

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

show more ...

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

Change-Id: Ib6a192fb0077b4ac8c7e7

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

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

show more ...

clang-format: copy latest and re-format

clang-format-17 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-17 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: I37706a2cb7da2a51ec6f6d42f59cea677e1fc111
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: I68b7c13d671fdbea99a10577e811c53cb5198aff
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig` keyword argument. Ensure meson 0.58 is required and update
the usage of all `get_pkgconfig_variable` and `get_variable` to be the
modern variant.

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

show more ...

prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

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

show more ...

ensure service start success

- In phosphor-ipmi-host, if service start failed, it will auto
restart. But kcs service will not start because dependency
service start failed if ipmi-host service s

ensure service start success

- In phosphor-ipmi-host, if service start failed, it will auto
restart. But kcs service will not start because dependency
service start failed if ipmi-host service start failed at first
time. So change dependency to use Wants instead of Requires.

Signed-off-by: Harvey.Wu <Harvey.Wu@quantatw.com>
Change-Id: I73aa563ab31a4c66802f67420291b87636e16a97

show more ...

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have the correct field.

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

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

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

show more ...

Add OWNERS file

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Idea57d97f07f0cafab3b1fe0053ccd343601eb1f

span: switch to std

C++20 has 'std::span' now. Favor this over the backport that
was in the stdplus repository.

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

span: switch to std

C++20 has 'std::span' now. Favor this over the backport that
was in the stdplus repository.

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

show more ...

kcsbridge: Daemon rewrite

This focuses on improving compile time, output size, and correctness
of command handling. Notably, it adds the ability for the host to cancel
outstanding IPMI calls when th

kcsbridge: Daemon rewrite

This focuses on improving compile time, output size, and correctness
of command handling. Notably, it adds the ability for the host to cancel
outstanding IPMI calls when the host decides they should time out. It
ensures that canceled calls do not end up writing over the response for
an incoming request. Since the host can now cancel calls, we remove
internal DBus timeouts and instead require the host to drive any timeout
logic it desires for communications. This gives the host more control
and better adheres to the wishes of the specification.

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

show more ...

clang-format: Update to latest file

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

build: Install bridge in libexec

Users aren't intended to run the server directly so it shouldn't be
included in their PATH.

Change-Id: Idc49664a7a07142ee27aac1318066bba6f67a6fd
Signed-off-by: Will

build: Install bridge in libexec

Users aren't intended to run the server directly so it shouldn't be
included in their PATH.

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

show more ...

build: Make a src subdirectory

This will allow us to organize our c++ source separately from our
top-level metadata.

Change-Id: I3141fa63bf9be2ae1b1efee7a6f95ba53a569f5d
Signed-off-by: William A. K

build: Make a src subdirectory

This will allow us to organize our c++ source separately from our
top-level metadata.

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

show more ...

build: Remove autotools and stale files

This removes all of the metadata files related to autotools and purges
some stale service and d-bus configurations that were not being
installed or referenced

build: Remove autotools and stale files

This removes all of the metadata files related to autotools and purges
some stale service and d-bus configurations that were not being
installed or referenced by the build.

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

show more ...

build: Add meson

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

bootstrap: fix shellcheck warnings

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

Don't use "-" in a DBus interface path and name

Current code use device name "ipmi-kcs*" as a part of a naming
for Dbus interface. It is not possible to use it as is, because
"-" symbol is not permi

Don't use "-" in a DBus interface path and name

Current code use device name "ipmi-kcs*" as a part of a naming
for Dbus interface. It is not possible to use it as is, because
"-" symbol is not permitted by DBus specification. Therefore
replace it with "_".

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: I7de6acb52d26015c6984156f7167fe228ccc3d1e

show more ...

MAINTAINERS: remove Ed Tanous as maintainer

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

sdbusplus: replace message::variant with std::variant

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

Remote Bios Configuration:Increasing buffer size.

Increasing the KCS buffersize to receive 4K bytes of data
from BIOS.

Tested:By giving SetPayload command we are able to get and set
4096 bytes of d

Remote Bios Configuration:Increasing buffer size.

Increasing the KCS buffersize to receive 4K bytes of data
from BIOS.

Tested:By giving SetPayload command we are able to get and set
4096 bytes of data between BIOS and BMC.

Signed-off-by: Rashmi RV <rashmi.r.v@linux.intel.com>
Change-Id: I6fa418ea84a216b30430bbd94b50d5faefe12a1f

show more ...