* [PATCH] sata_mv: Define module alias for platform device
@ 2008-02-16 17:15 Martin Michlmayr
2008-02-16 17:22 ` Mark Lord
2008-02-20 17:09 ` Jeff Garzik
0 siblings, 2 replies; 3+ messages in thread
From: Martin Michlmayr @ 2008-02-16 17:15 UTC (permalink / raw)
To: Mark Lord, linux-ide
The sata_mv driver can be loaded as a platform device, as is done by
various Orion (ARM) based devices. The driver needs to define a module
alias for the platform driver so udev will load it automatically.
Tested with Debian on a QNAP TS-209.
Signed-off-by: Martin Michlmayr <tbm@cyrius.com>
--- a/drivers/ata/sata_mv.c
+++ b/drivers/ata/sata_mv.c
@@ -3198,6 +3198,7 @@ MODULE_DESCRIPTION("SCSI low-level driver for Marvell SATA controllers");
MODULE_LICENSE("GPL");
MODULE_DEVICE_TABLE(pci, mv_pci_tbl);
MODULE_VERSION(DRV_VERSION);
+MODULE_ALIAS("platform:sata_mv");
#ifdef CONFIG_PCI
module_param(msi, int, 0444);
--
Martin Michlmayr
http://www.cyrius.com/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sata_mv: Define module alias for platform device
2008-02-16 17:15 [PATCH] sata_mv: Define module alias for platform device Martin Michlmayr
@ 2008-02-16 17:22 ` Mark Lord
2008-02-20 17:09 ` Jeff Garzik
1 sibling, 0 replies; 3+ messages in thread
From: Mark Lord @ 2008-02-16 17:22 UTC (permalink / raw)
To: Martin Michlmayr; +Cc: linux-ide
Martin Michlmayr wrote:
> The sata_mv driver can be loaded as a platform device, as is done by
> various Orion (ARM) based devices. The driver needs to define a module
> alias for the platform driver so udev will load it automatically.
> Tested with Debian on a QNAP TS-209.
>
> Signed-off-by: Martin Michlmayr <tbm@cyrius.com>
>
> --- a/drivers/ata/sata_mv.c
> +++ b/drivers/ata/sata_mv.c
> @@ -3198,6 +3198,7 @@ MODULE_DESCRIPTION("SCSI low-level driver for Marvell SATA controllers");
> MODULE_LICENSE("GPL");
> MODULE_DEVICE_TABLE(pci, mv_pci_tbl);
> MODULE_VERSION(DRV_VERSION);
> +MODULE_ALIAS("platform:sata_mv");
>
> #ifdef CONFIG_PCI
> module_param(msi, int, 0444);
>
..
Acked-by: Mark Lord <mlord@pobox.com>
Looks safe enough.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sata_mv: Define module alias for platform device
2008-02-16 17:15 [PATCH] sata_mv: Define module alias for platform device Martin Michlmayr
2008-02-16 17:22 ` Mark Lord
@ 2008-02-20 17:09 ` Jeff Garzik
1 sibling, 0 replies; 3+ messages in thread
From: Jeff Garzik @ 2008-02-20 17:09 UTC (permalink / raw)
To: Martin Michlmayr; +Cc: Mark Lord, linux-ide
Martin Michlmayr wrote:
> The sata_mv driver can be loaded as a platform device, as is done by
> various Orion (ARM) based devices. The driver needs to define a module
> alias for the platform driver so udev will load it automatically.
> Tested with Debian on a QNAP TS-209.
>
> Signed-off-by: Martin Michlmayr <tbm@cyrius.com>
>
> --- a/drivers/ata/sata_mv.c
> +++ b/drivers/ata/sata_mv.c
> @@ -3198,6 +3198,7 @@ MODULE_DESCRIPTION("SCSI low-level driver for Marvell SATA controllers");
> MODULE_LICENSE("GPL");
> MODULE_DEVICE_TABLE(pci, mv_pci_tbl);
> MODULE_VERSION(DRV_VERSION);
> +MODULE_ALIAS("platform:sata_mv");
>
> #ifdef CONFIG_PCI
> module_param(msi, int, 0444);
applied
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-02-20 17:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-02-16 17:15 [PATCH] sata_mv: Define module alias for platform device Martin Michlmayr
2008-02-16 17:22 ` Mark Lord
2008-02-20 17:09 ` Jeff Garzik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).