[flashrom] [7/7, RFC] Unify usbdev_status and pcidev_status into dev_status.

Stefan Tauner stefan.tauner at student.tuwien.ac.at
Tue Dec 25 12:41:18 CET 2012


On Fri, 16 Nov 2012 22:48:22 +0200
Kyösti Mälkki <kyosti.malkki at gmail.com> wrote:

> > diff --git a/programmer.h b/programmer.h
> > index 9f90fe5..b52727e 100644
> > --- a/programmer.h
> > +++ b/programmer.h
> > @@ -93,15 +93,22 @@ enum programmer_type {
> >  	OTHER,
> >  };
> >  
> > +struct dev_status {
> > +	uint16_t vendor_id;
> > +	uint16_t device_id;
> > +	const enum test_state status;
> > +	const char *vendor_name;
> > +	const char *device_name;
> > +};  
> 
> I find the name "dev_status" odd, these are the arrays for listing
> and probing the VID/DID too, and the "status" is just documentation.

i missed this before sending version 5.0 of my patches. you do have a
point. any suggestions? "dev" might be ok, but it begs for conflicts.

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




More information about the flashrom mailing list