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-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
