diff mbox

ata: pata_macio: Fix module autoload for OF platform driver

Message ID 20150915154552.GA3060@goodgumbo.baconseed.org
State Not Applicable
Delegated to: David Miller
Headers show

Commit Message

Luis de Bethencourt Sept. 15, 2015, 3:45 p.m. UTC
This platform driver has a OF device ID table but the OF module
alias information is not created so module autoloading won't work.

Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
---

Hello,

This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
that information so modules have the correct aliases built-in and
autoloading works correctly.

A longer explanation by Javier Canillas can be found here:
https://lkml.org/lkml/2015/7/30/519

Thanks,
Luis

 drivers/ata/pata_macio.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Tejun Heo Sept. 15, 2015, 3:52 p.m. UTC | #1
On Tue, Sep 15, 2015 at 05:45:52PM +0200, Luis de Bethencourt wrote:
> This platform driver has a OF device ID table but the OF module
> alias information is not created so module autoloading won't work.
> 
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> ---
> 
> Hello,
> 
> This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
> that information so modules have the correct aliases built-in and
> autoloading works correctly.
> 
> A longer explanation by Javier Canillas can be found here:
> https://lkml.org/lkml/2015/7/30/519

Can you please include that in the patch description?  Or at least add
Link: tag?  Also, should this be routed through -stable?

Thanks.
Luis de Bethencourt Sept. 15, 2015, 4:35 p.m. UTC | #2
On Tue, Sep 15, 2015 at 11:52:10AM -0400, Tejun Heo wrote:
> On Tue, Sep 15, 2015 at 05:45:52PM +0200, Luis de Bethencourt wrote:
> > This platform driver has a OF device ID table but the OF module
> > alias information is not created so module autoloading won't work.
> > 
> > Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> > ---
> > 
> > Hello,
> > 
> > This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
> > that information so modules have the correct aliases built-in and
> > autoloading works correctly.
> > 
> > A longer explanation by Javier Canillas can be found here:
> > https://lkml.org/lkml/2015/7/30/519
> 
> Can you please include that in the patch description?  Or at least add
> Link: tag?  Also, should this be routed through -stable?
> 
> Thanks.
> 
> -- 
> tejun

Tejun,

Sent a second version.

Thank you very much for the review,
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/ata/pata_macio.c b/drivers/ata/pata_macio.c
index a02f76f..aba634e 100644
--- a/drivers/ata/pata_macio.c
+++ b/drivers/ata/pata_macio.c
@@ -1344,6 +1344,7 @@  static struct of_device_id pata_macio_match[] =
 	},
 	{},
 };
+MODULE_DEVICE_TABLE(of, pata_macio_match);
 
 static struct macio_driver pata_macio_driver =
 {