* [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
@ 2009-06-26 8:10 Andre Prendel
2009-06-26 10:02 ` Hans de Goede
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: Andre Prendel @ 2009-06-26 8:10 UTC (permalink / raw)
To: lm-sensors
Hello Jean, Hans,
Here is an updated version of the tmp421 driver. I've fixed the issues
according to your review. Please take a closer look at the
is_visible() callback of the sysfs group. I'm using the attributes' index
to decide whether the file is visible or not.
Jean, I know you're on vacation, so take your time :)
BTW I spend some time with my girlfriend at the Cote d'Azur last
year. Probably the best vacation I've ever had. But speaking french is
very difficult :)
Thanks,
Andre
---
Documentation/hwmon/tmp421 | 36 ++++
drivers/hwmon/Kconfig | 10 +
drivers/hwmon/Makefile | 1
drivers/hwmon/tmp421.c | 349 +++++++++++++++++++++++++++++++++++++++++++++
4 files changed, 396 insertions(+)
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
@ 2009-06-26 10:02 ` Hans de Goede
2009-06-28 12:25 ` Jean Delvare
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Hans de Goede @ 2009-06-26 10:02 UTC (permalink / raw)
To: lm-sensors
Hi,
On 06/26/2009 10:10 AM, Andre Prendel wrote:
> Hello Jean, Hans,
>
> Here is an updated version of the tmp421 driver. I've fixed the issues
> according to your review. Please take a closer look at the
> is_visible() callback of the sysfs group. I'm using the attributes' index
> to decide whether the file is visible or not.
>
looks good to me,
Regards,
Hans
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
2009-06-26 10:02 ` Hans de Goede
@ 2009-06-28 12:25 ` Jean Delvare
2009-06-29 8:54 ` Andre Prendel
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Jean Delvare @ 2009-06-28 12:25 UTC (permalink / raw)
To: lm-sensors
On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> Hello Jean, Hans,
>
> Here is an updated version of the tmp421 driver. I've fixed the issues
> according to your review. Please take a closer look at the
> is_visible() callback of the sysfs group. I'm using the attributes' index
> to decide whether the file is visible or not.
>
> Jean, I know you're on vacation, so take your time :)
Now that the wiki is back, please update the TMP421 etc. entry.
> BTW I spend some time with my girlfriend at the Cote d'Azur last
> year. Probably the best vacation I've ever had. But speaking french is
> very difficult :)
Yes French is indeed very difficult :( but we have beautiful landscapes
and excellent cuisine :)
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
2009-06-26 10:02 ` Hans de Goede
2009-06-28 12:25 ` Jean Delvare
@ 2009-06-29 8:54 ` Andre Prendel
2009-06-29 16:48 ` Jean Delvare
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Andre Prendel @ 2009-06-29 8:54 UTC (permalink / raw)
To: lm-sensors
On Sun, Jun 28, 2009 at 02:25:51PM +0200, Jean Delvare wrote:
> On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> > Hello Jean, Hans,
> >
> > Here is an updated version of the tmp421 driver. I've fixed the issues
> > according to your review. Please take a closer look at the
> > is_visible() callback of the sysfs group. I'm using the attributes' index
> > to decide whether the file is visible or not.
> >
> > Jean, I know you're on vacation, so take your time :)
>
> Now that the wiki is back, please update the TMP421 etc. entry.
Is the driver scheduled for 2.6.32 or the next rc?
> > BTW I spend some time with my girlfriend at the Cote d'Azur last
> > year. Probably the best vacation I've ever had. But speaking french is
> > very difficult :)
>
> Yes French is indeed very difficult :( but we have beautiful landscapes
> and excellent cuisine :)
You're right :)
Thanks,
Andre
> --
> Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
` (2 preceding siblings ...)
2009-06-29 8:54 ` Andre Prendel
@ 2009-06-29 16:48 ` Jean Delvare
2009-06-30 8:12 ` Andre Prendel
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Jean Delvare @ 2009-06-29 16:48 UTC (permalink / raw)
To: lm-sensors
On Mon, 29 Jun 2009 10:54:10 +0200, Andre Prendel wrote:
> On Sun, Jun 28, 2009 at 02:25:51PM +0200, Jean Delvare wrote:
> > On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> > > Hello Jean, Hans,
> > >
> > > Here is an updated version of the tmp421 driver. I've fixed the issues
> > > according to your review. Please take a closer look at the
> > > is_visible() callback of the sysfs group. I'm using the attributes' index
> > > to decide whether the file is visible or not.
> > >
> > > Jean, I know you're on vacation, so take your time :)
> >
> > Now that the wiki is back, please update the TMP421 etc. entry.
>
> Is the driver scheduled for 2.6.32 or the next rc?
2.6.32.
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
` (3 preceding siblings ...)
2009-06-29 16:48 ` Jean Delvare
@ 2009-06-30 8:12 ` Andre Prendel
2009-06-30 8:17 ` Jean Delvare
2009-06-30 8:39 ` Andre Prendel
6 siblings, 0 replies; 8+ messages in thread
From: Andre Prendel @ 2009-06-30 8:12 UTC (permalink / raw)
To: lm-sensors
On Mon, Jun 29, 2009 at 06:48:29PM +0200, Jean Delvare wrote:
> On Mon, 29 Jun 2009 10:54:10 +0200, Andre Prendel wrote:
> > On Sun, Jun 28, 2009 at 02:25:51PM +0200, Jean Delvare wrote:
> > > On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> > > > Hello Jean, Hans,
> > > >
> > > > Here is an updated version of the tmp421 driver. I've fixed the issues
> > > > according to your review. Please take a closer look at the
> > > > is_visible() callback of the sysfs group. I'm using the attributes' index
> > > > to decide whether the file is visible or not.
> > > >
> > > > Jean, I know you're on vacation, so take your time :)
> > >
> > > Now that the wiki is back, please update the TMP421 etc. entry.
Done.
> > Is the driver scheduled for 2.6.32 or the next rc?
>
> 2.6.32.
Thanks,
Andre
> --
> Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
` (4 preceding siblings ...)
2009-06-30 8:12 ` Andre Prendel
@ 2009-06-30 8:17 ` Jean Delvare
2009-06-30 8:39 ` Andre Prendel
6 siblings, 0 replies; 8+ messages in thread
From: Jean Delvare @ 2009-06-30 8:17 UTC (permalink / raw)
To: lm-sensors
On Tue, 30 Jun 2009 10:12:20 +0200, Andre Prendel wrote:
> On Mon, Jun 29, 2009 at 06:48:29PM +0200, Jean Delvare wrote:
> > On Mon, 29 Jun 2009 10:54:10 +0200, Andre Prendel wrote:
> > > On Sun, Jun 28, 2009 at 02:25:51PM +0200, Jean Delvare wrote:
> > > > On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> > > > > Hello Jean, Hans,
> > > > >
> > > > > Here is an updated version of the tmp421 driver. I've fixed the issues
> > > > > according to your review. Please take a closer look at the
> > > > > is_visible() callback of the sysfs group. I'm using the attributes' index
> > > > > to decide whether the file is visible or not.
> > > > >
> > > > > Jean, I know you're on vacation, so take your time :)
> > > >
> > > > Now that the wiki is back, please update the TMP421 etc. entry.
>
> Done.
Thanks. Care to add support to sensors-detect too?
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
` (5 preceding siblings ...)
2009-06-30 8:17 ` Jean Delvare
@ 2009-06-30 8:39 ` Andre Prendel
6 siblings, 0 replies; 8+ messages in thread
From: Andre Prendel @ 2009-06-30 8:39 UTC (permalink / raw)
To: lm-sensors
On Tue, Jun 30, 2009 at 10:17:40AM +0200, Jean Delvare wrote:
> On Tue, 30 Jun 2009 10:12:20 +0200, Andre Prendel wrote:
> > On Mon, Jun 29, 2009 at 06:48:29PM +0200, Jean Delvare wrote:
> > > On Mon, 29 Jun 2009 10:54:10 +0200, Andre Prendel wrote:
> > > > On Sun, Jun 28, 2009 at 02:25:51PM +0200, Jean Delvare wrote:
> > > > > On Fri, 26 Jun 2009 10:10:31 +0200, Andre Prendel wrote:
> > > > > > Hello Jean, Hans,
> > > > > >
> > > > > > Here is an updated version of the tmp421 driver. I've fixed the issues
> > > > > > according to your review. Please take a closer look at the
> > > > > > is_visible() callback of the sysfs group. I'm using the attributes' index
> > > > > > to decide whether the file is visible or not.
> > > > > >
> > > > > > Jean, I know you're on vacation, so take your time :)
> > > > >
> > > > > Now that the wiki is back, please update the TMP421 etc. entry.
> >
> > Done.
>
> Thanks. Care to add support to sensors-detect too?
I will have a look at it. But as I said, no time at the moment. I will
move 550km back to my hometown for a new job.
> --
> Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2009-06-30 8:39 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-06-26 8:10 [lm-sensors] [PATCH v2 0/2] hwmon: Add support for TI's Andre Prendel
2009-06-26 10:02 ` Hans de Goede
2009-06-28 12:25 ` Jean Delvare
2009-06-29 8:54 ` Andre Prendel
2009-06-29 16:48 ` Jean Delvare
2009-06-30 8:12 ` Andre Prendel
2009-06-30 8:17 ` Jean Delvare
2009-06-30 8:39 ` Andre Prendel
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.