Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1707188 > unrolled thread
| Started by | Javier Martinez Canillas <javierm@redhat.com> |
|---|---|
| First post | 2017-08-09 11:40 +0200 |
| Last post | 2017-08-09 13:20 +0200 |
| Articles | 5 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases Javier Martinez Canillas <javierm@redhat.com> - 2017-08-09 11:40 +0200
Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases Kieran Bingham <kieran.bingham@ideasonboard.com> - 2017-08-09 12:30 +0200
Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases Javier Martinez Canillas <javierm@redhat.com> - 2017-08-09 13:00 +0200
Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases Kieran Bingham <kieran.bingham@ideasonboard.com> - 2017-08-09 13:10 +0200
Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases Javier Martinez Canillas <javierm@redhat.com> - 2017-08-09 13:20 +0200
| From | Javier Martinez Canillas <javierm@redhat.com> |
|---|---|
| Date | 2017-08-09 11:40 +0200 |
| Subject | [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases |
| Message-ID | <ucvD3-5jn-1@gated-at.bofh.it> |
The I2C core always reports a MODALIAS of the form i2c:<foo> even if the
device was registered via OF, and the driver is only exporting the OF ID
table entries as module aliases.
So if the driver is built as module, autoload won't work since udev/kmod
won't be able to match the registered OF device with its driver module.
Before this patch:
$ modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias
alias: of:N*T*Cadi,adv7482C*
alias: of:N*T*Cadi,adv7482
alias: of:N*T*Cadi,adv7481C*
alias: of:N*T*Cadi,adv7481
After this patch:
modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias
alias: of:N*T*Cadi,adv7482C*
alias: of:N*T*Cadi,adv7482
alias: of:N*T*Cadi,adv7481C*
alias: of:N*T*Cadi,adv7481
alias: i2c:adv7482
alias: i2c:adv7481
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
---
drivers/media/i2c/adv748x/adv748x-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/i2c/adv748x/adv748x-core.c b/drivers/media/i2c/adv748x/adv748x-core.c
index aeb6ae80cb18..5ee14f2c2747 100644
--- a/drivers/media/i2c/adv748x/adv748x-core.c
+++ b/drivers/media/i2c/adv748x/adv748x-core.c
@@ -807,6 +807,7 @@ static const struct i2c_device_id adv748x_id[] = {
{ "adv7482", 0 },
{ },
};
+MODULE_DEVICE_TABLE(i2c, adv748x_id);
static const struct of_device_id adv748x_of_table[] = {
{ .compatible = "adi,adv7481", },
--
2.13.3
[toc] | [next] | [standalone]
| From | Kieran Bingham <kieran.bingham@ideasonboard.com> |
|---|---|
| Date | 2017-08-09 12:30 +0200 |
| Subject | Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases |
| Message-ID | <ucwpr-5Qr-5@gated-at.bofh.it> |
| In reply to | #1707188 |
Hi Javier,
Thankyou for the patch
On 09/08/17 10:37, Javier Martinez Canillas wrote:
> The I2C core always reports a MODALIAS of the form i2c:<foo> even if the
> device was registered via OF, and the driver is only exporting the OF ID
> table entries as module aliases.
>
> So if the driver is built as module, autoload won't work since udev/kmod
> won't be able to match the registered OF device with its driver module.
Good catch, and perhaps I should have known better :D
I've only worked on this driver as a built-in so far :-) #BadExcuses
> Before this patch:
>
> $ modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias
> alias: of:N*T*Cadi,adv7482C*
> alias: of:N*T*Cadi,adv7482
> alias: of:N*T*Cadi,adv7481C*
> alias: of:N*T*Cadi,adv7481
>
> After this patch:
>
> modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias
> alias: of:N*T*Cadi,adv7482C*
> alias: of:N*T*Cadi,adv7482
> alias: of:N*T*Cadi,adv7481C*
> alias: of:N*T*Cadi,adv7481
> alias: i2c:adv7482
> alias: i2c:adv7481
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
> ---
>
> drivers/media/i2c/adv748x/adv748x-core.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/i2c/adv748x/adv748x-core.c b/drivers/media/i2c/adv748x/adv748x-core.c
> index aeb6ae80cb18..5ee14f2c2747 100644
> --- a/drivers/media/i2c/adv748x/adv748x-core.c
> +++ b/drivers/media/i2c/adv748x/adv748x-core.c
> @@ -807,6 +807,7 @@ static const struct i2c_device_id adv748x_id[] = {
> { "adv7482", 0 },
> { },
> };
> +MODULE_DEVICE_TABLE(i2c, adv748x_id);
>
> static const struct of_device_id adv748x_of_table[] = {
> { .compatible = "adi,adv7481", },
>
[toc] | [prev] | [next] | [standalone]
| From | Javier Martinez Canillas <javierm@redhat.com> |
|---|---|
| Date | 2017-08-09 13:00 +0200 |
| Subject | Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases |
| Message-ID | <ucwSt-61x-5@gated-at.bofh.it> |
| In reply to | #1707254 |
Hi Kieran, On 08/09/2017 12:29 PM, Kieran Bingham wrote: > Hi Javier, > > Thankyou for the patch > You are welcome. > On 09/08/17 10:37, Javier Martinez Canillas wrote: >> The I2C core always reports a MODALIAS of the form i2c:<foo> even if the >> device was registered via OF, and the driver is only exporting the OF ID >> table entries as module aliases. >> >> So if the driver is built as module, autoload won't work since udev/kmod >> won't be able to match the registered OF device with its driver module. > > Good catch, and perhaps I should have known better :D > > I've only worked on this driver as a built-in so far :-) #BadExcuses > A better excuse I think is that after all these years, one would had thought that the I2C OF modalias issue would had been finally fixed, but not yet :) >> Before this patch: >> >> $ modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias >> alias: of:N*T*Cadi,adv7482C* >> alias: of:N*T*Cadi,adv7482 >> alias: of:N*T*Cadi,adv7481C* >> alias: of:N*T*Cadi,adv7481 >> >> After this patch: >> >> modinfo drivers/media/i2c/adv748x/adv748x.ko | grep alias >> alias: of:N*T*Cadi,adv7482C* >> alias: of:N*T*Cadi,adv7482 >> alias: of:N*T*Cadi,adv7481C* >> alias: of:N*T*Cadi,adv7481 >> alias: i2c:adv7482 >> alias: i2c:adv7481 >> >> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> > > Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > Thanks! Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat
[toc] | [prev] | [next] | [standalone]
| From | Kieran Bingham <kieran.bingham@ideasonboard.com> |
|---|---|
| Date | 2017-08-09 13:10 +0200 |
| Subject | Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases |
| Message-ID | <ucx29-6kz-5@gated-at.bofh.it> |
| In reply to | #1707264 |
On 09/08/17 11:58, Javier Martinez Canillas wrote: > Hi Kieran, > > On 08/09/2017 12:29 PM, Kieran Bingham wrote: >> Hi Javier, >> >> Thankyou for the patch > > You are welcome. > >> On 09/08/17 10:37, Javier Martinez Canillas wrote: >>> The I2C core always reports a MODALIAS of the form i2c:<foo> even if the >>> device was registered via OF, and the driver is only exporting the OF ID >>> table entries as module aliases. >>> >>> So if the driver is built as module, autoload won't work since udev/kmod >>> won't be able to match the registered OF device with its driver module. >> >> Good catch, and perhaps I should have known better :D >> >> I've only worked on this driver as a built-in so far :-) #BadExcuses >> > > A better excuse I think is that after all these years, one would had thought > that the I2C OF modalias issue would had been finally fixed, but not yet :) Quite! Let's try to bubble that back up the todo list. Now - where did I put my free time. I'm sure I left it around here somewhere :-) -- Kieran
[toc] | [prev] | [next] | [standalone]
| From | Javier Martinez Canillas <javierm@redhat.com> |
|---|---|
| Date | 2017-08-09 13:20 +0200 |
| Subject | Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases |
| Message-ID | <ucxbP-6nH-1@gated-at.bofh.it> |
| In reply to | #1707267 |
On 08/09/2017 01:05 PM, Kieran Bingham wrote: > On 09/08/17 11:58, Javier Martinez Canillas wrote: >> Hi Kieran, >> >> On 08/09/2017 12:29 PM, Kieran Bingham wrote: >>> Hi Javier, >>> >>> Thankyou for the patch >> >> You are welcome. >> >>> On 09/08/17 10:37, Javier Martinez Canillas wrote: >>>> The I2C core always reports a MODALIAS of the form i2c:<foo> even if the >>>> device was registered via OF, and the driver is only exporting the OF ID >>>> table entries as module aliases. >>>> >>>> So if the driver is built as module, autoload won't work since udev/kmod >>>> won't be able to match the registered OF device with its driver module. >>> >>> Good catch, and perhaps I should have known better :D >>> >>> I've only worked on this driver as a built-in so far :-) #BadExcuses >>> >> >> A better excuse I think is that after all these years, one would had thought >> that the I2C OF modalias issue would had been finally fixed, but not yet :) > > Quite! Let's try to bubble that back up the todo list. > Now - where did I put my free time. I'm sure I left it around here somewhere :-) > We are getting there though. I'm just waiting for the patches in this [0] series to land and then I'll be able to post the I2C core uevent modalias patch. I've asked Wolfram if he can at least pick the driver patches [1], but he didn't answer me yet... [0]: https://www.spinics.net/lists/arm-kernel/msg588431.html [1]: https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1457427.html Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web