Hi,Linus Walleij On Mon, Jul 3, 2023 at 9:21 PM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > > This is two patches fixing things I would normally complain about > in reviews, but alas I missed this one, so I go in and fix it up > myself. > > Discovering that a completely unrelated driver has been merged > into this panel driver I had to bite the bullet and break it out. > I am pretty suspicious of the other recently added panel as well. Do you think the starry,himax83102-j02 panel needs to break it out? Thanks. > > I am surprised that contributors from manufacturers do not seem > to have datasheets for the display controllers embedded in the > panels of their products. Can you take a second look? Sorry, this panel datasheet is not open source, I can't share this datasheet. > > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > Changes in v3: > - Rebase on drm-misc-next > - Convert the two newly added Starry panels as well. > - Break out the obvious ILI9882t-based panel into its own driver. > - Link to v2: https://lore.kernel.org/r/20230615-fix-boe-tv101wum-nl6-v2-0-457d7ece4590@xxxxxxxxxx > > Changes in v2: > - Fix a missed static keyword > - Link to v1: https://lore.kernel.org/r/20230615-fix-boe-tv101wum-nl6-v1-0-8ac378405fb7@xxxxxxxxxx > > --- > Linus Walleij (4): > drm/panel: boe-tv101wum-nl6: Drop macros and open code sequences > drm/panel: boe-tv101wum-nl6: Drop surplus prepare tracking > drm/panel: ili9882t: Break out as separate driver > drm/panel: ili9882t: Break out function for switching page > > drivers/gpu/drm/panel/Kconfig | 9 + > drivers/gpu/drm/panel/Makefile | 1 + > drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c | 3037 ++++++++++-------------- > drivers/gpu/drm/panel/panel-ilitek-ili9882t.c | 759 ++++++ > 4 files changed, 2067 insertions(+), 1739 deletions(-) > --- > base-commit: 14806c6415820b1c4bc317655c40784d050a2edb > change-id: 20230615-fix-boe-tv101wum-nl6-6aa3fab22b44 > > Best regards, > -- > Linus Walleij <linus.walleij@xxxxxxxxxx> >