On Fri, Apr 23, 2021 at 1:00 PM Douglas Anderson <[email protected]> wrote: > > As of commit 5186421cbfe2 ("drm: Introduce epoch counter to > drm_connector") the drm_get_edid() function calls > drm_connector_update_edid_property() for us. There's no reason for us > to call it again. > > Signed-off-by: Douglas Anderson <[email protected]> > Reviewed-by: Bjorn Andersson <[email protected]>
Reviewed-by: Sean Paul <[email protected]> > --- > As Laurent pointed out [1] this is actually a pretty common > problem. His suggestion to do this more broadly is a good idea but > this series is probably a bit ambitious already so I would suggest > that be taken up separately. > > [1] https://lore.kernel.org/r/[email protected] > > (no changes since v1) > > drivers/gpu/drm/panel/panel-simple.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c > b/drivers/gpu/drm/panel/panel-simple.c > index bd208abcbf07..4de33c929a59 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -512,7 +512,6 @@ static int panel_simple_get_modes(struct drm_panel *panel, > if (p->ddc) { > struct edid *edid = drm_get_edid(connector, p->ddc); > > - drm_connector_update_edid_property(connector, edid); > if (edid) { > num += drm_add_edid_modes(connector, edid); > kfree(edid); > -- > 2.31.1.498.g6c1eba8ee3d-goog > > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
