[PATCH 0/2] add more multi functions for streamlined error handling
Doug Anderson
dianders at chromium.org
Mon Aug 12 22:58:28 UTC 2024
Hi,
On Fri, Aug 9, 2024 at 9:55 PM Tejas Vipin <tejasvipin76 at gmail.com> 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@gmail.com
More information about the dri-devel
mailing list