Hi, On Fri, Aug 9, 2024 at 9:55 PM Tejas Vipin <tejasvipin76@xxxxxxxxx> wrote: > > This patch adds mipi_dsi_dcs_set_tear_scanline_multi to the list of multi > functions and uses it with other multi functions in the jdi-fhd-r63452 > panel. > > Tejas Vipin (2): > drm/mipi-dsi: Add mipi_dsi_dcs_set_tear_scanline_multi > drm/panel: jdi-fhd-r63452: transition to mipi_dsi wrapped functions > > drivers/gpu/drm/drm_mipi_dsi.c | 31 +++++ > drivers/gpu/drm/panel/panel-jdi-fhd-r63452.c | 125 ++++++------------- > include/drm/drm_mipi_dsi.h | 2 + > 3 files changed, 72 insertions(+), 86 deletions(-) Not worth spinning just for this, but a few comments: 1. For the cover letter, it's better if you can make the subject more different than the subject of your previous patch series. Comparing this and the previous series you sent out side-by-side: [PATCH 0/2] add more multi functions for streamlined error handling [PATCH v3 0/2] add more multi functions to streamline error handling Maybe this patch's cover letter should have a subject more like: drm/mipi-dsi: convert jdi-fhd-r63452 to mipi_dsi "multi", adding more "multi" ...or something like that. 2. In your cover letter you should note that this series only applies cleanly if you apply it atop your previous series. You should point to it w/ lore links based on the Message-Id, like: https://lore.kernel.org/r/20240806135949.468636-1-tejasvipin76@xxxxxxxxx
