[PATCH 6/7] S3C6410: Add platform data for S3C IDE controller driver

Thomas Abraham thomas.ab at samsung.com
Mon Nov 2 01:18:00 EST 2009


On Sun, Nov 1, 2009 at 10:07 PM, Ben Dooks <ben-linux at fluff.org> wrote:
> On Sun, Nov 01, 2009 at 01:58:56PM +0900, Thomas Abraham wrote:
>> This patch adds platform data for S3C IDE controller driver in
>> S3C6410 platform.
>>
>> Signed-off-by: Abhilash Kesavan <a.kesavan at samsung.com>
>> Signed-off-by: Thomas Abraham <thomas.ab at samsung.com>
>> ---
>>  arch/arm/mach-s3c6410/mach-smdk6410.c |    8 +++++++-
>>  1 files changed, 7 insertions(+), 1 deletions(-)
>> -
>> +#ifdef CONFIG_BLK_DEV_IDE_S3C
>> +     &s3c_device_cfcon,
>> +#endif
>>       &smdk6410_smsc911x,
>>  };
>
> Firstly, doing stuff if the driver is enabled or not is not a good idea,
> as it means re-installing the kernel if you then go and build the driver
> as a module at a later date.
>
> Two, since I guess that the IDE gpio setup should be the same across all
> s3c6410 based systems, then you should look at supplying a standard
> platform data that the board can use instead of having to duplicate this
> across all the boards.
>

Ok. I will remove the conditional compilation and move the platform
data from SMDK board file. Since the IDE driver is not accepted, these
patches will be updated and resent again with the libata based driver.

Thanks Ben for reviewing the patches.

Thomas.



More information about the linux-arm-kernel mailing list