public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <guenter.roeck@ericsson.com>
To: "Stijn Devriendt (sdevrien)" <sdevrien@cisco.com>
Cc: "khali@linux-fr.org" <khali@linux-fr.org>,
	"lm-sensors@lm-sensors.org" <lm-sensors@lm-sensors.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3] Add support for the Philips SA56004 temperature sensor.
Date: Fri, 24 Jun 2011 06:49:14 -0700	[thread overview]
Message-ID: <20110624134914.GA1177@ericsson.com> (raw)
In-Reply-To: <6E4D2678AC543844917CA081C9D6B33F04C74F28@XMB-AMS-103.cisco.com>

On Fri, Jun 24, 2011 at 06:40:57AM -0400, Stijn Devriendt (sdevrien) wrote:
> Hi Guenter,
> 
> Is anything being done with this patch, yet?
> I haven't heard anything from Jean so far.
> 
Hi Stijn,

please don't top-post.

I don't recall a comment from Jean either.

Jean, any comments ? It is your driver, so I thought it would make sense
if you had a look.

Thanks,
Guenter

> Thanks,
> Stijn
> 
> > -----Original Message-----
> > From: Guenter Roeck [mailto:guenter.roeck@ericsson.com]
> > Sent: maandag 6 juni 2011 14:41
> > To: Stijn Devriendt (sdevrien)
> > Cc: khali@linux-fr.org; lm-sensors@lm-sensors.org; linux-
> > kernel@vger.kernel.org
> > Subject: Re: [PATCH v3] Add support for the Philips SA56004
> temperature
> > sensor.
> > 
> > On Mon, Jun 06, 2011 at 06:40:45AM -0400, sdevrien@cisco.com wrote:
> > > From: Stijn Devriendt <sdevrien@cisco.com>
> > >
> > > Add support for the Philips SA56004, an LM86
> > > compatible temperature sensor.
> > >
> > > Changes since v2:
> > > - Changed BUG_ON to WARN_ON
> > > Changes since v1:
> > > - Updated documentation
> > > - Trace replaced by BUG_ON
> > > - style updates
> > >
> > > Signed-off-by: Stijn Devriendt <sdevrien@cisco.com>
> > 
> > Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
> > 
> > Jean, do you want to take this and/or have a look, or should I ?
> > 
> > Thanks,
> > Guenter

  reply	other threads:[~2011-06-24 13:50 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-06 10:40 [PATCH v3] Add support for the Philips SA56004 temperature sensor sdevrien
2011-06-06 12:40 ` Guenter Roeck
2011-06-24 10:40   ` Stijn Devriendt (sdevrien)
2011-06-24 13:49     ` Guenter Roeck [this message]
2011-06-26 19:54 ` Fubo Chen
2011-06-26 20:20   ` Guenter Roeck
2011-06-27  7:04   ` Stijn Devriendt (sdevrien)
2011-06-28 17:50     ` Fubo Chen
2011-06-28 18:19       ` Guenter Roeck
2011-07-07 11:22   ` Jean Delvare
2011-06-26 20:26 ` [v3] " Guenter Roeck
2011-07-07 11:28   ` Jean Delvare

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=20110624134914.GA1177@ericsson.com \
    --to=guenter.roeck@ericsson.com \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=sdevrien@cisco.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox