On Thu, Feb 27, 2014 at 03:03:44PM +0100, Philippe Rétornaz wrote: > Hello > > This patch adds support for the mx31moboard board to bareboot. > I've based this on the pcm037 board with some minor modifications > in the clock and SDRAM initialisation. > > We were using redboot until now, barebox ease of use is quite a > shock :) Yeah, redboot is not exactly famous for good usability ;) > > BTW, I suspect there is a typo in the clock initialisation while > writing the CCM_PDR0 register on the pcm037 board: > > writel(MX31_PDR0_CSI_PODF(0xff1) | \ > MX31_PDR0_PER_PODF(7) | \ > MX31_PDR0_HSP_PODF(3) | \ > MX31_PDR0_NFC_PODF(5) | \ > MX31_PDR0_IPG_PODF(1) | \ > MX31_PDR0_MAX_PODF(3) | \ > MX31_PDR0_MCU_PODF(0), \ > MX31_CCM_BASE_ADDR + MX31_CCM_PDR0); > > The CSI_PODF field is only 9 bits wide, thus, MX31_PDR0_CSI_PODF(0xff1) > should better be MX31_PDR0_CSI_PODF(0x1ff). Thanks for noting. This indeed looks buggy. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox