Dear Bartlomiej Zolnierkiewicz, On Mon, 07 Apr 2014 19:50:06 +0200, Bartlomiej Zolnierkiewicz wrote: > > +static struct platform_driver ahci_mvebu_driver = { > > + .probe = ahci_mvebu_probe, > > + .remove = ata_platform_remove_one, > > + .driver = { > > + .name = "ahci-mvebu", > > + .owner = THIS_MODULE, > > + .of_match_table = ahci_mvebu_of_match, > > Why is the Power Management support missing from this driver? Because as of today, the only platform that can use this driver is the Armada 38x, and we don't yet have suspend/resume support for this platform: we haven't written yet the suspend to RAM code for it. And I wanted to avoid adding suspend/resume code that I'm not able to test. So my plan is to get back to PM support once we have working suspend to RAM support on Armada 38x. > > +MODULE_DESCRIPTION("Marvell EBU AHCI SATA driver"); > > +MODULE_AUTHOR("Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxxxxxxxxx>, Marcin Wojtas <mw@xxxxxxxxxxxx>"); > > +MODULE_LICENSE("GPL"); > > +MODULE_ALIAS("platform:ahci_mv"); > > platform:ahci_mvebu ? Indeed, thanks for spotting. > Otherwise it looks good. Thanks for the review! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- 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