Re: [PATCH v3] drm/panel: raydium-rm692e5: transition to mipi_dsi wrapped functions

[neil . armstrong]

On 20/06/2024 21:18, Doug Anderson wrote:
> Hi,
>
> On Thu, Jun 20, 2024 at 11:12 AM Tejas Vipin <tejasvipin76@xxxxxxxxx> wrote:
> > Use functions introduced in commit 966e397e4f60 ("drm/mipi-dsi:
> > Introduce mipi_dsi_*_write_seq_multi()") and commit f79d6d28d8fe
> > ("drm/mipi-dsi: wrap more functions for streamline handling") for the
> > raydium-rm692e5 panel.
> >
> > Additionally, the error handling in rm692e5_prepare() is changed to
> > properly power the panel off in the case of a wider range of
> > initialization commands failing than before.
> >
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
> > Signed-off-by: Tejas Vipin <tejasvipin76@xxxxxxxxx>
> > ---
> > Changes in v3:
> >      - Remove extra blank line
> >      - Expanded on error handling functionality in patch notes
> >
> > Changes in v2:
> >      - Change rm692e5_on to return void and take mipi_dsi_multi_context
> >        as an argument.
> >      - Remove unnecessary warnings.
> >      - More efficient error handling in rm692e5_prepare
> >
> > v1: https://lore.kernel.org/all/20240615093758.65431-1-tejasvipin76@xxxxxxxxx/
> >
> > v2: https://lore.kernel.org/all/20240619033351.230929-1-tejasvipin76@xxxxxxxxx/
> > ---
> >   drivers/gpu/drm/panel/panel-raydium-rm692e5.c | 238 ++++++++----------
> >   1 file changed, 99 insertions(+), 139 deletions(-)
>
> Thanks! I normally let non-urgent things sit for a week or so before
> applying them, but I'll be on vacation for some of next week and the
> one after. If this is still pending when I get back then I'll try to
> apply it as I'm getting through my inbox, though I'd also be happy if
> someone else (maybe Neil?) wanted to apply it next week.

Enjoy your vacations! I'll handle it.

Thanks,
Neil

> -Doug