[PATCH v6 1/1] ata: ahci_xgene: Add AHCI Support for 2nd HW version of APM X-Gene SoC AHCI SATA Host controller.

Suman Tripathi stripathi at apm.com
Mon May 11 06:31:32 PDT 2015


Hi Tejun,

On Mon, May 11, 2015 at 4:58 PM, Tejun Heo <tj at kernel.org> wrote:
> Hello, Suman.
>
> On Mon, May 11, 2015 at 02:06:15PM +0530, Suman Tripathi wrote:
>> -#ifdef CONFIG_ACPI
>> -static const struct acpi_device_id xgene_ahci_acpi_match[] = {
>> -     { "APMC0D0D", },
>> -     { }
>> -};
>> -MODULE_DEVICE_TABLE(acpi, xgene_ahci_acpi_match);
>> -#endif
>> -
>> -static const struct of_device_id xgene_ahci_of_match[] = {
>> -     {.compatible = "apm,xgene-ahci"},
>> -     {},
>> -};
>> -MODULE_DEVICE_TABLE(of, xgene_ahci_of_match);
>
> I applied the patch as-is this time but please don't combine code
> movement with actual changes in the future.  Split them out to
> separate patches.  It makes the changes a lot easier to follow and
> verify.

Sorry . I will take care of it next time onwards.
>
> Thanks.
>
> --
> tejun



-- 
Thanks,
with regards,
Suman Tripathi



More information about the linux-arm-kernel mailing list