All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] lm-sensors chip support
Date: Mon, 25 May 2009 16:09:47 +0000	[thread overview]
Message-ID: <20090525160947.GA5713@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <0331509C72A4CE4F925973DC8EC98E1B0165EC68@AMSACEX2.americas.ad.flextronics.com>

On Mon, May 25, 2009 at 03:58:36PM +0000, Jonathan Cameron wrote:

> > - POWR1014A (power monitor)

> That one does a whole lot more than would conventionally be supported by
> hwmon.  I'd guess you would want a multifunction device driver, with hwmon
> and pmic drivers hanging off it.  I'm not quite sure how to handle this
> one as, whilst designed to control supply voltages etc, at first glance
> it looks to simply be a dac (alongside all the hwmon style bits).  If you
> do want to support these elements I'd suggest a post to lkml to get opinions
> of the voltage and mfd users / maintainers.  In the meantime I've copied
> in Liam and Mark who may want to express an opinion on this.

Given that it looks like almost everything on the chip to do with power
control is handled by a CPLD it's probably not going to have any
regulator stuff that can go in a generic driver - it seems the intention
is more to sequence initial power up than for runtime power management.

Otherwise it looks like the control interface is fairly standard hwmon
stuff with some GPIOs as well.

> 
> > - SA56004EDP (temp sensor)
> I haven't checked this thoroughly but the data sheet refers to certain
> variants being address and pin compatible with the lm86 which is supported
> by the lm90 driver.  I'd suggest getting the data sheets for both and doing
> a comparison.  This one or the max6639 would probably be the best ones
> to try writing drivers for.
>  
> > I cross-checked these against the lm-sensors list of supported chips
> > (http://www.lm-sensors.org/wiki/Devices) and find that only the 1st one
> > is supported.  
> > 
> > Do you know if there are any plans to develop support for the latter 3
> > in the list?  Or, are any of these compatible with another chip that is
> > supported?  
> > 
> > If there are no plans and no compatibilities, is there documentation you
> > can point me to that describes a framework for adding a driver to be
> > supported by lm-sensors.  I'd like to consider this if this is not
> > difficult.  
> > 
> > As I understand it, there is no chip-specific code in lm-sensors; to add
> > support, one must add driver code into the kernel itself (ie. no code is
> > added to the lm-sensors domain).  Is this correct?  Is there an example
> > of a driver that was supported recently that I can use as an
> > example/framework?
> Basically browse the various drivers supported by the kernel until you find
> something close to what you are working on.  If it's close enough then try
> to add the support to the current driver.  If not try and replicate the way
> an existing driver is set up.
> 
> As for guidance, there is a lot of relevant documentation in Documentation
> directory of the kernel.
> 
> Have fun ;)
> 
> Jonathan

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

  parent reply	other threads:[~2009-05-25 16:09 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-25 15:17 [lm-sensors] lm-sensors chip support Rob Bloom
2009-05-25 15:58 ` Jonathan Cameron
2009-05-25 16:09 ` Mark Brown [this message]
2009-05-26  9:09 ` Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090525160947.GA5713@rakim.wolfsonmicro.main \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=lm-sensors@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.