Re: [PATCH v2] hwmon/mc13xxx-adc: add support for the MC13892 PMIC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 27 Jan 2012 20:48:58 +0100
Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:

> Hello Jean,
> 
> first of all thanks for your thoughts.
> 
> On Fri, Jan 27, 2012 at 05:53:29PM +0100, Jean Delvare wrote:
> > > diff --git a/Documentation/hwmon/mc13783-adc
> > > b/Documentation/hwmon/mc13783-adc index 044531a..8b717f5 100644
> > > --- a/Documentation/hwmon/mc13783-adc
> > > +++ b/Documentation/hwmon/mc13783-adc
> > > @@ -5,6 +5,9 @@ Supported chips:
> > >    * Freescale Atlas MC13783
> > >      Prefix: 'mc13783_adc'
> > >      Datasheet:
> > > http://www.freescale.com/files/rf_if/doc/data_sheet/MC13783.pdf?fsrch=1
> > > +  * Freescale Atlas MC13892
> > > +    Prefix: 'mc13892_adc'
> > > +    Datasheet:
> > > http://cache.freescale.com/files/analog/doc/data_sheet/MC13892.pdf?fsrch=1&sr=1 
> > >  Authors:
> > >      Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> > > @@ -13,20 +16,21 @@ Authors:
> > >  Description
> > >  -----------
> > >  
> > > -The Freescale MC13783 is a Power Management and Audio Circuit. Among
> > > -other things it contains a 10-bit A/D converter. The converter has 16
> > > -channels which can be used in different modes.
> > > -The A/D converter has a resolution of 2.25mV. Channels 0-4 have
> > > -a dedicated meaning with chip internal scaling applied. Channels 5-7
> > > -can be used as general purpose inputs or alternatively in a dedicated
> > > -mode. Channels 12-15 are occupied by the touchscreen if it's active.
> > > +The Freescale MC13783 and MC13892 are Power Management and Audio
> > > Circuits. +Among other things they contain a 10-bit A/D converter. The
> > > converter has 16 +(MC13783) resp. 12 (MC13892) channels which can be
> > > used in different modes. The +A/D converter has a resolution of 2.25mV.
> > >  
> > > -Currently the driver only supports channels 2 and 5-15 with no
> > > alternative -modes for channels 5-7.
> > > +Some channels can be used as General Purpose inputs or in a dedicated
> > > mode with +a chip internal scaling applied .
> > >  
> > > -See this table for the meaning of the different channels and their chip
> > > -internal scaling:
> > > +Currently the driver only supports BP, the General Purpose inputs and
> > > +touchscreen.
> > 
> > It might be a good idea to explain what BP is... Me, I have no idea.
> It's in the table below and means the "Application Supply" channel. No
> idea what the Freescale guys thought it should stand for.

"Battery Power". It is the "BP" power rail commonly found in Freescale
documentation, that is not actually connected directly to the battery....
almost. That's probably why calling it "Battery Power" seemed too misleading,
so they abbreviated it ;-) It basically powers the whole system.

Just my 2 cent ;-)

Thanks for picking this up finally.

Best regards,

-- 
David Jander

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors



[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux