Hi Doug,
On Fri, Apr 30, 2021 at 11:04 PM Doug Anderson dianders@chromium.org wrote:
Pushed this one patch. Rest of the series is pending adult supervision. Overall summary:
- I could probably push some of the early sn65dsi86 cleanup patches
in this series since they have Bjorn's review and are pretty much no-ops / simple cleanups, but there's probably not tons gained for shoving those in early.
Those look good to me as well. I'd say just apply them.
To me it looks like up until and including patch 18? Feel free to add my Acked-by: Linus Walleij linus.walleij@linaro.org
On these.
- The whole concept of breaking up the patch into sub-drivers has no
official Reviewed-by tags yet. Presumably Bjorn will give those a re-review when he has time again.
It looks good to me so I sent an explicit ACK on that patch.
- Laurent and I had a big discussion on #dri-devel yesterday about
the EDID reading. He's not totally convinced with the idea of doing this in the panel when the bridge could just do it by itself, but it sounded like he might be coming around. Right now this is waiting on Laurent to have time to get back to this.
I dare not speak of this. Laurent has the long and tedious experience with panels and pretty much anything related so if Laurent is hesitant then I am hesitant too. His buy-in is absolutely required.
But IIUC that is just for patch 19+20?
It'd be good to apply the rest and get down the stack.
Just to keep you busy and make sure you don't run out of work (haha) I noticed that the gpio_chip in this driver can use the new GPIO_REGMAP helper library with the fixes just landed in Torvald's tree.
At your convenience and when you think there is too little stuff in your sn65dsi86 TODO, check out pinctrl-bcm63xx.c for an example of select GPIO_REGMAP made very simple (this works fine as long as they are bit offsets starting from 0).
Yours, Linus Walleij