Re: [PATCH] drm/amd/pm/smu13: BACO is supported when it's in BACO state

["Lazar, Lijo" <lijo.lazar@xxxxxxx>]

On 1/10/2023 9:07 AM, Guchun Chen wrote:
> This leverages the logc in smu11. No need to talk to SMU to
> check BACO enablement as it's in BACO state already.
>
> Signed-off-by: Guchun Chen <guchun.chen@xxxxxxx>

Reviewed-by: Lijo Lazar <lijo.lazar@xxxxxxx>

Thanks,
Lijo
> ---
>   drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> index ccaedfcf977e..78945e79dbee 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
> @@ -2243,6 +2243,10 @@ bool smu_v13_0_baco_is_support(struct smu_context *smu)
>   	    !smu_baco->platform_support)
>   		return false;
>   
> +	/* return true if ASIC is in BACO state already */
> +	if (smu_v13_0_baco_get_state(smu) == SMU_BACO_STATE_ENTER)
> +		return true;
> +
>   	if (smu_cmn_feature_is_supported(smu, SMU_FEATURE_BACO_BIT) &&
>   	    !smu_cmn_feature_is_enabled(smu, SMU_FEATURE_BACO_BIT))
>   		return false;