On Tue, Mar 20, 2018 at 05:18:47PM +0100, Wolfram Sang wrote: > I used this driver as a template for a custom one. While hacking on my own, I > noticed some problems in this driver, too. This series fixes the first set of > them, related to the "rom-size" parameter. It fixes a segfault. > > I think the first patch is clearly suitable for stable. I think the second one, > too, but not as clearly. The third one is a cleanup and not for stable. Still, > I am open for opinions about these thoughts. Through which tree should these patches go? get_maintainer doesn't list a person specifically. Anything else I could do? > > Thanks, > > Wolfram > > Changes since v2: > > * removed '\n' from error_report-strings > * made sure checkpatch is happy > * added tags from Philippe (thanks!) > > Changes since v1: > > * reordered patches according to significance for stable > * use AT24C_ROMSIZE_DEFAULT instead of magic value > * patch 3 doesn't improve the ERR macro anymore but replaces > it completely with error_report(). > > > Wolfram Sang (3): > nvram: at24c: prevent segfault by checking "rom-size" > nvram: at24c: use a sane default for "rom-size" > nvram: at24c: use standard error reporting > > hw/nvram/eeprom_at24c.c | 25 ++++++++++++++----------- > 1 file changed, 14 insertions(+), 11 deletions(-) > > -- > 2.11.0 >
Attachment:
signature.asc
Description: PGP signature
