[2.6 patch] sata_fsl.c: fix 8315DS workaround

2008-08-04 Thread Adrian Bunk
Commit e7eac96e8f0e57a6e9f94943557bc2b23be31471 (ata/sata_fsl: Move 
MPC8315DS link speed limit workaround to specific ifdef) aimed at 
limiting the workaround only to the affected hardware, but since the 
#ifdef used a nonexisting kconfig variable it actually killed
the workaround.

Reported-by: Robert P. J. Day [EMAIL PROTECTED]
Signed-off-by: Adrian Bunk [EMAIL PROTECTED]

---
cd54dd8b6a8bca44ead212d12fe116702cc31ed7 
diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c
index 3924e72..45878ef 100644
--- a/drivers/ata/sata_fsl.c
+++ b/drivers/ata/sata_fsl.c
@@ -640,7 +640,7 @@ static int sata_fsl_port_start(struct ata_port *ap)
VPRINTK(HControl = 0x%x\n, ioread32(hcr_base + HCONTROL));
VPRINTK(CHBA  = 0x%x\n, ioread32(hcr_base + CHBA));
 
-#ifdef CONFIG_MPC8315_DS
+#ifdef CONFIG_MPC831x_RDB
/*
 * Workaround for 8315DS board 3gbps link-up issue,
 * currently limit SATA port to GEN1 speed

___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [2.6 patch] sata_fsl.c: fix 8315DS workaround

2008-08-04 Thread Adrian Bunk
On Mon, Aug 04, 2008 at 04:55:37PM +0800, Li Yang wrote:
  -Original Message-
  From: Adrian Bunk [mailto:[EMAIL PROTECTED] 
  Sent: Monday, August 04, 2008 4:46 PM
  To: Kalra Ashish; Li Yang; Jeff Garzik; [EMAIL PROTECTED]
  Cc: linuxppc-dev@ozlabs.org; [EMAIL PROTECTED]; 
  Robert P. J. Day
  Subject: [2.6 patch] sata_fsl.c: fix 8315DS workaround
  
  Commit e7eac96e8f0e57a6e9f94943557bc2b23be31471 
  (ata/sata_fsl: Move MPC8315DS link speed limit workaround to 
  specific ifdef) aimed at limiting the workaround only to the 
  affected hardware, but since the #ifdef used a nonexisting 
  kconfig variable it actually killed the workaround.
 
 This workaround is only for MPC8315_DS board which is not supported in the 
 main line yet.  We can remove the workaround if we need to do something here.

Thanks, in this case my patch is complete crap.

There's no need for any action.

 - Leo

cu
Adrian

-- 

   Is there not promise of rain? Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   Only a promise, Lao Er said.
   Pearl S. Buck - Dragon Seed

___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


RE: [2.6 patch] sata_fsl.c: fix 8315DS workaround

2008-08-04 Thread Li Yang
 -Original Message-
 From: Adrian Bunk [mailto:[EMAIL PROTECTED] 
 Sent: Monday, August 04, 2008 4:46 PM
 To: Kalra Ashish; Li Yang; Jeff Garzik; [EMAIL PROTECTED]
 Cc: linuxppc-dev@ozlabs.org; [EMAIL PROTECTED]; 
 Robert P. J. Day
 Subject: [2.6 patch] sata_fsl.c: fix 8315DS workaround
 
 Commit e7eac96e8f0e57a6e9f94943557bc2b23be31471 
 (ata/sata_fsl: Move MPC8315DS link speed limit workaround to 
 specific ifdef) aimed at limiting the workaround only to the 
 affected hardware, but since the #ifdef used a nonexisting 
 kconfig variable it actually killed the workaround.

This workaround is only for MPC8315_DS board which is not supported in the main 
line yet.  We can remove the workaround if we need to do something here.

- Leo

 
 Reported-by: Robert P. J. Day [EMAIL PROTECTED]
 Signed-off-by: Adrian Bunk [EMAIL PROTECTED]
 
 ---
 cd54dd8b6a8bca44ead212d12fe116702cc31ed7
 diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c 
 index 3924e72..45878ef 100644
 --- a/drivers/ata/sata_fsl.c
 +++ b/drivers/ata/sata_fsl.c
 @@ -640,7 +640,7 @@ static int sata_fsl_port_start(struct 
 ata_port *ap)
   VPRINTK(HControl = 0x%x\n, ioread32(hcr_base + HCONTROL));
   VPRINTK(CHBA  = 0x%x\n, ioread32(hcr_base + CHBA));
  
 -#ifdef CONFIG_MPC8315_DS
 +#ifdef CONFIG_MPC831x_RDB
   /*
* Workaround for 8315DS board 3gbps link-up issue,
* currently limit SATA port to GEN1 speed
 
 
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev