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@xxxxxxxxxxxxxx> > --- > v2: Add proper subject. Whoops, thank you. I've replaced the previous patch. regards Philipp