[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] nvram: at24c: use a sane default for "rom-s
From: |
Wolfram Sang |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] nvram: at24c: use a sane default for "rom-size" |
Date: |
Tue, 13 Mar 2018 21:16:49 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
Hi Philippe,
> > static Property at24c_eeprom_props[] = {
> > - DEFINE_PROP_UINT32("rom-size", EEPROMState, rsize, 0),
> > + DEFINE_PROP_UINT32("rom-size", EEPROMState, rsize, 128),
>
> This patch should goes before your 2/3 in your series.
I don't mind much, but why? My reasoning was "let's first fix the cause
and then the symptom"?
> Can you add a #define for this value? Such AT24C_ROMSIZE_MIN.
Can do, of course. But won't that give room for regressions because
people are already using it with lower values?
Ideally, we would have a "model" variable. The model type would define
the size of the memory. The "rom-size" variable could then be kept as is
(except for the 0 bugfix) or deprecated?
Thanks for the review,
Wolfram
signature.asc
Description: PGP signature
[Qemu-devel] [PATCH 1/3] nvram: at24c: remove doubled prefix for ERR, Wolfram Sang, 2018/03/12
[Qemu-devel] [PATCH 2/3] nvram: at24c: prevent segfault by checking "rom-size", Wolfram Sang, 2018/03/12