Re: [PATCH 1/1] i2c: s3c2410: Remove class based instantiation

[Tomasz Figa <tomasz.figa@gmail.com>]

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@the-dreams.de> 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@samsung.com>
>>> Cc: Olof Johansson <olof@lixom.net>
>>
>> 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