Hi Stefano, > > On 08/01/2014 08:24, Christian Gmeiner wrote: >> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com> >> --- >> board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg >> b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg >> index bb6c60b..b9e107a 100644 >> --- a/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg >> +++ b/board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg >> @@ -144,7 +144,7 @@ DATA 4 0x021b0404 0x00011006 >> >> /* set the default clock gate to save power */ >> DATA 4 0x020c4068 0x00C03F3F >> -DATA 4 0x020c406c 0x0030FC03 >> +DATA 4 0x020c406c 0x0030FC33 >> DATA 4 0x020c4070 0x0FFFC000 >> DATA 4 0x020c4074 0x3FF00000 >> DATA 4 0x020c4078 0x00FFF300 >> > > I do not think a good idea to enable the clock here. We have to set only > the clocks that are required for U-Boot, letting the other ones off to > save power. This is a common file, and then all boards using it will > have the ecspi-3 clock turned on, even if they do not require it. > > Better is to set the clock inside the board file only for the boards > (yours !) that need it, for example in board_early_init() >
Sounds like a good plan... will rework this part for the next version. thanks -- Christian Gmeiner, MSc _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot