[PATCH] drm/bridge: tc358767: don't warn if display side ASSR enable fails

Archit Taneja architt at codeaurora.org
Thu Dec 1 04:26:54 UTC 2016



On 11/30/2016 05:18 PM, Lucas Stach wrote:
> ASSR is an optional feature, so it's a valid operating condition for
> the display to reject ASSR enable. Demote the warning to the debug
> level.

Lgtm. Will pull it if Philipp or Andrey don't have any comments on it.

Thanks,
Archit

>
> Signed-off-by: Lucas Stach <l.stach at pengutronix.de>
> ---
>  drivers/gpu/drm/bridge/tc358767.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
> index 44d476ea6d2e..a7872400eed8 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -908,7 +908,7 @@ static int tc_main_link_setup(struct tc_data *tc)
>  			goto err_dpcd_read;
>
>  		if (tmp[0] != tc->assr) {
> -			dev_warn(dev, "Failed to switch display ASSR to %d, falling back to unscrambled mode\n",
> +			dev_dbg(dev, "Failed to switch display ASSR to %d, falling back to unscrambled mode\n",
>  				 tc->assr);
>  			/* trying with disabled scrambler */
>  			tc->link.scrambler_dis = 1;
>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project


More information about the dri-devel mailing list