Skip to contentSkip to conversation

MX28: extend print_cpuinfo() to use chip information

PATCH Superseded Otavio Salvador 2 participants 4 messages Jul 28, 2012 – Jul 29, 2012 Arm (32-bit)

lore
Share & tools
lore

Otavio Salvador

author
#0 lore raw

PATCH v4 1/1 MX28: extend print_cpuinfo() to use chip information

The information now is gathered from HW_DIGCTL_CHIPID register and includes the chip modem and revision on the output.

Signed-off-by Otavio Salvador <[email protected]>

---
Changes in v2:
- use ?? for unidentified revision and cpu type
- use numeric revisions

Changes in v3:
- drop mx23 data as it will be posted in a later patch
- fix info order (first rev, then clock)

Changes in v4:
- change short-description prefix to MX28

 arch/arm/cpu/arm926ejs/mx28/mx28.c           |   38 ++++++++++++++++++++++++--
 arch/arm/include/asm/arch-mx28/regs-digctl.h |    4 +++
 2 files changed, 40 insertions(+), 2 deletions(-)
arch/arm/cpu/arm926ejs/mx28/mx28.c +36 -2
index ff25772..cf7a50f 100644
@@ -188,13 +188,47 @@ int arch_cpu_init(void)
 }
 
 #if defined(CONFIG_DISPLAY_CPUINFO)
+static const char *get_cpu_type(void)
+{
+	struct mx28_digctl_regs *digctl_regs =
+		(struct mx28_digctl_regs *)MXS_DIGCTL_BASE;
+
+	switch (readl(&digctl_regs->hw_digctl_chipid) & HW_DIGCTL_CHIPID_MASK) {
+	case HW_DIGCTL_CHIPID_MX28:
+		return "28";
+	default:
+		return "??";
+	}
+}
+
+static const char *get_cpu_rev(void)
+{
+	struct mx28_digctl_regs *digctl_regs =
+		(struct mx28_digctl_regs *)MXS_DIGCTL_BASE;
+	uint8_t rev = readl(&digctl_regs->hw_digctl_chipid) & 0x000000FF;
+
+	switch (readl(&digctl_regs->hw_digctl_chipid) & HW_DIGCTL_CHIPID_MASK) {
+	case HW_DIGCTL_CHIPID_MX28:
+		switch (rev) {
+		case 0x1:
+			return "1.2";
+		default:
+			return "??";
+		}
+	default:
+		return "??";
+	}
+}
+
 int print_cpuinfo(void)
 {
 	struct mx28_spl_data *data = (struct mx28_spl_data *)
 		((CONFIG_SYS_TEXT_BASE - sizeof(struct mx28_spl_data)) & ~0xf);
 
-	printf("Freescale i.MX28 family at %d MHz\n",
-			mxc_get_clock(MXC_ARM_CLK) / 1000000);
+	printf("CPU:   Freescale i.MX%s rev%s at %d MHz\n",
+		get_cpu_type(),
+		get_cpu_rev(),
+		mxc_get_clock(MXC_ARM_CLK) / 1000000);
 	printf("BOOT:  %s\n", mx28_boot_modes[data->boot_mode_idx].mode);
 	return 0;
 }
arch/arm/include/asm/arch-mx28/regs-digctl.h +4 -0
index 9a63594..247da6e 100644
@@ -152,4 +152,8 @@ struct mx28_digctl_regs {
 };
 #endif
 
+/* Product code identification */
+#define HW_DIGCTL_CHIPID_MASK	(0xffff << 16)
+#define HW_DIGCTL_CHIPID_MX28	(0x2800 << 16)
+
 #endif /* __MX28_REGS_DIGCTL_H__ */
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH v4 1/1 MX28: extend print_cpuinfo() to use chip information

Quoted 5 lines · Otavio Salvador

The information now is gathered from HW_DIGCTL_CHIPID register and includes the chip modem and revision on the output.

Signed-off-by Otavio Salvador <[email protected]>

---

Applied to u-boot-imx, next branch, thanks.

Best regards,
Stefano Babic

Signature · 5 lines
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
#2 lore raw

PATCH v4 1/1 MX28: extend print_cpuinfo() to use chip information

In reply to Otavio Salvador
Quoted 9 lines · Otavio Salvador

The information now is gathered from HW_DIGCTL_CHIPID register and includes the chip modem and revision on the output.

Signed-off-by Otavio Salvador <[email protected]>

---
Changes in v2:
- use ?? for unidentified revision and cpu type
- use numeric revisions

Quoted 7 lines

Changes in v3:
- drop mx23 data as it will be posted in a later patch
- fix info order (first rev, then clock)

Changes in v4:
- change short-description prefix to MX28

Hi Otavio,

I was convinced I had already applied your patch, but I forgot to push my local -next tree to the server to make it public. Sorry for delay. I took in any case your V4, even if V3 was already ok :-)

Applied to u-boot-imx, next branch, thanks.

Best regards,
Stefano Babic

Signature · 5 lines
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
author
#3 lore raw

PATCH v4 1/1 MX28: extend print_cpuinfo() to use chip information

Quoted 10 lines · Stefano Babic
Quoted 4 lines · Otavio Salvador

The information now is gathered from HW_DIGCTL_CHIPID register and includes the chip modem and revision on the output.

Signed-off-by Otavio Salvador <[email protected]>

I was convinced I had already applied your patch, but I forgot to push my local -next tree to the server to make it public. Sorry for delay. I took in any case your V4, even if V3 was already ok :-)

Applied to u-boot-imx, next branch, thanks.

No problem. Thanks!

Signature · 3 lines
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br