On Fri, 26 Feb 2016 15:52:40 -0800
David Hendricks <dhend...@google.com> wrote:

> > +#define GIGADEVICE_GD25VQ21B   0x4212
> > +#define GIGADEVICE_GD25VQ41B   0x4213  /* Same as GD25VQ40C, can be 
> > distinguished by SFDP */  

> This comment should go in the .c file, above the model_id for the
> GD25VQ40C and GD25VQ41B.

The only real reason for having flashchips.h is having a nice
overview of IDs and relevant information thus that comment is perfectly
fine there - it would get lost in the shuffle in the .c file.
We have lots of similar comments in the .h file, most prominently the
one for the GIGADEVICE_GD25Q128 that the above was copied from ;)

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner

_______________________________________________
flashrom mailing list
flashrom@flashrom.org
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to