[lm-sensors] [PATCH 02/10] w83627ehf: Add voltage inputs support

[greg@kroah.com (Greg KH)]

On Sat, Mar 25, 2006 at 01:04:36PM -0500, Mark M. Hoffman wrote:
> Hi all:
> 
> * Jean Delvare <khali at linux-fr.org> [2006-03-23 16:25:22 +0100]:
> > From: Rudolf Marek <r.marek at sh.cvut.cz>
> > Content-Disposition: inline; filename=hwmon-w83627ehf-add-voltages.patch
> > 
> > Add the voltage measuring support to W83627EHF. The code is based
> > on the patch provided by Yuan Mu from Winbond.
> > 
> 
> [snip]
> 
> > +static inline u8 in_to_reg(u32 val, u8 nr)
> > +{
> > +	return SENSORS_LIMIT(((val + (scale_in[nr] / 2)) / scale_in[nr]), 0, 255);
> > +}
> 
> This reminds me... it seems that static inlines (outside of header files)
> are discouraged now - see chapter 14 of CodingStyle.  It would be pointless
> to take them all out of drivers/hwmon/*.c now, but we probably shouldn't
> add any more of them.

Yeah, especially for stuff like this that it doesn't make sense to
inline as there's no speed benifit, and only code bloat :)

thanks,

greg k-h