2024-03-08 15:25:57

by Doug Anderson

[permalink] [raw]
Subject: Re: [V2] drm/panel-edp: Add BOE NT116WHM-N44 and CMN N116BCA-EA1

Hi,

On Thu, Mar 7, 2024 at 4:48 PM Xuxin Xiong
<[email protected]> wrote:
>
> Add support for the following 2 panels:
> 1. BOE NT116WHM-N44
> 2. CMN N116BCA-EA1
>
> Signed-off-by: Xuxin Xiong <[email protected]>
> Reviewed-by: Douglas Anderson <[email protected]>

It's fine this time, but please be careful in the future. I never
actually gave you my "Reviewed-by" tag for v1, I just said "The patch
looks OK" [1]. You should only add/carry someone's "Reviewed-by" tag
if they explicitly give that tag.

I'll also note that the subject of your patch starts with "[V2]". I'd
normally expect it to start with "[PATCH v2]". Maybe something you can
fix about your process for next time?

In any case, I've applied to drm-misc-next.

dcb6c8ee6acc drm/panel-edp: Add BOE NT116WHM-N44 and CMN N116BCA-EA1


[1] https://lore.kernel.org/r/CAD=FV=U8wdT_5k-yrLVpmh=q4k18LntqujK7Mw88TdweBXCPgg@mail.gmail.com