Re: 2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-03 Thread Philippe Troin
Willy Tarreau <[EMAIL PROTECTED]> writes:

> On Wed, Mar 02, 2005 at 02:00:30PM -0800, Philippe Troin wrote:
>   
> > +   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
> > +   /* MBo April 2004 */
> > +   if( ((unsigned char)pAC->vpd.vpd_buf[0x3f] == 0x38) &&
> > +   ((unsigned char)pAC->vpd.vpd_buf[0x40] == 0x3c) &&
> > +   ((unsigned char)pAC->vpd.vpd_buf[0x41] == 0x45) ) {
> > +   printk("sk98lin : humm... Asus mainboard with buggy VPD ? 
> > correcting data.\n");
>   ^
> Please, could you put some KERN_XXX here to avoid a buggy message level ?

Yes, of course.

Phil.

Signed-Off-By: Philippe Troin <[EMAIL PROTECTED]>

diff -ruN linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c 
linux-2.4.29/drivers/net/sk98lin/skvpd.c
--- linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c   Wed Apr 14 06:05:30 2004
+++ linux-2.4.29/drivers/net/sk98lin/skvpd.cMon Feb 21 02:03:00 2005
@@ -466,6 +466,15 @@

pAC->vpd.vpd_size = vpd_size;
 
+   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
+   /* MBo April 2004 */
+   if( ((unsigned char)pAC->vpd.vpd_buf[0x3f] == 0x38) &&
+   ((unsigned char)pAC->vpd.vpd_buf[0x40] == 0x3c) &&
+   ((unsigned char)pAC->vpd.vpd_buf[0x41] == 0x45) ) {
+   printk(KERN_INFO "sk98lin : humm... Asus mainboard with buggy 
VPD ? correcting data.\n");
+   (unsigned char)pAC->vpd.vpd_buf[0x40] = 0x38;
+   }
+
/* find the end tag of the RO area */
if (!(r = vpd_find_para(pAC, VPD_RV, ))) {
SK_DBG_MSG(pAC, SK_DBGMOD_VPD, SK_DBGCAT_ERR | SK_DBGCAT_FATAL,
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: 2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-03 Thread Philippe Troin
Willy Tarreau [EMAIL PROTECTED] writes:

 On Wed, Mar 02, 2005 at 02:00:30PM -0800, Philippe Troin wrote:
   
  +   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
  +   /* MBo April 2004 */
  +   if( ((unsigned char)pAC-vpd.vpd_buf[0x3f] == 0x38) 
  +   ((unsigned char)pAC-vpd.vpd_buf[0x40] == 0x3c) 
  +   ((unsigned char)pAC-vpd.vpd_buf[0x41] == 0x45) ) {
  +   printk(sk98lin : humm... Asus mainboard with buggy VPD ? 
  correcting data.\n);
   ^
 Please, could you put some KERN_XXX here to avoid a buggy message level ?

Yes, of course.

Phil.

Signed-Off-By: Philippe Troin [EMAIL PROTECTED]

diff -ruN linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c 
linux-2.4.29/drivers/net/sk98lin/skvpd.c
--- linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c   Wed Apr 14 06:05:30 2004
+++ linux-2.4.29/drivers/net/sk98lin/skvpd.cMon Feb 21 02:03:00 2005
@@ -466,6 +466,15 @@

pAC-vpd.vpd_size = vpd_size;
 
+   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
+   /* MBo April 2004 */
+   if( ((unsigned char)pAC-vpd.vpd_buf[0x3f] == 0x38) 
+   ((unsigned char)pAC-vpd.vpd_buf[0x40] == 0x3c) 
+   ((unsigned char)pAC-vpd.vpd_buf[0x41] == 0x45) ) {
+   printk(KERN_INFO sk98lin : humm... Asus mainboard with buggy 
VPD ? correcting data.\n);
+   (unsigned char)pAC-vpd.vpd_buf[0x40] = 0x38;
+   }
+
/* find the end tag of the RO area */
if (!(r = vpd_find_para(pAC, VPD_RV, rp))) {
SK_DBG_MSG(pAC, SK_DBGMOD_VPD, SK_DBGCAT_ERR | SK_DBGCAT_FATAL,
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: 2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-02 Thread Willy Tarreau
On Wed, Mar 02, 2005 at 02:00:30PM -0800, Philippe Troin wrote:
  
> + /* Asus K8V Se Deluxe bugfix. Correct VPD content */
> + /* MBo April 2004 */
> + if( ((unsigned char)pAC->vpd.vpd_buf[0x3f] == 0x38) &&
> + ((unsigned char)pAC->vpd.vpd_buf[0x40] == 0x3c) &&
> + ((unsigned char)pAC->vpd.vpd_buf[0x41] == 0x45) ) {
> + printk("sk98lin : humm... Asus mainboard with buggy VPD ? 
> correcting data.\n");
  ^
Please, could you put some KERN_XXX here to avoid a buggy message level ?

Willy

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-02 Thread Philippe Troin
The EEPROM (or whatever that is) on Asus K8V SE Deluxe motherboards
contains buggy firmware.  This buggy firmware has one flipped bit, and
causes the sk98lin driver refuses to work correctly.  Please look at
this thread:

  http://www.ussg.iu.edu/hypermail/linux/kernel/0404.0/1439.html

It contains a patch for 2.6 that fixs the problem.  Enclosed is a copy
of this patch for 2.4.29.  Please consider applying.

Phil.

Signed-Off-By: Philippe Troin <[EMAIL PROTECTED]>

diff -ruN linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c 
linux-2.4.29/drivers/net/sk98lin/skvpd.c
--- linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c   Wed Apr 14 06:05:30 2004
+++ linux-2.4.29/drivers/net/sk98lin/skvpd.cMon Feb 21 02:03:00 2005
@@ -466,6 +466,15 @@

pAC->vpd.vpd_size = vpd_size;
 
+   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
+   /* MBo April 2004 */
+   if( ((unsigned char)pAC->vpd.vpd_buf[0x3f] == 0x38) &&
+   ((unsigned char)pAC->vpd.vpd_buf[0x40] == 0x3c) &&
+   ((unsigned char)pAC->vpd.vpd_buf[0x41] == 0x45) ) {
+   printk("sk98lin : humm... Asus mainboard with buggy VPD ? 
correcting data.\n");
+   (unsigned char)pAC->vpd.vpd_buf[0x40] = 0x38;
+   }
+
/* find the end tag of the RO area */
if (!(r = vpd_find_para(pAC, VPD_RV, ))) {
SK_DBG_MSG(pAC, SK_DBGMOD_VPD, SK_DBGCAT_ERR | SK_DBGCAT_FATAL,
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-02 Thread Philippe Troin
The EEPROM (or whatever that is) on Asus K8V SE Deluxe motherboards
contains buggy firmware.  This buggy firmware has one flipped bit, and
causes the sk98lin driver refuses to work correctly.  Please look at
this thread:

  http://www.ussg.iu.edu/hypermail/linux/kernel/0404.0/1439.html

It contains a patch for 2.6 that fixs the problem.  Enclosed is a copy
of this patch for 2.4.29.  Please consider applying.

Phil.

Signed-Off-By: Philippe Troin [EMAIL PROTECTED]

diff -ruN linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c 
linux-2.4.29/drivers/net/sk98lin/skvpd.c
--- linux-2.4.29.orig/drivers/net/sk98lin/skvpd.c   Wed Apr 14 06:05:30 2004
+++ linux-2.4.29/drivers/net/sk98lin/skvpd.cMon Feb 21 02:03:00 2005
@@ -466,6 +466,15 @@

pAC-vpd.vpd_size = vpd_size;
 
+   /* Asus K8V Se Deluxe bugfix. Correct VPD content */
+   /* MBo April 2004 */
+   if( ((unsigned char)pAC-vpd.vpd_buf[0x3f] == 0x38) 
+   ((unsigned char)pAC-vpd.vpd_buf[0x40] == 0x3c) 
+   ((unsigned char)pAC-vpd.vpd_buf[0x41] == 0x45) ) {
+   printk(sk98lin : humm... Asus mainboard with buggy VPD ? 
correcting data.\n);
+   (unsigned char)pAC-vpd.vpd_buf[0x40] = 0x38;
+   }
+
/* find the end tag of the RO area */
if (!(r = vpd_find_para(pAC, VPD_RV, rp))) {
SK_DBG_MSG(pAC, SK_DBGMOD_VPD, SK_DBGCAT_ERR | SK_DBGCAT_FATAL,
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: 2.4.29 sk98lin patch for Asus K8W SE Deluxe

2005-03-02 Thread Willy Tarreau
On Wed, Mar 02, 2005 at 02:00:30PM -0800, Philippe Troin wrote:
  
 + /* Asus K8V Se Deluxe bugfix. Correct VPD content */
 + /* MBo April 2004 */
 + if( ((unsigned char)pAC-vpd.vpd_buf[0x3f] == 0x38) 
 + ((unsigned char)pAC-vpd.vpd_buf[0x40] == 0x3c) 
 + ((unsigned char)pAC-vpd.vpd_buf[0x41] == 0x45) ) {
 + printk(sk98lin : humm... Asus mainboard with buggy VPD ? 
 correcting data.\n);
  ^
Please, could you put some KERN_XXX here to avoid a buggy message level ?

Willy

-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/