Hi, On Mon, Jul 3, 2023 at 6:22 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > > The DRM panel core already keeps track of if the panel is already > prepared so do not reimplement this. > > Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c | 12 ------------ > 1 file changed, 12 deletions(-) It does? Can you please point to where/when the DRM panel core keeps track of this? I know I've posted a patch for this at: https://lore.kernel.org/r/20230607144931.v2.2.I59b417d4c29151cc2eff053369ec4822b606f375@changeid ...but I haven't landed it because I'm still trying to get consensus on the rest of the series and a later patch in the series depends on it. If you have some evidence that my patch isn't needed, can you please point at it in the commit message? I would say at least that someone else seemed to agree that the core wasn't checking this [1], though I guess it's possible that person was running old code or was just as confused as I was. [1] https://lore.kernel.org/r/646e391f.810a0220.214ce.d680@xxxxxxxxxxxxx -Doug
