[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