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: I0e0749db217667d9c3f51f8fac6cb58e8915e8d2
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-format: update latest spec and reformat

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

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

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

show more ...

strgfnhandler: suppress unused parameter warnings

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

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

show more ...

Fix offset size

Signed-off-by: Jean-Marie Verdun <jean-marie.verdun@hpe.com>
Change-Id: Ic6128e836c3add71737022b9bfb4c96713f0772c

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: Idab99ed8fdb9bdf5cc0e130d576f55d256df9cce

show more ...

Convert host-ipmid -> libipmid

phosphor-host-ipmid now exposes a library along with headers for
interfacing with the ipmi daemon. Compile and link against the new
library.

Change-Id: Ib222df1605afe

Convert host-ipmid -> libipmid

phosphor-host-ipmid now exposes a library along with headers for
interfacing with the ipmi daemon. Compile and link against the new
library.

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

show more ...

strgfnhandler: rename function per openbmc style guide

Function names should be lower camel and not snake case.

Change-Id: I6f8683850ce017e63641415846e8a0f031fccba7
Signed-off-by: Patrick Venture <

strgfnhandler: rename function per openbmc style guide

Function names should be lower camel and not snake case.

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

show more ...

strgfnhandler: rename all variables per openbmc style

OpenBMC style declares variables be lower camel and not snake case.

Change-Id: I4a448b84ae3d930049d4cb761216459799a698fa
Signed-off-by: Patrick

strgfnhandler: rename all variables per openbmc style

OpenBMC style declares variables be lower camel and not snake case.

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

show more ...

update: use sdbusplus instead of sd_bus raw pointers

Update code to use sdbusplus instead of raw sd_bus pointers.

Change-Id: I80cd4492480824827c27fbf19eb54487e28d3b75
Signed-off-by: Patrick Venture

update: use sdbusplus instead of sd_bus raw pointers

Update code to use sdbusplus instead of raw sd_bus pointers.

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

show more ...

style: apply naming convention to ipmi_validate_fru_area

Apply openbmc style naming convention to ipmi_validate_fru_area as part
of a multi-step cleanup process.

Change-Id: I082189030441e57579d0725

style: apply naming convention to ipmi_validate_fru_area

Apply openbmc style naming convention to ipmi_validate_fru_area as part
of a multi-step cleanup process.

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

show more ...

strgfnhandler: swap in log<> calls, cleanup c calls

Transition debug and error output to use log<> mechanism.
Transition c-library or compiler choice calls to cpp libraries.

Change-Id: Iafdf71122e9

strgfnhandler: swap in log<> calls, cleanup c calls

Transition debug and error output to use log<> mechanism.
Transition c-library or compiler choice calls to cpp libraries.

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

show more ...

strgfnhandler: fixup c-style cast

[strgfnhandler.cpp:29]: (style) C-style pointer casting

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

add clang-format

Add clang-format file.

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

Add privilege levels for commands

Add privilege level and package the provider library into
/usr/lib/ipmid-providers

Change-Id: I33be3ab696ba477a938b1c0272119420353d6d99
Signed-off-by: Tom <tomjose

Add privilege levels for commands

Add privilege level and package the provider library into
/usr/lib/ipmid-providers

Change-Id: I33be3ab696ba477a938b1c0272119420353d6d99
Signed-off-by: Tom <tomjoseph@in.ibm.com>

show more ...

Change H->hpp & C->cpp

Change-Id: I4bb8cf7a58c517f348a524d3e027ebcd1dcd0dea
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>