[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAD=FV=U1=5MLUKOok2mTkEeTkT-r504+f9UJuvErLzVbFq_4Hg@mail.gmail.com>
Date: Mon, 22 Jul 2024 10:24:40 -0700
From: Doug Anderson <dianders@...omium.org>
To: Terry Hsiao <terry_hsiao@...pal.corp-partner.google.com>
Cc: linux-kernel@...r.kernel.org, Neil Armstrong <neil.armstrong@...aro.org>,
Jessica Zhang <quic_jesszhan@...cinc.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>,
dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v1] drm/panel-edp: Add panels with conservative timings
Hi,
On Sun, Jul 21, 2024 at 4:00 AM Terry Hsiao
<terry_hsiao@...pal.corp-partner.google.com> wrote:
>
> Hi Doug,
>
> Thank you for your reply.
> The patch has been modified and will be sent to you shortly.
For future reference, the Linux community frowns upon "top posting".
Search for "top-posting" on [1]
[1] https://www.arm.linux.org.uk/mailinglists/etiquette.php
> The timings are set based on the panel datasheets in IssueTracker
> (https://partnerissuetracker.corp.google.com/issues/348109270)
FWIW, if you want to privately provide links to datasheets to me to
double-check your work then that's fine, but the above link is useless
to others on the Linux kernel mailing list and people usually don't
appreciate such links. In this case you could have replied publicly
and told others that you'd gotten your work double-checked and that
would have been sufficient for the public lists.
> B116XTN02.3: B116XTN02.3(HW 9A)_HP_ Functional Spec_0617Y24.pdf
> B116XAN06.1: B116XAN06.1_7A_HP_ Final Functional Spec 0617Y24.pdf
> B116XAT04.1: B116XAT04.1 HW 0 A(HH)_ Pre Functional Spec_HP_ 0425Y24.pdf
> NV116WHM-A4D: NV116WHM-A4D V8.0 Teacake Product Specification-20240416.pdf
> N116BCA-EA2: Approval Specification N116BCA-EA2_C3_20231212.pdf
> N116BCP-EA2: TFT-LCD Tentative N116BCP-EA2 C2 for HP Ver 0.2-240502.pdf
>
> On page 24 of the N116BCP-EA2
> datasheet(https://partnerissuetracker.corp.google.com/action/issues/348109270/attachments/57530666?download=false),
> the value for t9 as disable is "null".
>
> If I have misunderstood what you mean, please correct me.
I've double-checked and this looks fine to me.
-Doug
Powered by blists - more mailing lists