| Message ID | 20200206150837.12009-1-codrin.ciubotariu@microchip.com |
|---|---|
| State | Superseded |
| Delegated to: | David Miller |
| Headers | show |
| Series | net: dsa: microchip: enable module autoprobe | expand |
On Thu, Feb 06, 2020 at 05:08:37PM +0200, Codrin Ciubotariu wrote: > From: Razvan Stefanescu <razvan.stefanescu@microchip.com> > > This matches /sys/devices/.../spi1.0/modalias content. > > Signed-off-by: Razvan Stefanescu <razvan.stefanescu@microchip.com> > Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> > --- > drivers/net/dsa/microchip/ksz9477_spi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/dsa/microchip/ksz9477_spi.c b/drivers/net/dsa/microchip/ksz9477_spi.c > index c5f64959a184..248b69c74b45 100644 > --- a/drivers/net/dsa/microchip/ksz9477_spi.c > +++ b/drivers/net/dsa/microchip/ksz9477_spi.c > @@ -101,6 +101,7 @@ static struct spi_driver ksz9477_spi_driver = { > > module_spi_driver(ksz9477_spi_driver); > > +MODULE_ALIAS("spi:ksz8563"); Is this sufficient for all the different variants this driver supports? Andrew
On 06.02.2020 17:34, Andrew Lunn wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On Thu, Feb 06, 2020 at 05:08:37PM +0200, Codrin Ciubotariu wrote: >> From: Razvan Stefanescu <razvan.stefanescu@microchip.com> >> >> This matches /sys/devices/.../spi1.0/modalias content. >> >> Signed-off-by: Razvan Stefanescu <razvan.stefanescu@microchip.com> >> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> >> --- >> drivers/net/dsa/microchip/ksz9477_spi.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/net/dsa/microchip/ksz9477_spi.c b/drivers/net/dsa/microchip/ksz9477_spi.c >> index c5f64959a184..248b69c74b45 100644 >> --- a/drivers/net/dsa/microchip/ksz9477_spi.c >> +++ b/drivers/net/dsa/microchip/ksz9477_spi.c >> @@ -101,6 +101,7 @@ static struct spi_driver ksz9477_spi_driver = { >> >> module_spi_driver(ksz9477_spi_driver); >> >> +MODULE_ALIAS("spi:ksz8563"); > > Is this sufficient for all the different variants this driver > supports? Probably not. All the available variants seem to be those from the compatible list. I can make aliases for all of them, if it's ok for everyone. Best regards, Codrin
diff --git a/drivers/net/dsa/microchip/ksz9477_spi.c b/drivers/net/dsa/microchip/ksz9477_spi.c index c5f64959a184..248b69c74b45 100644 --- a/drivers/net/dsa/microchip/ksz9477_spi.c +++ b/drivers/net/dsa/microchip/ksz9477_spi.c @@ -101,6 +101,7 @@ static struct spi_driver ksz9477_spi_driver = { module_spi_driver(ksz9477_spi_driver); +MODULE_ALIAS("spi:ksz8563"); MODULE_AUTHOR("Woojung Huh <Woojung.Huh@microchip.com>"); MODULE_DESCRIPTION("Microchip KSZ9477 Series Switch SPI access Driver"); MODULE_LICENSE("GPL");