[PATCH v4 0/6] cmd/fru: move FRU handling support to common region

Michal Simek michal.simek at amd.com
Thu Sep 15 16:38:31 CEST 2022



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


More information about the U-Boot mailing list