2011/1/6 Jeff Garzik [off-list ref]:

On 01/05/2011 12:43 AM, mkl0301 at gmail.com wrote:

quoted

From: Mac Lin<redacted>

v2:
?- Switch ahci_platform to module device table matching to add SoC
specific support

v1: http://www.spinics.net/lists/arm-kernel/msg106236.html
?- Add CNS3xxx SoC specific AHCI support

This patchset is based on linux-2.6.37-rc2

Mac Lin (3):
? ? ? ahci_platform: rename to ahci_pltfm, but keep the original module
name
? ? ? ahci_pltfm: switch to module device table matching
? ? ? ahci_platform: add support for CNS3xxx SoC devices

?arch/arm/mach-cns3xxx/devices.c | ? ?2 +-
?drivers/ata/Kconfig ? ? ? ? ? ? | ? 11 ++
?drivers/ata/Makefile ? ? ? ? ? ?| ? ?5 +-
?drivers/ata/ahci_cns3xxx.c ? ? ?| ? 62 +++++++++++
?drivers/ata/ahci_platform.c ? ? | ?197
------------------------------------
?drivers/ata/ahci_pltfm.c ? ? ? ?| ?212
+++++++++++++++++++++++++++++++++++++++
?drivers/ata/ahci_pltfm.h ? ? ? ?| ? 19 ++++
?7 files changed, 309 insertions(+), 199 deletions(-)
?create mode 100644 drivers/ata/ahci_cns3xxx.c
?delete mode 100644 drivers/ata/ahci_platform.c
?create mode 100644 drivers/ata/ahci_pltfm.c
?create mode 100644 drivers/ata/ahci_pltfm.h

It is overkill to rename the entirety of ahci_platform just for one override
function.
This sort of thing I would have expected to be added directly to
ahci_platform.c.

It might be overkill for only one controller. but it is more clean and
readable to have different SoC specific changes in separate files,
especially when more SoCs need to make similar changes.

I will add them directly to ahci_platform.c if you insist.

Best Regards,
Mac Lin