Re: [PATCH] i2c-algo-pcf: Drop unused struct members

[Daniel Smolik <marvin-0pWKB23IDFjrBKCeMvbIDA@public.gmane.org>]

Eric Brower napsal(a):
> On Fri, Jun 13, 2008 at 1:25 AM, Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org> wrote:
>> Struct members udelay and timeout aren't used anywhere, so drop them.
>>
>> Signed-off-by: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>
>> ---
>> Eric, Daniel, you're using i2c-algo-pcf more than me, can you please
>> check and confirm that this patch is correct? It applies on top of
>> Eric's patch for lost arbitration handling.
>>
>>  drivers/i2c/busses/i2c-elektor.c |    2 --
>>  include/linux/i2c-algo-pcf.h     |    4 ----
>>  2 files changed, 6 deletions(-)
>>
>> --- linux-2.6.26-rc5.orig/drivers/i2c/busses/i2c-elektor.c      2008-06-12 15:22:32.000000000 +0200
>> +++ linux-2.6.26-rc5/drivers/i2c/busses/i2c-elektor.c   2008-06-13 09:32:27.000000000 +0200
>> @@ -196,8 +196,6 @@ static struct i2c_algo_pcf_data pcf_isa_
>>        .getown     = pcf_isa_getown,
>>        .getclock   = pcf_isa_getclock,
>>        .waitforpin = pcf_isa_waitforpin,
>> -       .udelay     = 10,
>> -       .timeout    = 100,
>>  };
>>
>>  static struct i2c_adapter pcf_isa_ops = {
>> --- linux-2.6.26-rc5.orig/include/linux/i2c-algo-pcf.h  2008-06-13 09:08:00.000000000 +0200
>> +++ linux-2.6.26-rc5/include/linux/i2c-algo-pcf.h       2008-06-13 09:11:50.000000000 +0200
>> @@ -33,10 +33,6 @@ struct i2c_algo_pcf_data {
>>        int  (*getclock) (void *data);
>>        void (*waitforpin) (void);
>>
>> -       /* local settings */
>> -       int udelay;
>> -       int timeout;
>> -
>>        /* Multi-master lost arbitration back-off delay (msecs)
>>         * This should be set by the bus adapter or knowledgable client
>>         * if bus is multi-mastered, else zero
>>
>>
>> --
>> Jean Delvare
>>
> 
> ACKed on 2.6.26-rc1.  Compiles (and runs) fine.
> 
> [Daniel, this requires changes to i2c-envctrl, since it references the
> removed members when initializing the i2c_algo_pcf_data struct.]
> 
> Thanks again, Jean.
> 
> E
> 
> 
Do you includes this changes to SVN repository or can  do it myself on my source ?


	Dan



_______________________________________________
i2c mailing list
i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org
http://lists.lm-sensors.org/mailman/listinfo/i2c