On Jun 13, 2014, at 9:38 AM, Stanimir Varbanov <svarbanov@xxxxxxxxxx> wrote: > Hi Kumar, > >> + >> +static struct platform_driver qcom_ahci_driver = { >> + .probe = qcom_ahci_probe, >> + .remove = ata_platform_remove_one, >> + .driver = { >> + .name = "qcom_ahci_qcom", >> + .owner = THIS_MODULE, >> + .of_match_table = qcom_ahci_of_match, >> + }, >> +}; >> +module_platform_driver(qcom_ahci_driver); >> + >> +MODULE_DESCRIPTION("Qualcomm AHCI SATA platform driver"); >> +MODULE_LICENSE("GPL"); > > "GPL v2" ? Or only GPL is intentional? should be v2, will fix > >> +MODULE_ALIAS("ahci:qcom"); >> > > missing MODULE_AUTHOR > > -- > regards, > Stan > -- > To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html