00:54 <Dylanger> You can buy the Acer Chrombook 514 2022 now!

00:54 <Dylanger> https://www.acer.com/ac/en/US/content/model/NX.AS2AA.004

00:54 <Dylanger> Dat MediaTek Kompanio 828

01:46 <bamse> steev: https://lore.kernel.org/all/20220420231230.58499-1-bjorn.andersson@linaro.org/

03:24 <steev> bamse: so, will the 3rd patch paul sent not be included then?

03:33 <bamse> steev: i didn't spot that...

03:33 <steev> bamse: you did because you replied to it :P

03:34 <bamse> steev: was waiting for him to send something, but then i read his last reply and i thought he said that we should just give up

03:34 <robclark> fwiw I eventually realized that paul's patches where getting flagged as spam for me

03:34 <steev> i thought he was asking for... jagan (sp?) to chime in

03:34 <steev> https://patchwork.freedesktop.org/patch/481135/?series=102330&rev=1

03:35 <bamse> robclark: as i was writing my commit message i realized that the new form dsi { panel { }; }; wouldn't work for us (i mean the non-dp case)

03:35 <steev> tegra and ingenic have chimed in that it fixes their issues :(

03:35 <bamse> robclark: because we typically put the opp-table in there and 'o' sorts before most panels

03:35 <bamse> steev: the same thread that i replied to?

03:35 <steev> yes

03:36 <steev> in the link above

03:36 <steev> "Jagan, can you prove some insight on why this mechanism was needed in the first

03:36 <steev> place and if we can get rid of it without breaking any active binding?"

03:36 <steev> i mean, that patch does fix things for c630 and i was gonna send the t-b but then i saw you'd replied saying it still is broken for sc8180x

03:37 <robclark> bamse: still any solution that relies on alphabetical ordering to get lucky enough to work isn't great

03:38 <bamse> robclark: i would need to empty my head and look at the logic once more to convince me that the latest proposal will actually work

03:38 <bamse> robclark: i don't fancy the fact that we went from a trivial nice patch to 3 layers of band aid

03:39 <robclark> we are at -rc3 so revert-and-try-again-next-time isn't ever really a wrong answer.. but I hadn't seen the latest patch so not sure if that could work

04:00 <bamse> robclark: the last patch seems to be okay...

04:01 <bamse> i've not yet tested it, but the band aid logic seems resaonable to me

04:02 <bamse> robclark: it no longer suffer from the ordering problem of opp-table vs panel

04:02 <robclark> so, pretty sure I tried https://patchwork.freedesktop.org/patch/481135/?series=102330&rev=1 .. but would have to check tomorrow since it was on my workstation in office.. if there is something newer I can try it tomorrow

18:41 <steev> bamse: should i bother sending tested by when maxime has already applied to fixes?

18:59 <bamse> steev: doesn't sounds like there's a reason to do so, if the patch is up you may reply to my reverts and inform me that the fix merged solves your problem so you don't think my reverts are needed anymore

18:59 <bamse> steev: that will make sure that the maintainer(s) can pop them off their todo list

18:59 <steev> no no, the reverts are needed

19:04 <bamse> even with the third fix?

19:04 <bamse> maybe i'm just lost?

19:05 <steev> no, i removed that since it wasn't actually committed anywhere just posted, and your comment that things are still broken with that patch on other things