[U-Boot] [PATCH] board: dra71: Fix selection of OPPs

Lokesh Vutla lokeshvutla at ti.com
Thu Apr 20 08:39:43 UTC 2017



On Sunday 16 April 2017 10:13 AM, Lokesh Vutla wrote:
> As per the DM[1] Dated June 2016–Revised February 2017, Table 5-3,
> DRA71 supports the following OPPs for various voltage domains:
> 
> VDD_MPU:	OPP_NOM
> VDD_CORE:	OPP_NOM
> VDD_GPU:	OPP_NOM
> VDD_DSPEVE:	OPP_NOM, OPP_HIGH
> VDD_IVA:	OPP_NOM, OPP_HIGH
> 
> This patch add support for selection of the above OPPs instead of
> using OPP_NOM for all voltage domains.
> 
> [1] http://www.ti.com/lit/ds/symlink/dra718.pdf
> 
> Reported-by: Vishal Mahaveer <vishalm at ti.com>
> Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>

Just posted v2 of this patch as fixing efuse reg bits for OPP_HIGH as
pointed out by Suman.

Thanks and regards,
Lokesh

> ---
>  board/ti/dra7xx/evm.c | 29 +++++++++++++++++++++++------
>  1 file changed, 23 insertions(+), 6 deletions(-)
> 
> diff --git a/board/ti/dra7xx/evm.c b/board/ti/dra7xx/evm.c
> index ae2d59da43..0984475d3b 100644
> --- a/board/ti/dra7xx/evm.c
> +++ b/board/ti/dra7xx/evm.c
> @@ -456,27 +456,44 @@ int get_voltrail_opp(int rail_offset)
>  {
>  	int opp;
>  
> -	/*
> -	 * DRA71x supports only OPP_NOM.
> -	 */
> -	if (board_is_dra71x_evm())
> -		return OPP_NOM;
> -
>  	switch (rail_offset) {
>  	case VOLT_MPU:
>  		opp = DRA7_MPU_OPP;
> +		/* DRA71x supports only OPP_NOM for MPU */
> +		if (board_is_dra71x_evm())
> +			opp = OPP_NOM;
>  		break;
>  	case VOLT_CORE:
>  		opp = DRA7_CORE_OPP;
> +		/* DRA71x supports only OPP_NOM for CORE */
> +		if (board_is_dra71x_evm())
> +			opp = OPP_NOM;
>  		break;
>  	case VOLT_GPU:
>  		opp = DRA7_GPU_OPP;
> +		/* DRA71x supports only OPP_NOM for GPU */
> +		if (board_is_dra71x_evm())
> +			opp = OPP_NOM;
>  		break;
>  	case VOLT_EVE:
>  		opp = DRA7_DSPEVE_OPP;
> +		/*
> +		 * DRA71x does not support OPP_OD for EVE.
> +		 * If OPP_OD is selected by menuconfig, fallback
> +		 * to OPP_NOM.
> +		 */
> +		if (board_is_dra71x_evm() && opp == OPP_OD)
> +			opp = OPP_NOM;
>  		break;
>  	case VOLT_IVA:
>  		opp = DRA7_IVA_OPP;
> +		/*
> +		 * DRA71x does not support OPP_OD for IVA.
> +		 * If OPP_OD is selected by menuconfig, fallback
> +		 * to OPP_NOM.
> +		 */
> +		if (board_is_dra71x_evm() && opp == OPP_OD)
> +			opp = OPP_NOM;
>  		break;
>  	default:
>  		opp = OPP_NOM;
> 


More information about the U-Boot mailing list