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
>