[PATCH] drm/panel-edp: Add STA 116QHD024002

[Doug Anderson]

Hi,

On Wed, Jan 22, 2025 at 12:23 AM Langyan Ye
<yelangyan at huaqin.corp-partner.google.com> wrote:
>
> Add support for the STA 116QHD024002, pleace the EDID here for
> subsequent reference.
>
> 00 ff ff ff ff ff ff 00 4e 81 09 00 00 00 00 00
> 26 21 01 04 a5 1a 0e 78 02 1e b5 9a 5f 57 94 26
> 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> 01 01 01 01 01 01 8e 1c 56 a0 50 00 1e 30 28 20
> 55 00 00 90 10 00 00 18 00 00 00 00 00 00 00 00
> 00 00 00 00 00 00 00 00 00 00 00 00 00 fe 00 20
> 20 20 20 20 20 0a 20 20 20 20 20 20 00 00 00 fe
> 00 31 31 36 51 48 44 30 32 34 30 30 32 0a 00 3b
>
> Signed-off-by: Langyan Ye <yelangyan at huaqin.corp-partner.google.com>
> ---
>  drivers/gpu/drm/panel/panel-edp.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
> index 1721a3397dbf..73b8938f175b 100644
> --- a/drivers/gpu/drm/panel/panel-edp.c
> +++ b/drivers/gpu/drm/panel/panel-edp.c
> @@ -1808,6 +1808,13 @@ static const struct panel_delay delay_200_150_e50 = {
>         .enable = 50,
>  };
>
> +static const struct panel_delay delay_200_500_e250_po2e200 = {
> +       .hpd_absent = 200,
> +       .unprepare = 500,
> +       .enable = 250,
> +       .powered_on_to_enable = 200,
> +};

I don't believe you need "powered_on_to_enable" here. The docs for
"powered_on_to_enable" say:

* This doesn't normally need to be set if timings are already met by
* prepare_to_enable or enable.

In this case "enable" is 250 ms, which subsumes the 200 ms
"powered_on_to_enable".

Can you confirm and then re-post without the "powered_on_to_enable" setting?

Thanks!

-Doug