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