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 <eric.miao@xxxxxxxxxx> wrote: > On Mon, Aug 29, 2011 at 3:18 PM, Richard Zhu <richard.zhu@xxxxxxxxxx> wrote: >> Signed-off-by: Richard Zhu <richard.zhu@xxxxxxxxxx> >> --- >> 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. > >> 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@xxxxxxxxxxxxxxxxxxx >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >> > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html