[PATCH v1 0/2] cmd/fru: move FRU handling support to common region

Michal Simek michal.simek at amd.com
Mon Aug 1 12:29:07 CEST 2022



On 7/29/22 23:54, Jae Hyun Yoo wrote:
> Hello,
> 
> The FRU handling was added as a Xilinx board dependent support but it
> would be useful for other boards too, 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 board info field parsing,
> it defines 'fru_parse_board_custom' as a weak function so that it can
> be replaced with the board specific implementation. In the same way,
> OEM Multirecord type (0xc0 - 0xff) parsing logic can be replaced with
> a board specific 'fru_parse_multirec' implementation.
> 
> Also, this series adds 'Product Info' parsing support.
> 
> Please review!

In general I am fine with this but I want this to be done in steps to be able to 
better review it.
It means couple of preparation patches before this is moved to generic location.
Moving that part of xilinx private structure of board info as one step, 
multirecord OEM entries another one, etc.

Thanks,
Michal


More information about the U-Boot mailing list