[PATCH v2 2/8] clk: imx25: retain early UART clocks during kernel init

Lucas Stach l.stach at pengutronix.de
Mon Sep 7 02:18:25 PDT 2015


Am Montag, den 07.09.2015, 11:10 +0200 schrieb Uwe Kleine-König:
> On Sun, Sep 06, 2015 at 10:23:15PM +0800, Shawn Guo wrote:
> > 
> > I prefer to code it like:
> > 
> > static struct clk **uart_clks[] __initdata = {
> > 	&clk[uart_ipg_per],
> > 	&clk[uart1_ipg],
> > 	&clk[uart2_ipg],
> > 	&clk[uart3_ipg],
> > 	&clk[uart4_ipg],
> > 	&clk[uart5_ipg],
> > 	NULL
> > };
> 
> And maybe also use mx25_ as prefix to make the name unique.
> 
We already have a lot of non-unique names in those files. As this is a
static variable it will not leak into other compilation units, so I
prefer to keep the current style of the file.

Regards,
Lucas

-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |




More information about the linux-arm-kernel mailing list