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]

 



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




[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