Re: [PATCH 1/2] Added support for Designware SATA controller driver

2009-05-06 Thread Feng Kan

Hi Scott:

I agree with your statement, however this driver is wrapped with this 
AHB DMA controller.
It would be very hard for it to work on non 460EX platforms. I can 
expand the depend in

the future if it is available on more cores.

Thanks
Feng Kan

Scott Wood wrote:

Feng Kan wrote:

This adds support for the Designware SATA controller.

Signed-off-by: Feng Kan f...@amcc.com
Signed-off-by: Mark Miesfeld miesf...@gmail.com
---
 drivers/ata/Kconfig|   10 +
 drivers/ata/Makefile   |1 +
 drivers/ata/sata_dwc.c | 2053 


 3 files changed, 2064 insertions(+), 0 deletions(-)
 create mode 100644 drivers/ata/sata_dwc.c

diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
index 0bcf264..c3d0b24 100644
--- a/drivers/ata/Kconfig
+++ b/drivers/ata/Kconfig
@@ -72,6 +72,16 @@ config SATA_FSL
 
   If unsure, say N.
 
+config SATA_DWC

+tristate DesignWare Cores SATA support
+ depends on 460EX


That depends looks too specific -- we don't want to grow a list if 
this controller gets added to other chips.


Only depend on what this driver actually needs in order to function.

-Scott


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


Re: [PATCH 1/2] Added support for Designware SATA controller driver

2009-05-01 Thread Scott Wood

Feng Kan wrote:

This adds support for the Designware SATA controller.

Signed-off-by: Feng Kan f...@amcc.com
Signed-off-by: Mark Miesfeld miesf...@gmail.com
---
 drivers/ata/Kconfig|   10 +
 drivers/ata/Makefile   |1 +
 drivers/ata/sata_dwc.c | 2053 
 3 files changed, 2064 insertions(+), 0 deletions(-)
 create mode 100644 drivers/ata/sata_dwc.c

diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
index 0bcf264..c3d0b24 100644
--- a/drivers/ata/Kconfig
+++ b/drivers/ata/Kconfig
@@ -72,6 +72,16 @@ config SATA_FSL
 
 	  If unsure, say N.
 
+config SATA_DWC

+   tristate DesignWare Cores SATA support
+   depends on 460EX


That depends looks too specific -- we don't want to grow a list if 
this controller gets added to other chips.


Only depend on what this driver actually needs in order to function.

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


Re: [PATCH 1/2] Added support for Designware SATA controller driver

2009-05-01 Thread Jeff Garzik

Wolfgang Denk wrote:

Dear Feng Kan,

In message 1241211767-32697-1-git-send-email-f...@amcc.com you wrote:

This adds support for the Designware SATA controller.

Signed-off-by: Feng Kan f...@amcc.com
Signed-off-by: Mark Miesfeld miesf...@gmail.com
---
 drivers/ata/Kconfig|   10 +
 drivers/ata/Makefile   |1 +
 drivers/ata/sata_dwc.c | 2053 
 3 files changed, 2064 insertions(+), 0 deletions(-)
 create mode 100644 drivers/ata/sata_dwc.c


...

+
+MODULE_LICENSE(GPL);
+MODULE_AUTHOR(Mark Miesfeld mmiesf...@amcc.com);
+MODULE_DESCRIPTION(DesignWare Cores SATA controller low lever driver);

^

Guess that's a typo?


heh



Hmm... that's 2000+ lines of code in a single file. Maybe this should
be split?


A single file is just fine.

Jeff


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