Hi Tero,
[snip]

> @@ -378,7 +378,6 @@ void enable_basic_clocks(void)
>                &prcm->cm_l4per_gptimer2_clkctrl,
>                &prcm->cm_wkup_wdtimer2_clkctrl,
>                &prcm->cm_l4per_uart3_clkctrl,
> -               &prcm->cm_l3init_fsusb_clkctrl,
>                &prcm->cm_l3init_hsusbhost_clkctrl,
>                0
>        };

It should not have been in both places.
Removing here is fine.

> @@ -505,7 +504,6 @@ void enable_non_essential_clocks(void)
>                &prcm->cm_dss_dss_clkctrl,
>                &prcm->cm_sgx_sgx_clkctrl,
>                &prcm->cm_l3init_hsusbhost_clkctrl,
> -               &prcm->cm_l3init_fsusb_clkctrl,
>                0
>        };
 But instead of removing this here,how about removing the macro
 CONFIG_SYS_CLOCKS_ENABLE_ALL in omap4_common.h.
 This will get rid  of all non essential clocks which are currently enabled.

 One problem might be non-essential dplls should still be enabled in
 u-boot because kernel expects  all the dplls to be in locked state during
 boot without which it would  start throwing slew of warnings.

 I will send a patch to clean this first.

Thanks,
 Sricharan
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to