[PATCH v2] drm/imx: ipuv3-plane: fix PRG modifiers after drm managed resource conversion

Philipp Zabel p.zabel at pengutronix.de
Mon May 10 15:21:32 UTC 2021


On Mon, 2021-05-10 at 16:59 +0200, Lucas Stach wrote:
> The conversion to drm managed resources introduced two bugs: the plane is now
> always initialized with the linear-only list, while the list with the Vivante
> GPU modifiers should have been used when the PRG/PRE engines are present. This
> masked another issue, as ipu_plane_format_mod_supported() is now called before
> the private plane data is set up, so if a non-linear modifier is supplied in
> the plane modifier list, we run into a NULL pointer dereference checking for
> the PRG presence. To fix this just remove the check from this function, as we
> know that it will only be called with a non-linear modifier, if the plane init
> code has already determined that the PRG/PRE is present.
> 
> Fixes: 699e7e543f1a ("drm/imx: ipuv3-plane: use drm managed resources")
> Signed-off-by: Lucas Stach <l.stach at pengutronix.de>
> ---
> v2: Add proper subject.

Whoops, thank you. I've replaced the previous patch.

regards
Philipp


More information about the dri-devel mailing list