On Mon, Dec 2, 2019 at 8:33 PM Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > The panel drivers used drm_panel.drm for two purposes: > 1) Argument to drm_mode_duplicate() > 2) drm->dev was used in error messages > > The first usage is replaced with drm_connector.dev > - drm_connector is already connected to a drm_device > and we have a valid connector > > The second usage is replaced with drm_panel.dev > - this makes drivers more consistent in their dev argument > used for dev_err() and friends > > With these replacements there are no more uses of drm_panel.drm, > so it is removed from struct drm_panel. > With this change drm_panel_attach() and drm_panel_detach() > no logner has any use as they are empty functions. > > Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > Cc: Thierry Reding <thierry.reding@xxxxxxxxx> > Cc: Laurent Pinchart <Laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Maxime Ripard <mripard@xxxxxxxxxx> > Cc: David Airlie <airlied@xxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Cc: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > Cc: Stefan Mavrodiev <stefan@xxxxxxxxxx> > Cc: Robert Chiras <robert.chiras@xxxxxxx> > Cc: "Guido Günther" <agx@xxxxxxxxxxx> > Cc: Purism Kernel Team <kernel@xxxxxxx> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij