From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Lawnick Date: Mon, 13 Sep 2010 11:12:03 +0000 Subject: Re: [lm-sensors] [PATCH v3] hwmon: lm75: Add __devinit and Message-Id: <4C8E0703.3090802@gmx.de> List-Id: References: <1284365862-20438-1-git-send-email-shubhrajyoti@ti.com> In-Reply-To: <1284365862-20438-1-git-send-email-shubhrajyoti@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org Datta, Shubhrajyoti said the following: > > >> -----Original Message----- >> From: Michael Lawnick [mailto:ml.lawnick@gmx.de] >> Sent: Monday, September 13, 2010 2:52 PM >> To: Datta, Shubhrajyoti >> Cc: lm-sensors@lm-sensors.org >> Subject: Re: [lm-sensors] [PATCH v3] hwmon: lm75: Add __devinit and >> __devexit section initializers >> >> Shubhrajyoti D said the following: >> ... >> >> > -static int >> > +static int __devinit >> > lm75_probe(struct i2c_client *client, const struct i2c_device_id *id) >> Are you sure? >> If I understand correctly, this function will be thrown away after >> system/module init. But what's about devices that are added later? >> I would expect them to crash on init... > As all of these are I2C devices are I2C I thought it makes sense. > Are you aware of devices that can be plugged/wired runtime? Almost all if behind a multiplexer ;-) There is just some care in synchronization needed. We are currently working with such a system that dynamically adds buses and drops them again (triggered by hotplug). -- KR Michael _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors