* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx @ 2012-09-26 18:29 Ashok Kumar Reddy 2012-09-30 11:24 ` Ashok Kumar Reddy 2012-10-26 8:14 ` Minkyu Kang 0 siblings, 2 replies; 6+ messages in thread From: Ashok Kumar Reddy @ 2012-09-26 18:29 UTC (permalink / raw) To: u-boot ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx 2012-09-26 18:29 [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx Ashok Kumar Reddy @ 2012-09-30 11:24 ` Ashok Kumar Reddy 2012-09-30 13:21 ` Marek Vasut 2012-10-26 8:14 ` Minkyu Kang 1 sibling, 1 reply; 6+ messages in thread From: Ashok Kumar Reddy @ 2012-09-30 11:24 UTC (permalink / raw) To: u-boot Hi Marek Vasut , Hi Stephen Warren Please provide comments for the below patch, as i forgot to add you in cc for below mail. 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 <ashokkourla2000@gmail.com> > 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 <ashokkourla2000@gmail.com> > --- > 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<common.h> > + > +int arch_cpu_init(void) > +{ > + icache_enable(); > + > + return 0; > +} > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx 2012-09-30 11:24 ` Ashok Kumar Reddy @ 2012-09-30 13:21 ` Marek Vasut 2012-10-01 3:10 ` Ashok Kumar Reddy 0 siblings, 1 reply; 6+ messages in thread From: Marek Vasut @ 2012-09-30 13:21 UTC (permalink / raw) To: u-boot 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? > 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 <ashokkourla2000@gmail.com> > > 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 <ashokkourla2000@gmail.com> > > --- > > > > 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<common.h> > > + > > +int arch_cpu_init(void) > > +{ > > + icache_enable(); > > + > > + return 0; > > +} Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx 2012-09-30 13:21 ` Marek Vasut @ 2012-10-01 3:10 ` Ashok Kumar Reddy 2012-10-01 8:56 ` Marek Vasut 0 siblings, 1 reply; 6+ messages in thread From: Ashok Kumar Reddy @ 2012-10-01 3:10 UTC (permalink / raw) To: u-boot 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 <ashokkourla2000@gmail.com> >>> 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 <ashokkourla2000@gmail.com> >>> --- >>> >>> 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<common.h> >>> + >>> +int arch_cpu_init(void) >>> +{ >>> + icache_enable(); >>> + >>> + return 0; >>> +} > > Best regards, > Marek Vasut > Regards, Ashok ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx 2012-10-01 3:10 ` Ashok Kumar Reddy @ 2012-10-01 8:56 ` Marek Vasut 0 siblings, 0 replies; 6+ messages in thread From: Marek Vasut @ 2012-10-01 8:56 UTC (permalink / raw) To: u-boot Dear Ashok Kumar Reddy, > 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 Ok, CCing Albert. > >> 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 <ashokkourla2000@gmail.com> > >>> 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 <ashokkourla2000@gmail.com> > >>> --- > >>> > >>> 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<common.h> > >>> + > >>> +int arch_cpu_init(void) > >>> +{ > >>> + icache_enable(); > >>> + > >>> + return 0; > >>> +} > > > > Best regards, > > Marek Vasut > > Regards, > Ashok Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx 2012-09-26 18:29 [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx Ashok Kumar Reddy 2012-09-30 11:24 ` Ashok Kumar Reddy @ 2012-10-26 8:14 ` Minkyu Kang 1 sibling, 0 replies; 6+ messages in thread From: Minkyu Kang @ 2012-10-26 8:14 UTC (permalink / raw) To: u-boot Dear Ashok Kumar Reddy, On 27 September 2012 03:29, Ashok Kumar Reddy <ashokkourla2000@gmail.com> wrote: > From 0de61b9690c5f136ba5d3028be43877ea455d843 Mon Sep 17 00:00:00 2001 > From: Ashok Kumar Reddy <ashokkourla2000@gmail.com> > 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 <ashokkourla2000@gmail.com> > --- > 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 > applied to u-boot-samsung. Thanks. Minkyu Kang. -- from. prom. www.promsoft.net ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-10-26 8:14 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-09-26 18:29 [U-Boot] [PATCH] ARM: arm1176: Define arch_cpu_init() for s3c64xx Ashok Kumar Reddy 2012-09-30 11:24 ` Ashok Kumar Reddy 2012-09-30 13:21 ` Marek Vasut 2012-10-01 3:10 ` Ashok Kumar Reddy 2012-10-01 8:56 ` Marek Vasut 2012-10-26 8:14 ` Minkyu Kang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox