[PATCH v2] hwmon: (ina2xx) Add device tree support to pass the shunt resistor

Guenter Roeck linux at roeck-us.net
Thu Jun 20 12:40:18 EST 2013


On Thu, Jun 20, 2013 at 02:27:18AM +0000, Tang Yuantian-B29983 wrote:
> 
> 
> > -----Original Message-----
> > From: Guenter Roeck [mailto:linux at roeck-us.net]
> > Sent: 2013年6月19日 星期三 23:43
> > To: Tang Yuantian-B29983
> > Cc: khali at linux-fr.org; lm-sensors at lm-sensors.org; devicetree-
> > discuss at lists.ozlabs.org; linux-doc at vger.kernel.org
> > Subject: Re: [PATCH v2] hwmon: (ina2xx) Add device tree support to pass
> > the shunt resistor
> > 
> > On Wed, Jun 19, 2013 at 02:50:20PM +0800, Yuantian.Tang at freescale.com
> > wrote:
> > > From: Tang Yuantian <yuantian.tang at freescale.com>
> > >
> > > Adding another way that is device tree to pass the shunt resistor
> > > value to driver except for platform data.
> > >
> > > Signed-off-by: Tang Yuantian <Yuantian.Tang at freescale.com>
> > 
> > I noticed that of.h include was missing. No need to re-send; I added it
> > myself.
> > Applied to -next.
> > 
> 
> Thanks for your help. Of.h is already included in i2c.h.
> If it is really needed, I should have seen the warning or error when compile.
> Including it explicitly is also a good point.
> 
Common rule is to include a file if its definitions are used. We can not rely
on some other file including it for us, since it is not guaranteed that this
will always be the case.

Thanks,
Guenter


More information about the devicetree-discuss mailing list