All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: Axel Lin <axel.lin@gmail.com>
Cc: Dirk Eibach <eibach@gdsys.de>,
	David George <david.george@ska.ac.za>,
	Guillaume Ligneul <guillaume.ligneul@gmail.com>,
	Riku Voipio <riku.voipio@iki.fi>,
	"Hans J. Koch" <hjk@hansjkoch.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Mark M. Hoffman" <mhoffman@lightlink.com>,
	Steve Glendinning <steve.glendinning@smsc.com>,
	"lm-sensors@lm-sensors.org" <lm-sensors@lm-sensors.org>
Subject: Re: [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use
Date: Sun, 27 Nov 2011 17:32:04 +0000	[thread overview]
Message-ID: <20111127173204.GA31171@ericsson.com> (raw)
In-Reply-To: <20111125165708.GA11317@ericsson.com>

On Fri, Nov 25, 2011 at 11:57:08AM -0500, Guenter Roeck wrote:
> On Fri, Nov 25, 2011 at 02:06:27AM -0500, Axel Lin wrote:
> > This patch converts the drivers in drivers/hwmon/* to use the
> > module_i2c_driver() macro which makes the code smaller and a bit
> > simpler.
> > 
> > Cc: Corentin Labbe <corentin.labbe@geomatys.fr>
> > Cc: Dirk Eibach <eibach@gdsys.de>
> > Cc: "Mark M. Hoffman" <mhoffman@lightlink.com>
> > Cc: Steve Glendinning <steve.glendinning@smsc.com>
> > Cc: Riku Voipio <riku.voipio@iki.fi>
> > Cc: Guillaume Ligneul <guillaume.ligneul@gmail.com>
> > Cc: David George <david.george@ska.ac.za>
> > Cc: "Hans J. Koch" <hjk@hansjkoch.de>
> > Cc: Marc Hulsman <m.hulsman@tudelft.nl>
> > Cc: Rudolf Marek <r.marek@assembler.cz>
> > Signed-off-by: Axel Lin <axel.lin@gmail.com>
> 
> Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
> 
> What is the plan for those patches ? Are they expected to all go through the same tree,
> or through individual trees after the necessary defines are introduced into mainline ?
> 
My question was answered in another thread related to the misc subsystem.
The patch this one depends on will only be available in 3.3-rc1. It would have been
nice to let us know ahead of time, because I _did_ find out the hard way.

I would suggest to resubmit the i2c and spi patches against a working baseline
once it is available, to make sure it does not miss some new drivers.

On a side note, this patch does miss the drivers in hwmon/pmbus, but I can take care
of those myself.

Guenter

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

WARNING: multiple messages have this Message-ID (diff)
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: Axel Lin <axel.lin@gmail.com>
Cc: Dirk Eibach <eibach@gdsys.de>,
	David George <david.george@ska.ac.za>,
	Guillaume Ligneul <guillaume.ligneul@gmail.com>,
	Riku Voipio <riku.voipio@iki.fi>,
	"Hans J. Koch" <hjk@hansjkoch.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Mark M. Hoffman" <mhoffman@lightlink.com>,
	Steve Glendinning <steve.glendinning@smsc.com>,
	"lm-sensors@lm-sensors.org" <lm-sensors@lm-sensors.org>
Subject: Re: [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use module_i2c_driver()
Date: Sun, 27 Nov 2011 09:32:04 -0800	[thread overview]
Message-ID: <20111127173204.GA31171@ericsson.com> (raw)
In-Reply-To: <20111125165708.GA11317@ericsson.com>

On Fri, Nov 25, 2011 at 11:57:08AM -0500, Guenter Roeck wrote:
> On Fri, Nov 25, 2011 at 02:06:27AM -0500, Axel Lin wrote:
> > This patch converts the drivers in drivers/hwmon/* to use the
> > module_i2c_driver() macro which makes the code smaller and a bit
> > simpler.
> > 
> > Cc: Corentin Labbe <corentin.labbe@geomatys.fr>
> > Cc: Dirk Eibach <eibach@gdsys.de>
> > Cc: "Mark M. Hoffman" <mhoffman@lightlink.com>
> > Cc: Steve Glendinning <steve.glendinning@smsc.com>
> > Cc: Riku Voipio <riku.voipio@iki.fi>
> > Cc: Guillaume Ligneul <guillaume.ligneul@gmail.com>
> > Cc: David George <david.george@ska.ac.za>
> > Cc: "Hans J. Koch" <hjk@hansjkoch.de>
> > Cc: Marc Hulsman <m.hulsman@tudelft.nl>
> > Cc: Rudolf Marek <r.marek@assembler.cz>
> > Signed-off-by: Axel Lin <axel.lin@gmail.com>
> 
> Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
> 
> What is the plan for those patches ? Are they expected to all go through the same tree,
> or through individual trees after the necessary defines are introduced into mainline ?
> 
My question was answered in another thread related to the misc subsystem.
The patch this one depends on will only be available in 3.3-rc1. It would have been
nice to let us know ahead of time, because I _did_ find out the hard way.

I would suggest to resubmit the i2c and spi patches against a working baseline
once it is available, to make sure it does not miss some new drivers.

On a side note, this patch does miss the drivers in hwmon/pmbus, but I can take care
of those myself.

Guenter

  reply	other threads:[~2011-11-27 17:32 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-25  7:06 [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Axel Lin
2011-11-25  7:06 ` [PATCH] hwmon: convert drivers/hwmon/* to use module_i2c_driver() Axel Lin
2011-11-25  7:12 ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Axel Lin
2011-11-25  7:12   ` [PATCH] hwmon: convert drivers/hwmon/* to use module_spi_driver() Axel Lin
2011-11-25  9:01   ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Zhang, Sonic
2011-11-25  9:01     ` [PATCH] hwmon: convert drivers/hwmon/* to use module_spi_driver() Zhang, Sonic
2011-11-27 17:48   ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Guenter Roeck
2011-11-27 17:48     ` [PATCH] hwmon: convert drivers/hwmon/* to use module_spi_driver() Guenter Roeck
2011-11-25  7:31 ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Axel Lin
2011-11-25  7:31   ` [PATCH] hwmon: convert drivers/hwmon/* to use module_platform_driver() Axel Lin
2011-11-25 11:03   ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Mark Brown
2011-11-25 11:03     ` [PATCH] hwmon: convert drivers/hwmon/* to use module_platform_driver() Mark Brown
2011-11-27  1:37   ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Guenter Roeck
2011-11-27  1:37     ` [PATCH] hwmon: convert drivers/hwmon/* to use module_platform_driver() Guenter Roeck
2011-11-25 16:57 ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Guenter Roeck
2011-11-25 16:57   ` [PATCH] hwmon: convert drivers/hwmon/* to use module_i2c_driver() Guenter Roeck
2011-11-27 17:32   ` Guenter Roeck [this message]
2011-11-27 17:32     ` [lm-sensors] " Guenter Roeck
2011-11-27 23:58     ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use Axel Lin
2011-11-27 23:58       ` [lm-sensors] [PATCH] hwmon: convert drivers/hwmon/* to use module_i2c_driver() Axel Lin

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=20111127173204.GA31171@ericsson.com \
    --to=guenter.roeck@ericsson.com \
    --cc=axel.lin@gmail.com \
    --cc=david.george@ska.ac.za \
    --cc=eibach@gdsys.de \
    --cc=guillaume.ligneul@gmail.com \
    --cc=hjk@hansjkoch.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=mhoffman@lightlink.com \
    --cc=riku.voipio@iki.fi \
    --cc=steve.glendinning@smsc.com \
    /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.