Hi Eric:
I agree with your suggestion.
One more exception about ahci on imx53 smd board.
one sata_pwr_en is used to control the pwr supply on smd board.
It's better to enable this pwer_en obviously although that this pin is
enabled reserved.

the ahci driver use the default sata platform data on ard and loco boards, and
 the smd platform data on smd board refer to the sata_pwr_en usage.

Best Regards
Richard Zhu

On 29 August 2011 16:47, Eric Miao [off-list ref] wrote:

On Mon, Aug 29, 2011 at 3:18 PM, Richard Zhu [off-list ref] wrote:

quoted

Signed-off-by: Richard Zhu <redacted>
---
?arch/arm/mach-mx5/board-mx53_ard.c | ? ?7 +++++++
?1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-mx5/board-mx53_ard.c b/arch/arm/mach-mx5/board-mx53_ard.c
index 76a67c4..d265cee 100644
--- a/arch/arm/mach-mx5/board-mx53_ard.c
+++ b/arch/arm/mach-mx5/board-mx53_ard.c

@@ -27,6 +27,7 @@

?#include <mach/common.h>
?#include <mach/hardware.h>
?#include <mach/iomux-mx53.h>
+#include <mach/ahci_sata.h>

?#include <asm/mach-types.h>
?#include <asm/mach/arch.h>

@@ -220,6 +221,11 @@ static struct platform_device *devices[] __initdata = {

? ? ? ?&ard_smsc_lan9220_device,
?};

+static struct ahci_platform_data sata_data = {
+ ? ? ? .init = sata_init,
+ ? ? ? .exit = sata_exit,
+};
+

As this platform data is same across most of these boards, I'd suggest we
keep a common/default platform_data in

arch/arm/plat-mxc/devices/platform-imx-ahci.c

And pass NULL in these board files, making use of the above default one.
For those boards with exceptions (e.g. special operations like GPIO to
enable/disable something at init/exit), one can still have platform data
and pass the pointer.

quoted

?static void __init mx53_ard_board_init(void)
?{
? ? ? ?imx53_soc_init();

@@ -234,6 +240,7 @@ static void __init mx53_ard_board_init(void)

? ? ? ?imx53_add_imx_i2c(1, &mx53_ard_i2c2_data);
? ? ? ?imx53_add_imx_i2c(2, &mx53_ard_i2c3_data);
? ? ? ?imx_add_gpio_keys(&ard_button_data);
+ ? ? ? imx53_add_ahci_imx(0, &sata_data);
?}

?static void __init mx53_ard_timer_init(void)
--
1.7.1



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel