[PATCH v1] drm/panel-edp: Add panels with conservative timings

[Doug Anderson]

Hi,

On Sun, Jul 21, 2024 at 4:00 AM Terry Hsiao
<terry_hsiao at compal.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