[U-Boot] [PATCH 3/6] arm: am57xx: cl-som-am57x: invoke clock API to enable/disable clocks

[Marek Vasut]

On 02/09/2017 08:01 AM, Uri Mashiach wrote:
> Invoke enable_usb_clocks during board_usb_init and disable_usb_clocks
> during board_usb_exit to enable and disable clocks respectively.
> 
> Modifications:
> * Enable USB clocks in the OMAP version of the function
>   board_usb_init.
> * Disable USB clocks in the OMAP version of the function
>   board_usb_cleanup.
> 
> Signed-off-by: Uri Mashiach <uri.mashiach at compulab.co.il>

Reviewed-by: Marek Vasut <marex at denx.de>

> ---
>  board/compulab/cl-som-am57x/cl-som-am57x.c | 10 ----------
>  drivers/usb/host/xhci-omap.c               |  3 +++
>  2 files changed, 3 insertions(+), 10 deletions(-)
> 
> diff --git a/board/compulab/cl-som-am57x/cl-som-am57x.c b/board/compulab/cl-som-am57x/cl-som-am57x.c
> index fe1468f..4701b71 100644
> --- a/board/compulab/cl-som-am57x/cl-som-am57x.c
> +++ b/board/compulab/cl-som-am57x/cl-som-am57x.c
> @@ -53,16 +53,6 @@ int board_mmc_init(bd_t *bis)
>  }
>  #endif /* CONFIG_GENERIC_MMC */
>  
> -#ifdef CONFIG_USB_XHCI_OMAP
> -int omap_xhci_board_usb_init(int index, enum usb_init_type init)
> -{
> -	setbits_le32((*prcm)->cm_l3init_usb_otg_ss1_clkctrl,
> -		     OTG_SS_CLKCTRL_MODULEMODE_HW | OPTFCLKEN_REFCLK960M);
> -
> -	return 0;
> -}
> -#endif /* CONFIG_USB_XHCI_OMAP */
> -
>  int misc_init_r(void)
>  {
>  	cl_print_pcb_info();
> diff --git a/drivers/usb/host/xhci-omap.c b/drivers/usb/host/xhci-omap.c
> index ceee12a..f32fccc 100644
> --- a/drivers/usb/host/xhci-omap.c
> +++ b/drivers/usb/host/xhci-omap.c
> @@ -29,6 +29,7 @@ static struct omap_xhci omap;
>  
>  __weak int __omap_xhci_board_usb_init(int index, enum usb_init_type init)
>  {
> +	enable_usb_clocks(index);
>  	return 0;
>  }
>  
> @@ -40,8 +41,10 @@ int board_usb_init(int index, enum usb_init_type init)
>  	return omap_xhci_board_usb_init(index, init);
>  }
>  
> +
>  __weak int __omap_xhci_board_usb_cleanup(int index, enum usb_init_type init)
>  {
> +	disable_usb_clocks(index);
>  	return 0;
>  }
>  
> 


-- 
Best regards,
Marek Vasut