[U-Boot] [PATCH 1/2] common: Remove timer_init() call for x86

Bin Meng bmeng.cn at gmail.com
Thu Dec 3 02:52:51 CET 2015


Hi Simon,

On Thu, Dec 3, 2015 at 5:06 AM, Simon Glass <sjg at chromium.org> wrote:
> Hi Bin,
>
> On 2 December 2015 at 02:30, Bin Meng <bmeng.cn at gmail.com> wrote:
>> With driver model timer support, there should not be an explict
>> call to timer_init(). Remove this call for x86.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>  common/board_f.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/common/board_f.c b/common/board_f.c
>> index b035c90..92c42c5 100644
>> --- a/common/board_f.c
>> +++ b/common/board_f.c
>> @@ -807,7 +807,7 @@ static init_fnc_t init_sequence_f[] = {
>>         /* TODO: can we rename this to timer_init()? */
>>         init_timebase,
>>  #endif
>> -#if defined(CONFIG_X86) || defined(CONFIG_ARM) || defined(CONFIG_MIPS) || \
>> +#if defined(CONFIG_ARM) || defined(CONFIG_MIPS) || \
>>                 defined(CONFIG_BLACKFIN) || defined(CONFIG_NDS32)
>>         timer_init,             /* initialize timer */
>>  #endif
>> --
>> 1.8.2.1
>>
>
> Acked-by: Simon Glass <sjg at chromium.org>
>
> I have two questions:
>
> 1. Should we define a point during boot where the timer becomes
> available? I can't think of a good reason, but we should think about
> it.
>

I would rather not create such a special point for timer, instead we
just use the generic driver model limitations for the timer as well.
That is, with dm timer support, these U-Boot timer APIs should only be
available after driver model is initialized.

> 2. Should we provide an 'early timer' before DM is ready, a bit like
> we do with the UART?
>

I cannot think of a use case that uses timer in that early time.

Regards,
Bin


More information about the U-Boot mailing list