Dear Otavio Salvador, > The information now is gathered from HW_DIGCTL_CHIPID register and > includes the revision of the chip on the output. > > Signed-off-by: Otavio Salvador <ota...@ossystems.com.br> > Cc: Marek Vasut <ma...@denx.de> > Cc: Stefano Babic <sba...@denx.de> > Cc: Fabio Estevam <fabio.este...@freescale.com> > --- > arch/arm/cpu/arm926ejs/mx28/mx28.c | 29 +++++++++++++++++++++++++++-- > 1 file changed, 27 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/cpu/arm926ejs/mx28/mx28.c > b/arch/arm/cpu/arm926ejs/mx28/mx28.c index a82ff25..ac2f2e0 100644 > --- a/arch/arm/cpu/arm926ejs/mx28/mx28.c > +++ b/arch/arm/cpu/arm926ejs/mx28/mx28.c > @@ -190,13 +190,38 @@ int arch_cpu_init(void) > #endif > > #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) >> 16) {
#define that >> 16 ... maybe offset? I'd define a mask though, see below. #define PRODUCT_CODE_MASK (0xffff << 16) > + case 0x2800: #define this as something ... #define PRODUCT_CODE_MX28 (0x2800 << 16) > + return "28"; > + case 0x3728: > + return "23"; > + default: > + return "<unidentified>"; So X-Files-ish ;-) > + } > +} > + > +static u8 get_cpu_rev(void) uint8_t ... maybe? it's used throughout the code, so let's keep consistent. > +{ > + struct mx28_digctl_regs *digctl_regs = > + (struct mx28_digctl_regs *)MXS_DIGCTL_BASE; > + > + return readl(&digctl_regs->hw_digctl_chipid) & 0x0000F; > +} > + > 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%d 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; > } Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot