[Patch] winbond w83627DHG superio support in superiotool

[bxshi]

This patch is for winbond w83627DHG superio support in superiotool . I have
test that on my board, it works ;)
Thanks for Uwe to help me make a SVN diff patch.

signed-off-by : Bingxun Shi

Bingxun Shi

[Patch] winbond w83627DHG superio support in superiotool

[bxshi]

Sorry, I may forget the attachment.

Bingxun Shi

2008/1/18, bxshi :
> This patch is for winbond w83627DHG superio support in superiotool . I
> have test that on my board, it works ;)
> Thanks for Uwe to help me make a SVN diff patch.
>
> signed-off-by : Bingxun Shi
>
> Bingxun Shi

[Patch] winbond w83627DHG superio support in superiotool

[Peter Stuge]

Hi,

On Fri, Jan 18, 2008 at 05:11:15PM +0800, bxshi wrote:
> 2008/1/18, bxshi :
> > This patch is for winbond w83627DHG superio support in superiotool . I
> > have test that on my board, it works ;)
> > Thanks for Uwe to help me make a SVN diff patch.

Looks good!
> > signed-off-by : Bingxun Shi

Acked-by: Peter Stuge
> Index: winbond.c
>
> --- winbond.c	(Revision 3051)
> +++ winbond.c	(Arbeitskopie)
> @@ -143,6 +143,57 @@
>    	{EOT}}},
>  #endif
>    {0xa02, "W83627DHG", {
> {NOLDN, NULL,
> 	{0x02,0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x27,0x28,
> 	 0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
> 	{0x00,0xa0,NANA,0xff,0x00,MISC,0x00,MISC,RSVD,0x50,
> 	 0x00,0x00,RSVD,0xe2,0x21,0x00,0x00,EOT}},
> {0x0, "Floppy",
> 	{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,0xf2,0xf4,0xf5,
> 	 EOT},
> 	{0x01,0x03,0xf0,0x06,0x02,0x8e,0x00,0xff,0x00,0x00,
> 	 EOT}},
> {0x1, "Parallel port",
> 	{0x30,0x60,0x61,0x70,0x74,0xf0,EOT},
> 	{0x01,0x03,0x78,0x07,0x04,0x3f,EOT}},
> {0x2, "COM1",
> 	{0x30,0x60,0x61,0x70,0xf0,EOT},
> 	{0x01,0x03,0xf8,0x04,0x00,EOT}},
> {0x3, "COM2",
> 	{0x30,0x60,0x61,0x70,0xf0,0xf1,EOT},
> 	{0x01,0x02,0xf8,0x03,0x00,0x00,EOT}},
> {0x5, "Keyboard",
> 	{0x30,0x60,0x61,0x62,0x63,0x70,0x72,0xf0,EOT},
> 	{0x01,0x00,0x60,0x00,0x64,0x01,0x0c,0x83,EOT}},
> {0x6, "Serial peripheral interface",
> 	{0x30,0x62,0x63,EOT},
> 	{0x00,0x00,0x00,EOT}},
> {0x7, "GPIO 6",
> 	{0x30,0xf4,0xf5,0xf6,0xf7,EOT},
> 	{0x00,0xff,0x00,0x00,0x00,EOT}},
> {0x8, "WDTO#, PLED",
> 	{0x30,0xf5,0xf6,0xf7,EOT},
> 	{0x00,0x00,0x00,0x00,EOT}},
> {0x9, "GPIO 2, GPIO 3, GPIO 4, GPIO 5",
> 	{0x30,0xe0,0xe1,0xe2,0xe3,0xe4,0xe5,0xe6,0xe7,0xe8,
> 	 0xe9,0xf0,0xf1,0xf2,0xf3,0xf4,0xf5,0xf6,0xf7,0xfe,
> 	 EOT},
> 	{0x00,0xff,0x00,0x00,0xff,0x00,0x00,0x00,0x00,0x00,
> 	 0x00,0xff,0x00,0x00,0x00,0xff,0x00,0x00,0x00,0x00,
> 	 EOT}},
> {0xa, "ACPI",
> 	{0x30,0x70,0xe0,0xe1,0xe2,0xe3,0xe4,0xe5,0xe6,0xe7,
> 	 0xe8,0xe9,0xf2,0xf3,0xf4,0xf6,0xf7,0xfe,EOT},
> 	{0x00,0x00,0x01,0x00,0xff,0x08,0x00,RSVD,0x1c,0x00,
> 	 RSVD,RSVD,0x7c,0x00,0x00,0x00,0x00,0x00,EOT}},
> {0xb, "Hardware monitor",
> 	{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf2,EOT},
> 	{0x00,0x00,0x00,0x00,0x81,0x00,0x00,EOT}},
> {0xc, "PECI, SST",
> 	{0xe0,0xe1,0xe2,0xe3,0xe4,0xe5,0xe8,0xf1,0xfe,0xff,
> 	 EOT},
> 	{0x00,0x48,0x48,0x48,0x48,0x00,0x00,0x48,0x00,0x00,
> 	 EOT}},
> {EOT}}},
> {0x886, "W83627EHF/EF/EHG/EG", { /* sensors-detect: 0x8853 possible */
> {NOLDN, NULL,

[Patch] winbond w83627DHG superio support in superiotool

[Carl-Daniel Hailfinger]

On 18.01.2008 15:39, Peter Stuge wrote:
> Hi,
>
> On Fri, Jan 18, 2008 at 05:11:15PM +0800, bxshi wrote:
> > 2008/1/18, bxshi :
> > > This patch is for winbond w83627DHG superio support in superiotool . I
> > > have test that on my board, it works ;)
> > > Thanks for Uwe to help me make a SVN diff patch.
>
> Looks good!
> > > signed-off-by : Bingxun Shi
>
> Acked-by: Peter Stuge

Acked-by: Carl-Daniel Hailfinger

Thanks, committed in r3062.

@Peter: Sorry, I forgot to include your Acked-by: line.

Regards,
Carl-Daniel

[Patch] winbond w83627DHG superio support in superiotool

[Uwe Hermann]

Hi,

On Sat, Jan 19, 2008 at 01:35:17AM +0100, Carl-Daniel Hailfinger wrote:
> > > > signed-off-by : Bingxun Shi
>
> Thanks, committed in r3062.

As this is now committed, can you please post your 'superiotool -d'
output from the board (which one btw?) here for archival purposes?

Thanks, Uwe.

[Patch] winbond w83627DHG superio support in superiotool

[bxshi]

> As this is now committed, can you please post your 'superiotool -d'
> output from the board (which one btw?) here for archival purposes?

Ok, I will post an output file tomorrow. The board is an video embeded
system based on intel x86 structure. It is kdm2860(KEDACOM).

Thanks
bxshi

[Patch] winbond w83627DHG superio support in superiotool

[bxshi]

> As this is now committed, can you please post your 'superiotool -d'
> output from the board

OK, here is the output file .

bxshi

[Patch] winbond w83627DHG superio support in superiotool

[Uwe Hermann]

On Mon, Jan 21, 2008 at 10:07:02AM +0800, bxshi wrote:
> OK, here is the output file .
>
> bxshi
> superiotool r
> Found Winbond W83627DHG (id=0xa0, rev=0x23) at 0x4e
> Register dump:
> idx 02 20 21 22 23 24 25 26  27 28 29 2a 2b 2c 2d 2e  2f
> val ff a0 23 ff 00 40 00 40  ff 50 80 02 00 03 21 00  ff
> def 00 a0 NA ff 00 MM 00 MM  RR 50 00 00 RR e2 21 00  00

Thanks, I added a link to the wiki.

Uwe.