Re: [PATCH v3] hwmon: lm75: Add __devinit and __devexit section initializers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Datta, Shubhrajyoti said the following:
> 
> 
>> -----Original Message-----
>> From: Michael Lawnick [mailto:ml.lawnick@xxxxxx]
>> Sent: Monday, September 13, 2010 2:52 PM
>> To: Datta, Shubhrajyoti
>> Cc: lm-sensors@xxxxxxxxxxxxxx
>> Subject: Re:  [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@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux