[U-Boot] [PATCH v2] Move ICS CLK chip frequenty calculation code into a common board library
Kumar Gala
galak at kernel.crashing.org
Mon May 24 20:57:12 CEST 2010
On May 24, 2010, at 9:52 AM, Timur Tabi wrote:
> On Sat, May 22, 2010 at 5:36 PM, Kumar Gala <galak at kernel.crashing.org> wrote:
>
>> +static unsigned long
>> +ics307_clk_freq(unsigned char cw0, unsigned char cw1, unsigned char cw2)
>
> You forgot to change these to u8
Oops, thought that was fixed.
> Also, you can declare this function as "const". You could just use
> the version of this function that I posted, you just need to remove my
> rounding code.
What in the world does 'const' get me in C.
>> + * Adding 1 to get a "nicely" rounded number, but this needs
>> + * more tweaking to get a "properly" rounded number.
>> + */
>> +
>> + freq = 1 + (input_freq * 2 * (vdw + 8) / ((rdw + 2) * od));
>
> The "1 +" is wrong and should be removed. It doesn't do what the
> comment says it does, and as you said earlier, rounding is wrong for
> this function.
Dropped the 1+
- k
More information about the U-Boot
mailing list