Re: [PATCH] mach-s3c64xx: Remove all defintions not related to regs-clocks

[Paul Bolle]

On Mon, 2014-07-14 at 11:23 -0700, Randy Dunlap wrote:
> On 07/14/14 11:14, Nicholas Krause wrote:
> > This patch addresses the fix me message in this file that states to
> > remove all definitions not related to reg-clocks in this header
> > file.
> > 
> > Signed-off-by: Nicholas Krause <xerofoify@xxxxxxxxx>
> > ---
> >  arch/arm/mach-s3c64xx/include/mach/regs-clock.h | 22 +---------------------
> >  1 file changed, 1 insertion(+), 21 deletions(-)
> > 
> > diff --git a/arch/arm/mach-s3c64xx/include/mach/regs-clock.h b/arch/arm/mach-s3c64xx/include/mach/regs-clock.h
> > index 4f44aac..46e64cc 100644
> > --- a/arch/arm/mach-s3c64xx/include/mach/regs-clock.h
> > +++ b/arch/arm/mach-s3c64xx/include/mach/regs-clock.h
> > @@ -15,24 +15,4 @@
> >  #ifndef __PLAT_REGS_CLOCK_H
> >  #define __PLAT_REGS_CLOCK_H __FILE__
> >  
> > -/*
> > - * FIXME: Remove remaining definitions
> > - */
> > -
> > -#define S3C_CLKREG(x)		(S3C_VA_SYS + (x))
> > -
> > -#define S3C_PCLK_GATE		S3C_CLKREG(0x34)
> > -#define S3C6410_CLK_SRC2	S3C_CLKREG(0x10C)
> > -#define S3C_MEM_SYS_CFG		S3C_CLKREG(0x120)
> > -
> > -/* PCLK GATE Registers */
> > -#define S3C_CLKCON_PCLK_UART3		(1<<4)
> > -#define S3C_CLKCON_PCLK_UART2		(1<<3)
> > -#define S3C_CLKCON_PCLK_UART1		(1<<2)
> > -#define S3C_CLKCON_PCLK_UART0		(1<<1)
> > -
> > -/* MEM_SYS_CFG */
> > -#define MEM_SYS_CFG_INDEP_CF		0x4000
> > -#define MEM_SYS_CFG_EBI_FIX_PRI_CFCON	0x30
> > -
> > -#endif /* _PLAT_REGS_CLOCK_H */
> > +/#endif /* _PLAT_REGS_CLOCK_H */
> 
> I don't know about the other changes, but that last line looks like an error.
> Did you build anything that #includes this header file?

I just did
    git grep -n S3C_CLKCON_PCLK_UART

It was just a guess. I did not cherry pick the defines that covers! The
output of that command is, I think, pretty clear. 

Nicholas, I think you've been told already to stop doing what you're
doing again here. I try not to use colorful language on the net, but
you're really pushing me here.

Please go find something other to do than grepping the kernel tree for
FIXMEs!


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/