Re: [PATCH aspeed-5.2] aspeed: g220a-bmc: Add an FRU

[Cédric Le Goater]

On 12/10/20 11:36 AM, John Wang wrote:
> Add an eeprom device and fill it with fru
> information
> 
> $ ipmitool fru print 0
> Product Manufacturer  : Bytedance
> Product Name          : G220A
> 
> Signed-off-by: John Wang <wangzhiqiang.bj@bytedance.com>


Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks,

C. 

> ---
>  hw/arm/aspeed.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
> index 04c8ad2bcd..fc80d45513 100644
> --- a/hw/arm/aspeed.c
> +++ b/hw/arm/aspeed.c
> @@ -595,6 +595,18 @@ static void g220a_bmc_i2c_init(AspeedMachineState *bmc)
>      object_property_set_int(OBJECT(dev), "temperature0", 31000, 
> &error_abort);
>      object_property_set_int(OBJECT(dev), "temperature1", 28000, 
> &error_abort);
>      object_property_set_int(OBJECT(dev), "temperature2", 20000, 
> &error_abort);
> +
> +    static uint8_t eeprom_buf[2 * 1024] = {
> +            0x01, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0xfe,
> +            0x01, 0x06, 0x00, 0xc9, 0x42, 0x79, 0x74, 0x65,
> +            0x64, 0x61, 0x6e, 0x63, 0x65, 0xc5, 0x47, 0x32,
> +            0x32, 0x30, 0x41, 0xc4, 0x41, 0x41, 0x42, 0x42,
> +            0xc4, 0x43, 0x43, 0x44, 0x44, 0xc4, 0x45, 0x45,
> +            0x46, 0x46, 0xc4, 0x48, 0x48, 0x47, 0x47, 0xc1,
> +            0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xa7,
> +    };
> +    smbus_eeprom_init_one(aspeed_i2c_get_bus(&soc->i2c, 4), 0x57,
> +                          eeprom_buf);
>  }
>  
>  static bool aspeed_get_mmio_exec(Object *obj, Error **errp)
>