Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756269AbaGNSkq (ORCPT ); Mon, 14 Jul 2014 14:40:46 -0400 Received: from cpsmtpb-ews02.kpnxchange.com ([213.75.39.5]:55962 "EHLO cpsmtpb-ews02.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753087AbaGNSkh (ORCPT ); Mon, 14 Jul 2014 14:40:37 -0400 Message-ID: <1405363234.13661.12.camel@x220> Subject: Re: [PATCH] mach-s3c64xx: Remove all defintions not related to regs-clocks From: Paul Bolle To: Randy Dunlap Cc: Nicholas Krause , ben-linux@fluff.org, kgene.kim@samsung.com, linux@arm.linux.org.uk, tomasz.figa@gmail.com, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 14 Jul 2014 20:40:34 +0200 In-Reply-To: <53C4201E.3020708@infradead.org> References: <1405361664-5210-1-git-send-email-xerofoify@gmail.com> <53C4201E.3020708@infradead.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-2.fc20) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 14 Jul 2014 18:40:35.0953 (UTC) FILETIME=[19D67E10:01CF9F93] X-RcptDomain: vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > > --- > > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/