[U-Boot] [PATCH 1/1] at91sam9x: driver to use the internal RTT as RTC

Tom Tom.Rix at windriver.com
Mon Apr 5 15:19:38 CEST 2010


Alexander Holler wrote:
> Am 03.04.2010 20:33, schrieb Tom:
>> Alexander Holler wrote:
>>
>> Change parameter name to something more descriptive.
>> 'tmp' implies used locally and this value is passed back.
>> Maybe 'tm' as to_tmp(.., struct rtc_time *tm) ?
>>
> 
> Maybe someone should place a README in drivers/rtc which reads "Don't 
> use this stuff as template" as almost every source there uses this tmp.
> 

The important change for this patch is to move to the new Soc access.

The other uses of rtc_time *tmp, I would also consider as problems.
Changing tmp -> tm for your patch is an improvement.
This is good time to make it.

Tom



More information about the U-Boot mailing list