[U-Boot] [PATCH 2/3] sunxi: Add clock support for TV encoder

["Jernej Škrabec" <jernej.skrabec@siol.net>]

Hi Maxime,

Dne petek, 12. maj 2017 ob 17:47:17 CEST je Maxime Ripard napisal(a):
> Hi Jernej,
> 
> On Wed, May 10, 2017 at 06:46:29PM +0200, Jernej Skrabec wrote:
> > This patch adds support for TV encoder clocks which will be used later.
> > 
> > Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net>
> > ---
> > 
> >  arch/arm/include/asm/arch-sunxi/clock_sun6i.h | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm/include/asm/arch-sunxi/clock_sun6i.h
> > b/arch/arm/include/asm/arch-sunxi/clock_sun6i.h index
> > a44ea77576..dc011cc964 100644
> > --- a/arch/arm/include/asm/arch-sunxi/clock_sun6i.h
> > +++ b/arch/arm/include/asm/arch-sunxi/clock_sun6i.h
> > @@ -83,7 +83,8 @@ struct sunxi_ccm_reg {
> > 
> >  	u32 lcd0_ch0_clk_cfg;	/* 0x118 LCD0 CH0 module clock */
> >  	u32 lcd1_ch0_clk_cfg;	/* 0x11c LCD1 CH0 module clock */
> >  
> >  #endif
> > 
> > -	u32 reserved14[3];
> > +	u32 tve_clk_cfg;	/* 0x120 TVE module clock */
> > +	u32 reserved14[2];
> 
> Maybe we should protect that with a comment, or an ifdef, that this is
> only for the H3-generation SoCs?

I vote for extending the comment, like: 
/* 0x120 H3/H5 TVE module clock */

Would that be ok?

Best regards,
Jernej