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/  |

--
To unsubscribe from this list: send the line "unsubscribe linux-clk" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to