[U-Boot] [PATCH] add print_cpuinfo to s3c24x0

[U-Boot] [PATCH] add print_cpuinfo to s3c24x0

[David Müller]

Hello

This patch adds support for "print_cpuinfo" on Samsung s3c24x0
based targets. If activated by "#define CONFIG_DISPLAY_CPUINFO", the
chip ID and the 3 main cpu frequencies will be displayed.

Dave

Signed-off-by: David M??ller <d.mueller@elsoft.ch>

diff --git a/arch/arm/cpu/arm920t/s3c24x0/Makefile b/arch/arm/cpu/arm920t/s3c24x0/Makefile
index bd53724..0029700 100644
--- a/arch/arm/cpu/arm920t/s3c24x0/Makefile
+++ b/arch/arm/cpu/arm920t/s3c24x0/Makefile
@@ -26,6 +26,7 @@ include $(TOPDIR)/config.mk
 LIB	= $(obj)lib$(SOC).o
 
 COBJS-$(CONFIG_USE_IRQ) += interrupts.o
+COBJS-$(CONFIG_DISPLAY_CPUINFO)	+= cpu_info.o
 COBJS-y	+= speed.o
 COBJS-y	+= timer.o
 COBJS-y	+= usb.o
diff --git a/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c b/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c
new file mode 100644
index 0000000..14c5c6a
--- /dev/null
+++ b/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c
@@ -0,0 +1,54 @@
+/*
+ * (C) Copyright 2010
+ * David Mueller <d.mueller@elsoft.ch>
+ *
+ * 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.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <asm/io.h>
+#include <asm/arch/s3c24x0_cpu.h>
+
+typedef ulong (*getfreq)(void);
+
+static const getfreq freq_f[] = {
+	get_FCLK,
+	get_HCLK,
+	get_PCLK,
+};
+
+static const char freq_c[] = { 'F', 'H', 'P' };
+
+int print_cpuinfo(void)
+{
+	int i;
+	char buf[32];
+/* the S3C2400 seems to be lacking a CHIP ID register */
+#ifndef CONFIG_S3C2400
+	ulong cpuid;
+	struct s3c24x0_gpio * const gpio = s3c24x0_get_base_gpio();
+
+	cpuid = readl(&gpio->gstatus1);
+	printf("CPUID: %8lX\n", cpuid);
+#endif
+	for (i = 0; i < ARRAY_SIZE(freq_f); i++)
+		printf("%cCLK: %8s MHz\n", freq_c[i], strmhz(buf, freq_f[i]()));
+
+	return 0;
+}

[U-Boot] [PING] Re: [PATCH] add print_cpuinfo to s3c24x0

["David Müller (ELSOFT AG)"]

Hello

Any news?

http://lists.denx.de/pipermail/u-boot/2010-December/083032.html

[U-Boot] [PING #2] Re: [PATCH] add print_cpuinfo to s3c24x0

["David Müller (ELSOFT AG)"]

Hello

Anybody home?

http://lists.denx.de/pipermail/u-boot/2010-December/083032.html

[U-Boot] [PATCH] add print_cpuinfo to s3c24x0

[Albert ARIBAUD]

Hi David,

Le 03/12/2010 16:12, David M?ller a ?crit :
> Hello
>
> This patch adds support for "print_cpuinfo" on Samsung s3c24x0
> based targets. If activated by "#define CONFIG_DISPLAY_CPUINFO", the
> chip ID and the 3 main cpu frequencies will be displayed.
>
> Dave
>
> Signed-off-by: David M??ller<d.mueller@elsoft.ch>
>
> diff --git a/arch/arm/cpu/arm920t/s3c24x0/Makefile b/arch/arm/cpu/arm920t/s3c24x0/Makefile
> index bd53724..0029700 100644
> --- a/arch/arm/cpu/arm920t/s3c24x0/Makefile
> +++ b/arch/arm/cpu/arm920t/s3c24x0/Makefile
> @@ -26,6 +26,7 @@ include $(TOPDIR)/config.mk
>   LIB	= $(obj)lib$(SOC).o
>
>   COBJS-$(CONFIG_USE_IRQ) += interrupts.o
> +COBJS-$(CONFIG_DISPLAY_CPUINFO)	+= cpu_info.o
>   COBJS-y	+= speed.o
>   COBJS-y	+= timer.o
>   COBJS-y	+= usb.o
> diff --git a/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c b/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c
> new file mode 100644
> index 0000000..14c5c6a
> --- /dev/null
> +++ b/arch/arm/cpu/arm920t/s3c24x0/cpu_info.c
> @@ -0,0 +1,54 @@
> +/*
> + * (C) Copyright 2010
> + * David Mueller<d.mueller@elsoft.ch>
> + *
> + * 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.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +
> +#include<common.h>
> +#include<asm/io.h>
> +#include<asm/arch/s3c24x0_cpu.h>
> +
> +typedef ulong (*getfreq)(void);
> +
> +static const getfreq freq_f[] = {
> +	get_FCLK,
> +	get_HCLK,
> +	get_PCLK,
> +};
> +
> +static const char freq_c[] = { 'F', 'H', 'P' };
> +
> +int print_cpuinfo(void)
> +{
> +	int i;
> +	char buf[32];
> +/* the S3C2400 seems to be lacking a CHIP ID register */
> +#ifndef CONFIG_S3C2400
> +	ulong cpuid;
> +	struct s3c24x0_gpio * const gpio = s3c24x0_get_base_gpio();
> +
> +	cpuid = readl(&gpio->gstatus1);
> +	printf("CPUID: %8lX\n", cpuid);
> +#endif
> +	for (i = 0; i<  ARRAY_SIZE(freq_f); i++)
> +		printf("%cCLK: %8s MHz\n", freq_c[i], strmhz(buf, freq_f[i]()));
> +
> +	return 0;
> +}

For some reason and despite David's pings, I'd missed this roughly 
one-year-old patch. Applied to u-boot-arm/master, with apologies for the 
delay. :(

Note:

include/configs/VCMA9.h:/* to be activated as soon as s3c24x0 has 
print_cpuinfo
include/configs/smdk2410.h:/* may be activated as soon as s3c24x0 has 
print_cpuinfo

Amicalement,
-- 
Albert.