clang-format: update latest spec and reformat

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

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

show more ...

oem: ibm: fileTable: Add lids for Balcones system

Change-Id: Ice0d0734e7adf27c3d96fb345f6615d45842594b
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

clang-tidy: Replace NULL with nullptr

Replaced all instances of NULL with nullptr to improve type safety
and clarity, as nullptr is the modern C++ standard for null pointers.

Tested: Build verified

clang-tidy: Replace NULL with nullptr

Replaced all instances of NULL with nullptr to improve type safety
and clarity, as nullptr is the modern C++ standard for null pointers.

Tested: Build verified

Change-Id: If9e6c34c48821a7cf8577a2166727ce7db06fadc
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

oem-ibm: Change the primary postcode signature

The signature of the primary SRC has been changed from uint64_t to
array[uint8_t] [1], and pldm code is still using the old signature
hence the progres

oem-ibm: Change the primary postcode signature

The signature of the primary SRC has been changed from uint64_t to
array[uint8_t] [1], and pldm code is still using the old signature
hence the progress code function is broken. This commit fixes the
problem.

[1]: https://gerrit.openbmc.org/q/topic:%22primary-postcode-new-intf%22

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

show more ...

oem: ibm: fileTable: Add hostboot hash lids

Add new hostboot hash lids 80d0004* and hostboot tool data lid 81e006f0.

Change-Id: Id0578f4cce8a5248b2cd1e41da520690cd9f1183
Signed-off-by: Adriana Koby

oem: ibm: fileTable: Add hostboot hash lids

Add new hostboot hash lids 80d0004* and hostboot tool data lid 81e006f0.

Change-Id: Id0578f4cce8a5248b2cd1e41da520690cd9f1183
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

oem-ibm: Adding BIOS attribute for IPL version

This commit introduces two new string bios attributes to capture system
code level version with which last CEC full IPL was performed and
another attri

oem-ibm: Adding BIOS attribute for IPL version

This commit introduces two new string bios attributes to capture system
code level version with which last CEC full IPL was performed and
another attribute to capture version level in case of MPIPL (after
concurrent code update) scenario. Both the attributes will be updated by
remote terminus during every IPL.

Change-Id: I566c2f4a9cf606f5c9df73a796699b496c5b947f
Signed-off-by: Riya Dixit <riyadixitagra@gmail.com>

show more ...

oem-ibm: Change in dynamic deallocation BIOS attribute

This commit enhances the display name and help text of
hb_predictive_mem_guard and hb_predictive_mem_guard_current bios enum
attributes.

Teste

oem-ibm: Change in dynamic deallocation BIOS attribute

This commit enhances the display name and help text of
hb_predictive_mem_guard and hb_predictive_mem_guard_current bios enum
attributes.

Tested: Verified the change with redfish BIOS registry and verified with
dbus BIOS config manager.

Change-Id: I33fc426b92a90a826403142bb85e0db3de094b0a
Signed-off-by: Riya Dixit <riyadixitagra@gmail.com>

show more ...

oem-ibm: Add the dbus timeout while setting UAK

This commit adds the dbus timeout while setting the changed firmware
UAK value.

Tested:
Verified that UAK update is getting reflected in the correspo

oem-ibm: Add the dbus timeout while setting UAK

This commit adds the dbus timeout while setting the changed firmware
UAK value.

Tested:
Verified that UAK update is getting reflected in the corresponding
UTIL D8 VPD keyword and the redfish

Change-Id: I86e75a51e324b8ac0df8684cd005ceddada85bce
Signed-off-by: Jayashankar Padath <jayashankar.padath@in.ibm.com>

show more ...

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

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

show more ...

oem_ibm: Add Slot enable infrastructure in BMC

With this commit PDRs are created for each of these
PCIe slots and adapters which have a corresponding
entry in the entity associate map. The sensor/ef

oem_ibm: Add Slot enable infrastructure in BMC

With this commit PDRs are created for each of these
PCIe slots and adapters which have a corresponding
entry in the entity associate map. The sensor/effector
states are monitored and accordingly D-Bus calls are
sent for enabling a slot, process property change
of a PCIe slot, etc.

Tested By: SIMICS power on/off and reset reload.

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

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: Ia7dbb04ead7565d63437a5878fd8a543029d9a93
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

show more ...

clang-tidy: Enable modernize-deprecated-headers check

Some headers from C library were deprecated in C++ and are no
longer welcome in C++ codebases. Some have no effect in C++ [1].

[1]: https://rel

clang-tidy: Enable modernize-deprecated-headers check

Some headers from C library were deprecated in C++ and are no
longer welcome in C++ codebases. Some have no effect in C++ [1].

[1]: https://releases.llvm.org/13.0.1/tools/clang/tools/extra/docs/clang-tidy/checks/modernize-deprecated-headers.html

Change-Id: Ia3b1df10175e2e661c8fffb82e357c9db81b2e9c
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

show more ...

Format meson files with meson.format

Meson 1.5.0 introduced a new feature to format all the meson
files. Formatting all the meson files is now as simple as running
`meson format -i -r` command in th

Format meson files with meson.format

Meson 1.5.0 introduced a new feature to format all the meson
files. Formatting all the meson files is now as simple as running
`meson format -i -r` command in the repository root folder.

more details : https://mesonbuild.com/Commands.html#format

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

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

show more ...

PEL: Adding a few PELs

This commit adds some new PELs [1].

[1]: https://gerrit.openbmc.org/c/openbmc/phosphor-logging/+/73483

Change-Id: Ie2b46f966b4df423e7ba76d3507772259d5a8727
Signed-off-by: Ri

PEL: Adding a few PELs

This commit adds some new PELs [1].

[1]: https://gerrit.openbmc.org/c/openbmc/phosphor-logging/+/73483

Change-Id: Ie2b46f966b4df423e7ba76d3507772259d5a8727
Signed-off-by: Riya Dixit <riyadixitagra@gmail.com>

show more ...

oem: ibm: fileTable: Add new lids and update permissions

Add 2 new lids and update the permissions for hostboot's DEVTREE to be
read-write.

Change-Id: I95006e2b37d15a148f072f66804950e983ea1aab
Sign

oem: ibm: fileTable: Add new lids and update permissions

Add 2 new lids and update the permissions for hostboot's DEVTREE to be
read-write.

Change-Id: I95006e2b37d15a148f072f66804950e983ea1aab
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

oem: ibm: fileTable: Add lids for Blueridge and Fuji systems

Change-Id: Ic3d7d1b95b1b1e6559be15b95c5713b58e0b448d
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

pldm: Drop pessimizing moves

Fixes issues such as:

```
../host-bmc/host_pdr_handler.cpp:684:9: error: moving a temporary object prevents copy elision [-Werror,-Wpessimizing-move]
684 | st

pldm: Drop pessimizing moves

Fixes issues such as:

```
../host-bmc/host_pdr_handler.cpp:684:9: error: moving a temporary object prevents copy elision [-Werror,-Wpessimizing-move]
684 | std::move(std::vector<uint8_t>(1, PLDM_PDR_ENTITY_ASSOCIATION)),
| ^
../host-bmc/host_pdr_handler.cpp:684:9: note: remove std::move call here
684 | std::move(std::vector<uint8_t>(1, PLDM_PDR_ENTITY_ASSOCIATION)),
| ^~~~~~~~~~ ~
```

Change-Id: Id26f0d6fc21837e1eb76ae3c294c222782a4e69f
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

oem-ibm: requester: dbus_to_file_handler: Drop unused mctp_fd

```
../oem/ibm/requester/dbus_to_file_handler.hpp:83:9: error: private field 'mctp_fd' is not used [-Werror,-Wunused-private-field]
8

oem-ibm: requester: dbus_to_file_handler: Drop unused mctp_fd

```
../oem/ibm/requester/dbus_to_file_handler.hpp:83:9: error: private field 'mctp_fd' is not used [-Werror,-Wunused-private-field]
83 | int mctp_fd;
| ^
```

Change-Id: I03751463e91b5f62c1e60c1ef47c22dcfcc20215
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

oem-ibm: libpldmresponder: oem_ibm_handler: Remove unused `this`

```
../oem/ibm/libpldmresponder/oem_ibm_handler.hpp:101:14: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture

oem-ibm: libpldmresponder: oem_ibm_handler: Remove unused `this`

```
../oem/ibm/libpldmresponder/oem_ibm_handler.hpp:101:14: error: lambda capture 'this' is not used [-Werror,-Wunused-lambda-capture]
101 | [this](sdbusplus::message_t& msg) {
| ^~~~
```

Change-Id: I155758a25488d747eed2af0ac24407561ab3cc9f
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

oem-ibm: libpldmresponder: file_io_type_vpd: Remove unused vpdFileType

```
In file included from ../oem/ibm/libpldmresponder/file_io_type_vpd.cpp:1:
../oem/ibm/libpldmresponder/file_io_type_vpd.hpp:

oem-ibm: libpldmresponder: file_io_type_vpd: Remove unused vpdFileType

```
In file included from ../oem/ibm/libpldmresponder/file_io_type_vpd.cpp:1:
../oem/ibm/libpldmresponder/file_io_type_vpd.hpp:72:14: error: private field 'vpdFileType' is not used [-Werror,-Wunused-private-field]
72 | uint16_t vpdFileType; //!< type of the VPD file
| ^
```

Change-Id: If1208f21bcedaff912079371cf2e79df4b17093f
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

oem-ibm: libpldmresponder: file_io: Remove unused fields

```
../oem/ibm/libpldmresponder/file_io.hpp:418:9: error: private field 'hostSockFd' is not used [-Werror,-Wunused-private-field]
418 |

oem-ibm: libpldmresponder: file_io: Remove unused fields

```
../oem/ibm/libpldmresponder/file_io.hpp:418:9: error: private field 'hostSockFd' is not used [-Werror,-Wunused-private-field]
418 | int hostSockFd;
| ^
../oem/ibm/libpldmresponder/file_io.hpp:419:13: error: private field 'hostEid' is not used [-Werror,-Wunused-private-field]
419 | uint8_t hostEid;
| ^
../oem/ibm/libpldmresponder/file_io.hpp:420:25: error: private field 'instanceIdDb' is not used [-Werror,-Wunused-private-field]
420 | pldm::InstanceIdDb* instanceIdDb;
| ^
../oem/ibm/libpldmresponder/file_io.hpp:433:57: error: private field 'handler' is not used [-Werror,-Wunused-private-field]
433 | pldm::requester::Handler<pldm::requester::Request>* handler;
|
```

Change-Id: Ifd2e813423b7bc0bd67780c0fc21e32cf8de3e85
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

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

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

show more ...

Add support for the coreCount property in DBus

This commit introduces support for counting the number of CPU cores
during BMC power-on and populates this information to a DBus property
named coreCou

Add support for the coreCount property in DBus

This commit introduces support for counting the number of CPU cores
during BMC power-on and populates this information to a DBus property
named coreCount. Upon BMC power-on, the remote terminus detects the
number of CPU cores and send this data and PLDM will updates the
coreCount property accordingly.

Tested:
tested on simulator for hosting DBus property

Change-Id: I37adbe399414fcff3f089fb819349ca4bb537edd
Signed-off-by: Kamalkumar Patel <kamalkumar.patel@ibm.com>

show more ...

oem-ibm: Implement Host lamp test interface

IBM has a feature called LampTest and what it does is this:
- BMC would set the state of LEDs to [ON] on all the BMC accessible
LEDs.
- For LEDs t

oem-ibm: Implement Host lamp test interface

IBM has a feature called LampTest and what it does is this:
- BMC would set the state of LEDs to [ON] on all the BMC accessible
LEDs.
- For LEDs that are not accessible by BMC, a message is sent to the
Host, so that the Host can exercise all the LEDs on the connected
drawers and anything else that the BMC does not have access to.

This commit adds support to send message to Host when BMC has to tell
the Host to exercise LEDs accessible by host.

When Host sees the effector is set, it goes ahead and turns [ON] all
the LEDs for a specified time and automatically restores the states
at timer expiration. BMC does not have to tell the Host to stop the
test and it is part of Host's lamp test requirement.

TESTED: By setting the asserted property of the LED groups object

busctl set-property xyz.openbmc_project.LED.GroupManager
/xyz/openbmc_project/led/groups/lamp_test
xyz.openbmc_project.Led.Group Asserted b true

And verifying that the setEffecterStates call was made to the
remote state effecter to turn on all LEDs

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I990dce28d3017f20c73ff9029fced0e7ac84868f

show more ...