Remove powercycle units / scripts

These are moving outside of this repo so we don't have to install ipmi
dependencies on non-IPMI platforms.

Change-Id: Ie7376a3a91f6694c3a8717088732f2d4cd441fd7
Sig

Remove powercycle units / scripts

These are moving outside of this repo so we don't have to install ipmi
dependencies on non-IPMI platforms.

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

show more ...

remove IPMI_CC

Since IPMI_CC declared in api.h has been gradually deprecated,
this submission will use ipmi::cc in api.hpp instead.

Change-Id: I4de358f8db628269b64f33acc4d6f250e27f6405
Signed-off-b

remove IPMI_CC

Since IPMI_CC declared in api.h has been gradually deprecated,
this submission will use ipmi::cc in api.hpp instead.

Change-Id: I4de358f8db628269b64f33acc4d6f250e27f6405
Signed-off-by: George Liu <liuxiwei@ieisystem.com>

show more ...

Add support for setting VoutSetpoint and NominalTargetVout

Change-Id: I21e0064bcd8c7ba0352954b639556cc2f8af870f
Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>

test: Fix test with read_array/append_array

After the sdbusplus [1], change the expected dbus call changed and
required us to update the unit tests.

[1] https://github.com/openbmc/sdbusplus/commit/

test: Fix test with read_array/append_array

After the sdbusplus [1], change the expected dbus call changed and
required us to update the unit tests.

[1] https://github.com/openbmc/sdbusplus/commit/242677a2b3dafdb3b79464eb64e0ae8bc7c5a7d5

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

show more ...

markdownlint: minor fixes

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

Add support for VoltageMargin setting

Tested: Can set/get the voltage_margin setting
Change-Id: I8d1042a90ad75a312a30daae43e231edea195281
Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>

meson: reformat with meson formatter

Apply the `meson format` results.

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

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: I46c47707399c23ec5c84c8e654d6e912cd340f66
Signed-off-by: Patrick Williams <p

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

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

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.

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

show more ...

Use custom dbus interface for SetVrSettings

Change-Id: I690e051e47bd55ef93147f5170ffa23bcd0d7030
Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>

build: use allowed over enabled or not-disabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`)

build: use allowed over enabled or not-disabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`) no longer return true for auto features. Instead, the
expectation is to use `allowed()` which is true for both enabled and
auto.

Switch all uses of `enabled` to `allowed`.
Switch all uses of `not disabled` to `allowed`.

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

show more ...

Use new template gbmc-bare-metal-active@.target

The systemd target gbmc-bare-metal-active.target is being changed to a
template, in order to support multi-host systems. The LinuxBootDone IPMI
comman

Use new template gbmc-bare-metal-active@.target

The systemd target gbmc-bare-metal-active.target is being changed to a
template, in order to support multi-host systems. The LinuxBootDone IPMI
command handler starts this target, so going forward, it needs to use
the systemd target gbmc-bare-metal-active@0.target, to indicate host 0.
This IPMI command is only used on single-host machines, so we can
hard-code the host number in this case.

Tested:
Exercised the LinuxBootDone IPMI command. First, set the flags
/run/bm-drive-cleaning-done-ack.flag and /run/bm-ready.flag on the BMC,
then ran the following IPMI command on the host
$ ./ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x11
79 2b 00 11
Confirmed that the gbmc-bare-metal-active@0.target got started.

Change-Id: Iba16c3fb378eb13f17e8d48fd3a45b8b1f1da84a
Signed-off-by: John Wedig <johnwedig@google.com>

show more ...

bios_setting: Add a write handler

Tested:
```
// Test that it writes a new file if it doesn't exist
~# rm /run/oem_bios_setting

// Invalid command (no size / payload)
~# ipmitool raw 0x2e 0x32 0x79

bios_setting: Add a write handler

Tested:
```
// Test that it writes a new file if it doesn't exist
~# rm /run/oem_bios_setting

// Invalid command (no size / payload)
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x19
Unable to send RAW command (channel=0x0 netfn=0x2e lun=0x0 cmd=0x32 rsp=0xc7): Request data length invalid

// Invalid command (size doesn't match)
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x19 0x01 0x30 0x31
Unable to send RAW command (channel=0x0 netfn=0x2e lun=0x0 cmd=0x32 rsp=0xc7): Request data length invalid

// Command success
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x19 0x02 0x30 0x31
79 2b 00 19 02

// Read back
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x18
79 2b 00 18 02 30 31
~# cat /run/oem_bios_setting
01~#

// Verify overwrite works (truncates the previous bytes)
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x19 0x01 0x33
79 2b 00 19 01
~# cat /run/oem_bios_setting
3~#
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x18
79 2b 00 18 01 33
```

Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I84db782da9b2f121c0a81a855692b5ca25ffda54

show more ...

bios_setting: Add a read handler

Tested:
```
~# echo -n "01234567" > /run/oem_bios_setting
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x18
79 2b 00 18 08 30 31 32 33 34 35 36 37
~# rm /run/oem_bios_s

bios_setting: Add a read handler

Tested:
```
~# echo -n "01234567" > /run/oem_bios_setting
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x18
79 2b 00 18 08 30 31 32 33 34 35 36 37
~# rm /run/oem_bios_setting
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x18
Unable to send RAW command (channel=0x0 netfn=0x2e lun=0x0 cmd=0x32 rsp=0xca): Cannot return number of requested data bytes
```

Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: Icde2de4799a751634f56a580351bf10254dd7e4f

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

show more ...

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://githu

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://github.com/codespell-project/codespell

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

show more ...

handler: Fix getBMInstanceProperty to read spaces

std::getline should be used to avoid terminating the read when there are
spaces.

Tested:
Before:
~# cat /run/bm-instance/asset-tag
testing space
~#

handler: Fix getBMInstanceProperty to read spaces

std::getline should be used to avoid terminating the read when there are
spaces.

Tested:
Before:
~# cat /run/bm-instance/asset-tag
testing space
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x17 0x00
79 2b 00 17 07 74 65 73 74 69 6e 67

After:
~# cat /run/bm-instance/asset-tag
testing space
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x17 0x00
79 2b 00 17 0d 74 65 73 74 69 6e 67 20 73 70 61 63 65

Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: Ia13880e385150487dfe07c4aabf7b3edc814ccf0

show more ...

bm_instance: Create a new handler

This OEM handler will get properties from tmpfs to serve over IPMI.

Tested:
~# cat /run/bm-instance/asset-tag
12345
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x17 0

bm_instance: Create a new handler

This OEM handler will get properties from tmpfs to serve over IPMI.

Tested:
~# cat /run/bm-instance/asset-tag
12345
~# ipmitool raw 0x2e 0x32 0x79 0x2b 0x00 0x17 0x00
79 2b 00 17 05 31 32 33 34 35
...
...
(Verified this works for all 7 sub commands)

Signed-off-by: Brandon Kim <brandonkim@google.com>
Change-Id: I191b6f4994d91ada49a3332a8e93a3a305561904

show more ...

Update accel setting name from power_break to power_brake

Change-Id: I09465b488eac355c1e1ae847fe350a82e7ad4729
Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>

Reserve boot-time-monitor cmd on 18-20

The implementation is delayed and still work in progress. It is being
used for testing internally and needs to be reserved to make sure it
doesn't break later

Reserve boot-time-monitor cmd on 18-20

The implementation is delayed and still work in progress. It is being
used for testing internally and needs to be reserved to make sure it
doesn't break later on.

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

show more ...

Add new oem command to set accel power mode

Signed-off-by: Gaurav Gandhi <gauravgandhi@google.com>
Change-Id: Ie0b47477b7ae9e8ae206ac0645571a7a79dbda40

google-sys-ipmi: Remove gbmc-bare-metal-active target

The target will be a separate recipe so multiple services can depend on
it without relying on google-sys-ipmi.

Change-Id: I957784a38b4356cf6dcd

google-sys-ipmi: Remove gbmc-bare-metal-active target

The target will be a separate recipe so multiple services can depend on
it without relying on google-sys-ipmi.

Change-Id: I957784a38b4356cf6dcde2920c398df2e76cb81f
Signed-off-by: Yuxiao Zhang <yuxiaozhang@google.com>

show more ...

meson: adjust nlohmann-json dependency

Simplify the nlohmann-json dependency in meson and align with the
usage from other repositories for consistency.

Change-Id: I6dcbd19a055b83fc8a5955cb6fe0bbdc9

meson: adjust nlohmann-json dependency

Simplify the nlohmann-json dependency in meson and align with the
usage from other repositories for consistency.

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

show more ...

Add wrapper script to setup delay powercycle

This script allow us to have a common way to setup the delay powercycle
without going through the ipmi command if needed.

Change-Id: Ida63f066551404b900

Add wrapper script to setup delay powercycle

This script allow us to have a common way to setup the delay powercycle
without going through the ipmi command if needed.

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

show more ...

fmt: Remove fmtlib dependencies

Remove the remaining usage for the fmtlib.

Tested: working fine with the error messages.

Change-Id: I68dc70428a75706b2fc70e43f8da321d84f052cc
Signed-off-by: Willy T

fmt: Remove fmtlib dependencies

Remove the remaining usage for the fmtlib.

Tested: working fine with the error messages.

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

show more ...