[lm-sensors] [PATCH v3] hwmon: Add submitting-patches checklist to documentation
Guenter Roeck
guenter.roeck at ericsson.com
Thu Apr 7 16:05:06 CEST 2011
On Thu, Apr 07, 2011 at 04:12:23AM -0400, Jean Delvare wrote:
> On Wed, 6 Apr 2011 10:17:53 -0700, Guenter Roeck wrote:
> > When writing hardware monitoring drivers, there are some common pitfalls which
> > keep coming up in code reviews. This patch provides a document describing all
> > those pitfalls and how to avoid them.
> >
> > Signed-off-by: Guenter Roeck <guenter.roeck at ericsson.com>
>
> Acked-by: Jean Delvare <khali at linux-fr.org>
>
> Minor suggestion below - but no need to resend.
>
> > +* Provide a detect function only if a chip can be detected reliably. On the
> > + other side, if the chip can be detected reliably, provide a detect function.
>
> Could be expressed in a more compact way as:
>
> * Provide a detect function if and only if a chip can be detected reliably.
>
Mathematically correct, but would people understand it ?
> (But maybe this is the mathematician in me speaking!)
>
Mathematician myself, I prefer your wording as well. I'll go with it.
Thanks,
Guenter
More information about the lm-sensors
mailing list