Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1707264

Re: [PATCH] media: i2c: adv748x: Export I2C device table entries as module aliases

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[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

csiph-web