Re: [PATCH] i2c: skip address detection if provided in board_info

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

 



On Thu, 14 Oct 2010 01:04:22 +0800, Feng Tang wrote:
> But I still have some concern (not specific to our platforms), I just checked,
> there are 42 i2c adapter drivers set their class to I2C_CLASS_HWMON and 50
> hwmon i2c device drivers set it as well. So if a kernel has an adapter driver
> and hwmon driver which both set I2C_CLASS_HWMON, the long init time problem
> will show up again, and it will be worse if there are multiple adapters HW
> there.

It has been that way for a couple years now and has never been reported
to be a problem in practice.

The overall number of drivers setting class flag I2C_CLASS_HWMON is
irrelevant, as on a given system, only a few I2C adapter and device
drivers will be loaded. As I said before, it would become a problem if
you loaded a huger number of I2C-based hwmon device drivers you don't
need, which would happen if you were to build a kernel with all hwmon
drivers build-in. But this doesn't sound like a sane thing to do.

Also please note that I still believe that your I2C adapter driver
could be improved to better deal with this case. Loading the adm1021
hwmon driver on my systems only takes 21 ms, even though it probes 9
addresses. If proving 4 * 5 addresses on your system took a noticeable
time, then I suspect that your adapter driver isn't dealing with that
case properly.

But anyway, if you're on an embedded system and care a lot about very
fast boot times, then you don't want to do any I2c device probing at
all. You want all your devices enumerated as platform data, that's the
fastest option.

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux