Prev: remove q->prepare_flush_fn hook
Next: block: remove q->prepare_flush_fn completely
From: 宾西蒙 on 3 Jul 2010 06:20
在 2010-07-03六的 10:35 +0200，Jean Delvare写道：
> Hi Guenter,
>
> On Fri, 2 Jul 2010 02:23:26 -0700, Guenter Roeck wrote:
> > + w83667hg-i 9 5 3 3 0xb340 0xc1 0x5ca3
>
> As discussed before, this one is a 677 series, not 667. That being
> said, I admit that the device ID proximity between the 667HG-B and the
> 677HG-I is troubling, and invites to optimism as far as compatibility
> between these two chips is concerned.

I think we got something confused here, I was looking for the W836
*seven* 7 HG-I driver (which has ID 0xb470) and not the W836 *six* 7
HG-I one, although maybe someone else might benefit from that one.

It has at least 5 fans, with 3 pwm and 1 voltage regulatory. Using the
w83667hg driver, I can only see the speed of fan2 and fan3 in sensors
output. the w83627dhg-p already seems to read out 3 of the 5 fan speeds.
The temperatures seem rather wonky, I might need to compare with the
bios readings some more.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Guenter Roeck on 3 Jul 2010 11:20
On Sat, Jul 03, 2010 at 04:35:36AM -0400, Jean Delvare wrote:
[...]
>
> Other than this, I have no objection, your patch looks sane.
>
> After your next iteration, I'll publish a standalone driver with your
> patch, for easier testing.
>
I submitted v3 a minute ago. I had to make a couple of additional changes,
so you'll have to at least browse through it again. This is still a single
patch. There are some generic changes, though, so I plan to split it into
a generic patch and a -B specific patch next time around. Would be great
to get some test feedback first, though.

Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Guenter Roeck on 3 Jul 2010 17:30
On Sat, Jul 03, 2010 at 04:56:13PM -0400, Jean Delvare wrote:
> Hi 宾西蒙,
>
> On Sat, 03 Jul 2010 12:15:48 +0200, 宾西蒙 wrote:
> > 在 2010-07-03六的 10:35 +0200，Jean Delvare写道：
> > > Hi Guenter,
> > >
> > > On Fri, 2 Jul 2010 02:23:26 -0700, Guenter Roeck wrote:
> > > > + w83667hg-i 9 5 3 3 0xb340 0xc1 0x5ca3
> > >
> > > As discussed before, this one is a 677 series, not 667. That being
> > > said, I admit that the device ID proximity between the 667HG-B and the
> > > 677HG-I is troubling, and invites to optimism as far as compatibility
> > > between these two chips is concerned.
> >
> > I think we got something confused here, I was looking for the W836
> > *seven* 7 HG-I driver (which has ID 0xb470) and not the W836 *six* 7
> > HG-I one, although maybe someone else might benefit from that one.
>
> Yes, there was some confusion. For one thing, the W83667HG-I does not
> exist. And for another, I have no idea where Guenter found the device
> id 0xb340. None of the Nuvoton W836x7HG datasheets I have mention this
> ID. All I know of is 0xb350 for the W83667HG-B and 0xb470 for the
> W83677HG-I (which apparently was renamed to something different
> meanwhile.)
>
> --
> Jean Delvare

Most likely I just got it wrong.

Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: remove q->prepare_flush_fn hook
Next: block: remove q->prepare_flush_fn completely