[U-Boot] [PATCH 1/4] OMAP3 I2C Fix the sampling clock.

[Tom <Tom.Rix@windriver.com>]

Menon, Nishanth wrote:
>> -----Original Message-----
>> From: Tom [mailto:Tom.Rix at windriver.com]
>> Sent: Wednesday, June 10, 2009 9:44 PM
>>     
>>> This is a repeat story of what happened in linux-omap and kernel. We had
>>>       
>> a similar discussion in [1] and related patch [2] to change equations. I
>> have the same reservations with this patch:
>>     
>>> a) using speed as default does not scale for all board combinations.
>>> b) need flexible option to provide scll and sclh on a platform basis.
>>> Regards,
>>> Nishanth Menon
>>> Ref:
>>> [1] http://marc.info/?t=123540865900002&r=1&w=2
>>> [2] http://marc.info/?l=linux-omap&m=122770723311340&w=2
>>>
>>>       
>> Do you think this could be handled with just config files?
>> Or maybe like fsl_i2c does by passing clk data through the global_data ?
>>     
> #defines in config header file might be a viable option.. Though the gd might be cleaner I think.. 
>
> Regards,
> Nishanth Menon
>   
How about something like this?
The timing calculation is moved to a separate function that is weakly 
aliased to a function that the boards can define?  This is similar to 
led initializing in lib_arm.   I have put a stub omap_i2c_timing in 
zoom1.c to show how the board would do the define.

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-OMAP3-I2C-Fix-the-sampling-clock.patch
Type: text/x-patch
Size: 0 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20090611/31609f4f/attachment.bin