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

[Jae Hyun Yoo]

Hi Michal,

On 8/1/2022 3:29 AM, Michal Simek wrote:
> 
> 
> 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.

Sure, I'll split them into individual patches in v2.

Thanks,
Jae