Hi Wolfram, Sachin, On 03.06.2014 13:21, Sachin Kamat wrote: > Hi Wolfram, > > On Tue, Jun 3, 2014 at 3:35 PM, Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: >> On Tue, Jun 03, 2014 at 02:30:22PM +0530, Sachin Kamat wrote: >>> With multiplatform support enabled for Exynos, class based >>> instantiation causes a boot time delay due to I2C_CLASS_HWMON >>> triggered probe. Since class based instantiation has been >>> deprecated since some time now, remove it for Samsung I2C driver. >>> >>> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxxx> >>> Cc: Olof Johansson <olof@xxxxxxxxx> >> >> In my book, not long enough. I suggest I will take care of the removal. >> I'll do it for all class deprecating drivers. Couldn't it be made configurable with a Kconfig entry? E.g. "Enable deprecated class-based instantiation" under Drivers / I2C? Turning it off could make the code ignore the class field completely. This way people that don't want it could just disable it and others could also test whether disabling it affects them in any way. > > The reason for the removal is the boot delay it is causing on Exynos 5250 > based boards by throwing the following errors continuously. > > [ 55.005300] s3c-i2c 12ce0000.i2c: cannot get bus (error -110) > [ 59.005300] s3c-i2c 12ce0000.i2c: cannot get bus (error -110) > [ 63.005300] s3c-i2c 12ce0000.i2c: cannot get bus (error -110) Are you sure this is directly related to class-based instantiation? This looks like the bus being driven low by some external component or incorrectly configured pins. What is this bus used for on affected board? Best regards, Tomasz -- 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
