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.

-- 
Jean Delvare

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

Reply via email to