[Intel-gfx] [PATCH 3/9] drm/i915/dsb: Turn the "DSB is busy" into an error

[Sharma, Swati2]

On 29-Jan-20 11:50 PM, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> The DSB should never be busy here or we messed up somewhere else.
> Make this a DRM_ERROR().

This is required, at-least to catch the issue in CI.

Reviewed-by: Swati Sharma <swati2.sharma at intel.com>

> 
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>   drivers/gpu/drm/i915/display/intel_dsb.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_dsb.c b/drivers/gpu/drm/i915/display/intel_dsb.c
> index a16a78cd0ebe..5bb4ac1610a5 100644
> --- a/drivers/gpu/drm/i915/display/intel_dsb.c
> +++ b/drivers/gpu/drm/i915/display/intel_dsb.c
> @@ -52,7 +52,7 @@ static inline bool intel_dsb_enable_engine(struct intel_dsb *dsb)
>   
>   	dsb_ctrl = intel_de_read(dev_priv, DSB_CTRL(pipe, dsb->id));
>   	if (DSB_STATUS & dsb_ctrl) {
> -		DRM_DEBUG_KMS("DSB engine is busy.\n");
> +		DRM_ERROR("DSB engine is busy.\n");
>   		return false;
>   	}
>   
> 

-- 
~Swati Sharma