Jean Delvare wrote:
> Hi Troy,
> 
> On Fri, 28 Mar 2008 21:36:49 +0100, Jean Delvare wrote:
>> On Fri, 28 Mar 2008 13:04:10 -0700, Troy Kisky wrote:
>>> Jean Delvare wrote:
>>>> Hi Troy,
>>>>
>>>> On Thu, 20 Mar 2008 20:06:07 -0700, Troy Kisky wrote:
>>>>> Ensure psc value gives a clock between 7-12 Mhz
>>>> Correct spelling is MHz.
>>>>
>>>>> Signed-off-by: Troy Kisky <[EMAIL PROTECTED]>
>>>>> Signed-off-by: Kevin Hilman <[EMAIL PROTECTED]>
>>>>> ---
>>>>>  drivers/i2c/busses/i2c-davinci.c |   19 ++++++++++++-------
>>>>>  1 files changed, 12 insertions(+), 7 deletions(-)
>>>>>
>>>>> diff --git a/drivers/i2c/busses/i2c-davinci.c 
>>>>> b/drivers/i2c/busses/i2c-davinci.c
>>>>> index fde2634..82d4b7f 100755
>>>>> --- a/drivers/i2c/busses/i2c-davinci.c
>>>>> +++ b/drivers/i2c/busses/i2c-davinci.c
>>>>> @@ -142,6 +142,7 @@ static int i2c_davinci_init(struct davinci_i2c_dev 
>>>>> *dev)
>>>>>   struct davinci_i2c_platform_data *pdata = dev->dev->platform_data;
>>>>>   u16 psc;
>>>>>   u32 clk;
>>>>> + u32 d;
>>>>>   u32 clkh;
>>>>>   u32 clkl;
>>>>>   u32 input_clock = clk_get_rate(dev->clk);
>>>>> @@ -171,23 +172,29 @@ static int i2c_davinci_init(struct davinci_i2c_dev 
>>>>> *dev)
>>>>>    *       if PSC > 1 , d = 5
>>>>>    */
>>>>>  
>>>>> - psc = 26; /* To get 1MHz clock */
>>>>> + /* get minimum of 7mHz clock, but max of 12mHz */
>>>> Correct spelling is MHz.
>>>
>>> Hmm... I seems weird that kHz and MHz are correct, as opposed to
>>> kHz and mHz,  or KHz and MHz.
>> These are the standard international unit prefixes. Please see:
>> http://en.wikipedia.org/wiki/SI_prefix#List_of_SI_prefixes
>> kilo- is "k", mega- is "M". "m" would be milli-, which isn't what you
>> want here.
> 
> <snip>
> 
> Any update available for this patch? If you want it (and patch 3/5 of
> this series) to go in 2.6.26 we better hurry up.
> 
Kevin's approval for 1 of the patches has temporarily been revoked.

I'm waiting to get it back.

Thanks
Troy


_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c

Reply via email to