From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ashok Kumar Reddy Date: Mon, 01 Oct 2012 08:40:01 +0530 Subject: [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx In-Reply-To: <201209301521.29487.marex@denx.de> References: <5063496F.1080309@gmail.com> <50682BEA.2060804@gmail.com> <201209301521.29487.marex@denx.de> Message-ID: <50690989.7020000@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Marek Vasut, On 30-09-2012 18:51, Marek Vasut wrote: > Dear Ashok Kumar Reddy, > >> Hi Marek Vasut , Hi Stephen Warren >> >> Please provide comments for the below patch, as i forgot to add you in >> cc for below mail. > > You CC wrong people, CCing Minkyu. > > Anyway, this doesn't seem right, removing CONFIG_ICACHE_OFF from your config > file should have the same impact, right? CONFIG_ICACHE_OFF is valid for only blackfin as per grep in u-boot code, the icache enable is done in arch_cpu_init() for arm1176 > >> Regards, >> Ashok >> >> On 26-09-2012 23:59, Ashok Kumar Reddy wrote: >>> From 0de61b9690c5f136ba5d3028be43877ea455d843 Mon Sep 17 00:00:00 2001 >>> >>> From: Ashok Kumar Reddy >>> Date: Wed, 26 Sep 2012 23:44:59 +0530 >>> Subject: [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx >>> >>> arch_cpu_init() is removed from cpu level to SOC level for arm1176 >>> in commit 4ea6d6b,the same is done for s3c64xx >>> >>> Signed-off-by: Ashok Kumar Reddy >>> --- >>> >>> arch/arm/cpu/arm1176/s3c64xx/Makefile | 2 +- >>> arch/arm/cpu/arm1176/s3c64xx/init.c | 26 ++++++++++++++++++++++++++ >>> 2 files changed, 27 insertions(+), 1 deletion(-) >>> create mode 100644 arch/arm/cpu/arm1176/s3c64xx/init.c >>> >>> diff --git a/arch/arm/cpu/arm1176/s3c64xx/Makefile >>> b/arch/arm/cpu/arm1176/s3c64xx/Makefile index 0785b19..266a073 100644 >>> --- a/arch/arm/cpu/arm1176/s3c64xx/Makefile >>> +++ b/arch/arm/cpu/arm1176/s3c64xx/Makefile >>> @@ -31,7 +31,7 @@ LIB = $(obj)lib$(SOC).o >>> >>> SOBJS = reset.o >>> >>> COBJS-$(CONFIG_S3C6400) += cpu_init.o speed.o >>> >>> -COBJS-y += timer.o >>> +COBJS-y += timer.o init.o >>> >>> OBJS := $(addprefix $(obj),$(SOBJS) $(COBJS-y)) >>> >>> diff --git a/arch/arm/cpu/arm1176/s3c64xx/init.c >>> b/arch/arm/cpu/arm1176/s3c64xx/init.c new file mode 100644 >>> index 0000000..f113d8e >>> --- /dev/null >>> +++ b/arch/arm/cpu/arm1176/s3c64xx/init.c >>> @@ -0,0 +1,26 @@ >>> +/* >>> + * (C) Copyright 2012 Ashok Kumar Reddy Kourla >>> + * ashokkourla2000 at gmail.com >>> + * >>> + * See file CREDITS for list of people who contributed to this >>> + * project. >>> + * >>> + * This program is free software; you can redistribute it and/or >>> + * modify it under the terms of the GNU General Public License as >>> + * published by the Free Software Foundation; either version 2 of >>> + * the License, or (at your option) any later version. >>> + * >>> + * This program is distributed in the hope that it will be useful, >>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >>> + * GNU General Public License for more details. >>> + */ >>> + >>> +#include >>> + >>> +int arch_cpu_init(void) >>> +{ >>> + icache_enable(); >>> + >>> + return 0; >>> +} > > Best regards, > Marek Vasut > Regards, Ashok