On 9/15/22 16:01, Jae Hyun Yoo wrote:
Gentle ping for this series.
On 8/25/2022 9:42 AM, Jae Hyun Yoo wrote:
Hello,
The FRU handling was added as a Xilinx board dependent support but it is also useful for other boards, so this commit moves the FRU handling support to the common region so that it can be enabled by CONFIG_CMD_FRU.
To provide manufacturer specific custom info fields and multi-records parsing, it refactors the FRU handling logic using linked list so that each board support can utilize them in their own custom way. This series adds 'Product Info' parsing support, usage document and unit test script too.
Please review!
Thanks, Jae
Graeme Gregory (1): cmd: fru: move FRU handling support to common region
Jae Hyun Yoo (5): xilinx: common: refactor FRU handling support cmd: fru: fix a sandbox segfault issue cmd: fru: add product info area parsing support doc: fru: add documentation for the fru command and APIs test: cmd: fru: add unit test for the fru command
board/xilinx/Kconfig | 8 - board/xilinx/common/Makefile | 3 - board/xilinx/common/board.c | 68 ++- board/xilinx/common/fru.h | 108 ----- board/xilinx/common/fru_ops.c | 415 ----------------- cmd/Kconfig | 8 + cmd/Makefile | 1 + {board/xilinx/common => cmd}/fru.c | 54 ++- doc/usage/cmd/fru.rst | 144 ++++++ doc/usage/index.rst | 1 + include/fru.h | 328 +++++++++++++ include/test/suites.h | 1 + lib/Makefile | 1 + lib/fru_ops.c | 724 +++++++++++++++++++++++++++++ test/cmd/Makefile | 1 + test/cmd/fru.c | 84 ++++ test/cmd_ut.c | 6 + 17 files changed, 1398 insertions(+), 557 deletions(-) delete mode 100644 board/xilinx/common/fru.h delete mode 100644 board/xilinx/common/fru_ops.c rename {board/xilinx/common => cmd}/fru.c (50%) create mode 100644 doc/usage/cmd/fru.rst create mode 100644 include/fru.h create mode 100644 lib/fru_ops.c create mode 100644 test/cmd/fru.c
Will look soon. Was busy with other stuff. Sorry for delay.
M