|
From: | Cédric Le Goater |
Subject: | Re: [PATCH v4 3/5] hw/nvram/eeprom_at24c: Add init_rom field and at24c_eeprom_init_rom helper |
Date: | Thu, 26 Jan 2023 18:19:22 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 |
On 1/26/23 17:23, Ninad S Palsule wrote:
Hi Cedric, Good suggestion but we will not be able to share those EEPROM image files yet. We are trying to figure out how to sanitize them.
For eeprom tests, they could contain any data with some well know pattern to check. It could/should be generated. No need to use real data. Real eeprom data would be for testing the rainier-bmc machine with a real firmware image, such as OpenBMC : https://jenkins.openbmc.org/job/ci-openbmc/distro=ubuntu,label=docker-builder,target=p10bmc/lastSuccessfulBuild/artifact/openbmc/build/tmp/deploy/images/p10bmc/ This is useful too but these are system level tests. Thanks, C.
On 1/26/23, 1:09 AM, "Cédric Le Goater" <clg@kaod.org <mailto:clg@kaod.org>> wrote: Hello Ninad, On 1/25/23 17:53, Ninad S Palsule wrote:Signed-off-by: Peter Delevoryas peter@pjd.dev <mailto:peter@pjd.dev> <mailto:peter@pjd.dev <mailto:peter@pjd.dev>> Reviewed-by: Joel Stanley joel@jms.id.au <mailto:joel@jms.id.au> <mailto:joel@jms.id.au <mailto:joel@jms.id.au>> Tested-by: Ninad Palsule ninadpalsule@us.ibm.com <mailto:ninadpalsule@us.ibm.com> <mailto:ninadpalsule@us.ibm.com <mailto:ninadpalsule@us.ibm.com>> Hi Peter, I applied your patches and made sure that different EEPROM images can be loaded from appropriate image files and it is working as expected.May be you could contribute an eeprom qtest ? I would put the data under tests/data/eeprom. Thanks, C.# Used following command to invoke the qemu. qemu-system-arm -M rainier-bmc -nographic \ -kernel fitImage-linux.bin \ -dtb aspeed-bmc-ibm-rainier.dtb \ -initrd obmc-phosphor-initramfs.rootfs.cpio.xz \ -drive file=obmc-phosphor-image.rootfs.wic.qcow2,if=sd,index=2 \ -append "rootwait console=ttyS4,115200n8 root=PARTLABEL=rofs-a" \ -device at24c-eeprom,bus=aspeed.i2c.bus.0,address=0x51,drive=a,rom-size=32768 -drive file=tpm.eeprom.bin,format=raw,if=none,id=a \ -device at24c-eeprom,bus=aspeed.i2c.bus.7,address=0x50,drive=b,rom-size=65536 -drive file=oppanel.eeprom.bin,format=raw,if=none,id=b \ -device at24c-eeprom,bus=aspeed.i2c.bus.7,address=0x51,drive=c,rom-size=65536 -drive file=lcd.eeprom.bin,format=raw,if=none,id=c \ -device at24c-eeprom,bus=aspeed.i2c.bus.8,address=0x50,drive=d,rom-size=65536 -drive file=baseboard.eeprom.bin,format=raw,if=none,id=d \ -device at24c-eeprom,bus=aspeed.i2c.bus.8,address=0x51,drive=e,rom-size=65536 -drive file=bmc.eeprom.bin,format=raw,if=none,id=e \ -device at24c-eeprom,bus=aspeed.i2c.bus.9,address=0x50,drive=f,rom-size=131072 -drive file=vrm.eeprom.bin,format=raw,if=none,id=f \ -device at24c-eeprom,bus=aspeed.i2c.bus.10,address=0x50,drive=g,rom-size=131072 -drive file=vrm.eeprom.bin,format=raw,if=none,id=g \ -device at24c-eeprom,bus=aspeed.i2c.bus.13,address=0x50,drive=h,rom-size=65536 -drive file=nvme.eeprom.bin,format=raw,if=none,id=h \ -device at24c-eeprom,bus=aspeed.i2c.bus.14,address=0x50,drive=i,rom-size=65536 -drive file=nvme.eeprom.bin,format=raw,if=none,id=i \ -device at24c-eeprom,bus=aspeed.i2c.bus.15,address=0x50,drive=j,rom-size=65536 -drive file=nvme.eeprom.bin,format=raw,if=none,id=j
[Prev in Thread] | Current Thread | [Next in Thread] |