On 12/19/2011 02:37 AM, Igor Grinberg wrote: > Hi Peter, > > Thanks for fixing all the issues pointed. > I have one last question (sorry for not seeing it earlier) and one neat below. > > On 12/18/11 19:25, Peter Barada wrote: >> This patch adds basic support for OMAP35x/DM37x SOM LV/Torpedo >> reference boards. It assumes U-boot is loaded to SDRAM with the >> help of another small bootloader (x-load) running from SRAM. >> >> Signed-off-by: Peter Barada <peter.bar...@logicpd.com> >> Cc: Tom Rini <tom.r...@gmail.com> >> Cc: Igor Grinberg <grinb...@compulab.co.il> >> Cc: Wolfgang Denk <w...@denx.de> >> --- [ ... ] > > > The TRM says that the initial state of pins is the safe mode (M7). > Also not every pin has this state by definition. > So the question is, wouldn't it be better to remove those lines > that setup the safe mode, or is there a purpose, which I don't see? > Removing those lines will make the file/patch much shorter. Back when we first started with u-boot on these boards, it was loaded from another bootloader that did its own pinmux setup and this was how I put all the pins back to a known state. Moving to SPL/u-boot will preclude having to tweak all the pins.
I'll trim the pinmux setup back to only those pins that are different than their reset values (or those that need PTD/EN pulldowns to minimise power consumption when the kernel goes into suspend). >> +/* DDR - I use Micron DDR */ >> +#define CONFIG_OMAP3_MICRON_DDR > You forgot to remove that one - it is not used anymore... Done... > [...] > > -- > Regards, > Igor. -- Peter Barada peter.bar...@logicpd.com _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
