Thomas Zimmermann <[email protected]> writes:

Hello Thomas,

> The plane state in __drm_gem_reset_shadow_plane) can be NULL. Do not

That ) is off. I guess you meant either () or just the function name.

> deref that pointer, but forward NULL to the other plane-reset helpers.
> Clears plane->state to NULL.
>
> Signed-off-by: Thomas Zimmermann <[email protected]>
> Fixes: b71565022031 ("drm/gem: Export implementation of shadow-plane helpers")
> Reported-by: Dan Carpenter <[email protected]>
> Closes: https://lore.kernel.org/dri-devel/[email protected]/
> Cc: Thomas Zimmermann <[email protected]>
> Cc: Melissa Wen <[email protected]>
> Cc: Maarten Lankhorst <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: David Airlie <[email protected]>
> Cc: Simona Vetter <[email protected]>
> Cc: [email protected]
> Cc: <[email protected]> # v5.15+
> ---
>  drivers/gpu/drm/drm_gem_atomic_helper.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>

Reviewed-by: Javier Martinez Canillas <[email protected]>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

Reply via email to