Re: [PATCH 1/2] drm/panel: boe-tv101wum-nl6: Drop macros and open code sequences
From: Sam Ravnborg
Date: Fri Jun 16 2023 - 12:43:55 EST
Hi Linus,
On Thu, Jun 15, 2023 at 10:21:37PM +0200, Linus Walleij wrote:
> The boe-tv101wum-nl6 is reinventing the mechanism to send command
> sequences that we usually nix during review, but I missed this one
> so fixing it up myself.
>
> Also use the explicit function calls to mipi_dsi_dcs_exit_sleep_mode()
> and mipi_dsi_dcs_set_display_on() instead of reimplementing them
> with homegrown sequences.
It looks like the same sequence of calls are used in all init functions,
maybe move them out to the caller.
>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
I have browsed the patch, but it was impossible for me to review in
details due to the massive changes.
But the final result looks good!
With the static added as the robot noticed:
Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx>