[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