[lkml]   [2008]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [i2c] [PATCH] update module-init-tools to support the i2c subsystem
    Hi Geert,

    On Mon, 14 Jan 2008 11:57:52 +0100 (CET), Geert Uytterhoeven wrote:
    > On Sun, 13 Jan 2008, Jon Smirl wrote:
    > > I don't know exactly what those modules tables are used for. I just
    > > copied what the other subsystems do. Maybe they are used when you make
    > > an initrd to know which drivers to copy into the image.
    > Module-init-tools needs those table to create module aliases in the *.ko
    > files from the MODULE_DEVICE_TABLE(), so udev can load the modules based
    > on the device IDs when the devices appear in sysfs.

    I thought that the module aliases were generated by
    scripts/mod/modpost? As a matter of fact, I did not apply Jon's patch
    to module-init-tools, and "modinfo" shows me module aliases properly
    for i2c drivers that call MODULE_DEVICE_TABLE():

    $ /sbin/modinfo lm90
    filename: /lib/modules/2.6.24-rc7-git4/kernel/drivers/hwmon/lm90.ko
    author: Jean Delvare <>
    description: LM90/ADM1032 driver
    license: GPL
    vermagic: 2.6.24-rc7-git4 mod_unload
    depends: hwmon
    alias: i2c:Nlm90*
    alias: i2c:Nadm1032*
    alias: i2c:Nlm99*
    alias: i2c:Nlm86*
    alias: i2c:Nmax6657*
    alias: i2c:Nadt7461*
    alias: i2c:Nmax6680*

    "modprobe i2c:Nadm1032" loads the lm90 driver as expected.

    > That's the generic part. How this applies to i2c devices on platforms
    > without Open Firmware device trees is another question. I guess that's
    > where Jean gets confused (i2c_device_id got _removed_ last year,
    > because it didn't make sense (at the time?)).

    The way it was implemented back then did not make sense. As it was not
    clear whether we would implement something different or nothing at all,
    I decided to plain remove it. Now that it seems that we want to
    implement it differently, I'm looking into it again.

    Jean Delvare

     \ /
      Last update: 2008-01-14 17:41    [W:0.021 / U:12.736 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site