Re: [PATCH 02/18] staging: iio: hmc5843: Export missing SPI module alias information
On 01/09/15 00:09, Javier Martinez Canillas wrote: > Hello Jonathan, > > On 08/22/2015 07:59 PM, Jonathan Cameron wrote: >> On 20/08/15 08:07, Javier Martinez Canillas wrote: >>> The SPI core always reports the MODALIAS uevent as "spi:" >>> regardless of the mechanism that was used to register the device >>> (i.e: OF or board code) and the table that is used later to match >>> the driver with the device (i.e: SPI id table or OF match table). >>> >>> So drivers needs to export the SPI id table and this be built into >>> the module or udev won't have the necessary information to autoload >>> the needed driver module when the device is added. >>> >>> Signed-off-by: Javier Martinez Canillas>> Applied to the togreg branch of iio.git. >> >> This is too late for the upcoming merge window so it will be queued up for >> the next one. >> > > IMHO this patch and "[PATCH 01/18] iio: Export SPI module alias > information in missing drivers" [0] are fixing broken module > autoloading which are bugs so are material for the 4.3 -rc cycle. It's a corner case. Could also be argued that this isn't a bug but rather a case of a feature (autoprobing) being added that wasn't supported before. It's not obligatory to support autoloading (even if we would normally aim to do so). I'm happy enough for a request to apply these to stable occurs after they hit Linus' tree (as a trivial backport) but I don't think they really deserve being sent on as fixes. Of course, I might be missing something that means something is actually broken, as opposed to not present. Jonathan > >> Thanks, >> >> Jonathan > > [0]: https://lkml.org/lkml/2015/8/20/111 > > Best regards, > ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 02/18] staging: iio: hmc5843: Export missing SPI module alias information
Hello Jonathan, On 09/05/2015 06:31 PM, Jonathan Cameron wrote: > On 01/09/15 00:09, Javier Martinez Canillas wrote: >> Hello Jonathan, >> >> On 08/22/2015 07:59 PM, Jonathan Cameron wrote: >>> On 20/08/15 08:07, Javier Martinez Canillas wrote: The SPI core always reports the MODALIAS uevent as "spi:" regardless of the mechanism that was used to register the device (i.e: OF or board code) and the table that is used later to match the driver with the device (i.e: SPI id table or OF match table). So drivers needs to export the SPI id table and this be built into the module or udev won't have the necessary information to autoload the needed driver module when the device is added. Signed-off-by: Javier Martinez Canillas>>> Applied to the togreg branch of iio.git. >>> >>> This is too late for the upcoming merge window so it will be queued up for >>> the next one. >>> >> >> IMHO this patch and "[PATCH 01/18] iio: Export SPI module alias >> information in missing drivers" [0] are fixing broken module >> autoloading which are bugs so are material for the 4.3 -rc cycle. > > It's a corner case. Could also be argued that this isn't > a bug but rather a case of a feature (autoprobing) being added > that wasn't supported before. It's not obligatory to support > autoloading (even if we would normally aim to do so). > > I'm happy enough for a request to apply these to stable occurs > after they hit Linus' tree (as a trivial backport) but I don't > think they really deserve being sent on as fixes. > > Of course, I might be missing something that means something is > actually broken, as opposed to not present. > Well, as a user I would expect that if I have a driver built-in and it works, building it as a module will also work so I think module autoload it's a bugfix and not a new feature. And the patch is trivial and won't cause any issues so I don't see why it can't be -rc material. But of course is up to you, git log shows me that these drivers have been since v3.10 so it seems that nobody cared anyways. > Jonathan > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 02/18] staging: iio: hmc5843: Export missing SPI module alias information
Hello Jonathan, On 08/22/2015 07:59 PM, Jonathan Cameron wrote: > On 20/08/15 08:07, Javier Martinez Canillas wrote: >> The SPI core always reports the MODALIAS uevent as "spi:" >> regardless of the mechanism that was used to register the device >> (i.e: OF or board code) and the table that is used later to match >> the driver with the device (i.e: SPI id table or OF match table). >> >> So drivers needs to export the SPI id table and this be built into >> the module or udev won't have the necessary information to autoload >> the needed driver module when the device is added. >> >> Signed-off-by: Javier Martinez Canillas> Applied to the togreg branch of iio.git. > > This is too late for the upcoming merge window so it will be queued up for > the next one. > IMHO this patch and "[PATCH 01/18] iio: Export SPI module alias information in missing drivers" [0] are fixing broken module autoloading which are bugs so are material for the 4.3 -rc cycle. > Thanks, > > Jonathan [0]: https://lkml.org/lkml/2015/8/20/111 Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 02/18] staging: iio: hmc5843: Export missing SPI module alias information
On 20/08/15 08:07, Javier Martinez Canillas wrote: The SPI core always reports the MODALIAS uevent as spi:modalias regardless of the mechanism that was used to register the device (i.e: OF or board code) and the table that is used later to match the driver with the device (i.e: SPI id table or OF match table). So drivers needs to export the SPI id table and this be built into the module or udev won't have the necessary information to autoload the needed driver module when the device is added. Signed-off-by: Javier Martinez Canillas jav...@osg.samsung.com Applied to the togreg branch of iio.git. This is too late for the upcoming merge window so it will be queued up for the next one. Thanks, Jonathan --- drivers/staging/iio/magnetometer/hmc5843_spi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/magnetometer/hmc5843_spi.c b/drivers/staging/iio/magnetometer/hmc5843_spi.c index 8e658f736e1f..4dfb372d2fec 100644 --- a/drivers/staging/iio/magnetometer/hmc5843_spi.c +++ b/drivers/staging/iio/magnetometer/hmc5843_spi.c @@ -81,6 +81,7 @@ static const struct spi_device_id hmc5843_id[] = { { hmc5983, HMC5983_ID }, { } }; +MODULE_DEVICE_TABLE(spi, hmc5843_id); static struct spi_driver hmc5843_driver = { .driver = { ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel