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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

-Doug




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux