Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754471AbYANTij (ORCPT ); Mon, 14 Jan 2008 14:38:39 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751568AbYANTib (ORCPT ); Mon, 14 Jan 2008 14:38:31 -0500 Received: from nz-out-0506.google.com ([64.233.162.236]:49543 "EHLO nz-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751045AbYANTi3 (ORCPT ); Mon, 14 Jan 2008 14:38:29 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references:x-google-sender-auth; b=Lq8mzlFXN0+rxnfby6tIbDEefiiR9tH3bDY4kkZtPnxQ+U5LmKRnUQHt78E22cnzkxgdTDu3vM4P1C/uijig71ZQgcQwyglU4HQ6Ib6DcFS3BigwrrIkTwOnsrvqcZhyQi53bklBiHNTR3upB2mZITGuf1BHD3YrenaTqqD0+zY= Message-ID: <3ae72650801141138t27666bcerd66e96f836d1ab2b@mail.gmail.com> Date: Mon, 14 Jan 2008 20:38:28 +0100 From: "Kay Sievers" To: "Jean Delvare" Subject: Re: [i2c] [PATCH] update module-init-tools to support the i2c subsystem Cc: "Geert Uytterhoeven" , linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org, i2c@lm-sensors.org In-Reply-To: <20080114185056.05b75fd1@hyperion.delvare> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <9e4733910712171839h214e0bc3q179266c86e5987b1@mail.gmail.com> <9e4733910801130714m1be16826jf240171972324d16@mail.gmail.com> <20080113163543.16e3d26b@hyperion.delvare> <9e4733910801130826l799e4897pe7380ccb973b5fd0@mail.gmail.com> <20080113184337.5d4cd546@hyperion.delvare> <9e4733910801131004i669276e2pde04b195dd10bd90@mail.gmail.com> <20080114173835.5fe907db@hyperion.delvare> <20080114185056.05b75fd1@hyperion.delvare> X-Google-Sender-Auth: 1a36ba5cb7e8d9de Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2510 Lines: 59 On Jan 14, 2008 6:50 PM, Jean Delvare wrote: > On Mon, 14 Jan 2008 18:08:16 +0100 (CET), Geert Uytterhoeven wrote: > > On Mon, 14 Jan 2008, Jean Delvare wrote: > > > I thought that the module aliases were generated by > > > scripts/mod/modpost? As a matter of fact, I did not apply Jon's patch > > > > Sorry, you're right. Too early in the morning :-) > > > > > to module-init-tools, and "modinfo" shows me module aliases properly > > > for i2c drivers that call MODULE_DEVICE_TABLE(): > > > > I've just looked it up again (I had to do a similar thing for Zorro bus > > support). Module-init-tools (depmod) also creates the modules.*map files, > > which are used to map from device IDs to module names. I think these are used > > by udev to load the appropriate module when a device with a specific device ID > > pops up in sysfs. > > Ah, right. I see it now, there's modules.isapnpmap, > modules.ieee1394map, modules.pcimap etc. but no modules.i2cmap. > However, there is modules.alias which contains the i2c aliases for all > device types (including one ieee1394 and many pci aliases) which seems > somewhat redundant with the modules.*map files. > > > > $ /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. > > > > Yes, it's also still not 100% clear to me when `i2c:Nadm1032' is used, and when > > modules.i2cmap would be used... > > I am under the impression that modules.*map are the old way to get > automatic driver loading and aliases are the new way to do the same. > But maybe that's just me. Right, nothing on recent systems is using the map files. This patch should not be needed. The plan is to deprecate the creation of these files in depmod. Thanks, Kay -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/