[PATCH v1 2/4] arm: pxa: move init functions into generic.h

Robert Jarzmik robert.jarzmik at free.fr
Mon Sep 29 10:48:45 PDT 2014


Arnd Bergmann <arnd at arndb.de> writes:

> On Monday 29 September 2014 08:51:46 Robert Jarzmik wrote:
>> 
>>  #ifdef CONFIG_PXA25x
>> +#define pxa25x_handle_irq icip_handle_irq
>>  extern unsigned pxa25x_get_clk_frequency_khz(int);
>> +extern void __init pxa25x_init_irq(void);
>> +extern void __init pxa25x_map_io(void);
>> +#ifdef CONFIG_CPU_PXA26x
>> +extern void __init pxa26x_init_irq(void);
>> +#endif
>>  #else
>>  #define pxa25x_get_clk_frequency_khz(x)                (0)
>>  #endif
>> 
>
> It's better to add the declarations outside of the #ifdef. I believe
> the existing #ifdef is meant to provide a dummy alternative, but you
> don't actually want that for the other declarations, so just add them
> to a globally visible location.
OK for v2.

I will also put a note on my todo list to kill them. Their remaining purpose is
to provide frequency read accessor for get_clk_frequency(). Once the clock
framework will be done for all PXAs, get_clk_frequency() will become a simple
clk_get_rate(), or even better will be removed if I manage to transform viper.c
and the cpu frequency driver.

Cheers.

-- 
Robert



More information about the linux-arm-kernel mailing list