Hi,
On Mon, Apr 21, 2025 at 4:37 AM Zhengqiao Xia
<xiazhengq...@huaqin.corp-partner.google.com> wrote:
>
> CSW MNE007QS3-8 EDID:
> edid-decode (hex):
>
> 00 ff ff ff ff ff ff 00 0e 77 57 14 00 00 00 00
> 34 22 01 04 a5 1e 13 78 07 ee 95 a3 54 4c 99 26
> 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> 01 01 01 01 01 01 cd 7c 80 a0 70 b0 50 40 30 20
> 26 04 2e bc 10 00 00 1a cd 7c 80 a0 70 b0 50 45
> 30 20 26 04 2e bc 10 00 00 1a 00 00 00 fd 00 1e
> 78 9a 9a 20 01 0a 20 20 20 20 20 20 00 00 00 fc
> 00 4d 4e 45 30 30 37 51 53 33 2d 38 0a 20 01 3f
>
> 70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
> 80 3d 8a 54 cd a4 99 66 62 0f 02 45 54 7c 5d 7c
> 5d 00 43 12 78 2b 00 0c 27 00 1e 77 00 00 27 00
> 1e 3b 00 00 2e 00 06 00 43 7c 5d 7c 5d 81 00 20
> 74 1a 00 00 03 01 1e 78 00 00 5a ff 5a ff 78 00
> 00 00 00 8d 00 e3 05 04 00 e6 06 01 01 5a 5a ff
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 00 00 00 00 00 00 00 00 00 00 00 00 00 00 76 90
>
> Signed-off-by: Zhengqiao Xia <xiazhengq...@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 92844ab4cb9c..71cbee86fe81 100644
> --- a/drivers/gpu/drm/panel/panel-edp.c
> +++ b/drivers/gpu/drm/panel/panel-edp.c
> @@ -1770,6 +1770,13 @@ static const struct panel_delay delay_80_500_e50 = {
> .enable = 50,
> };
>
> +static const struct panel_delay delay_80_500_e80_p2e200 = {
> + .hpd_absent = 80,
> + .unprepare = 500,
> + .enable = 80,
> + .prepare_to_enable = 200,
> +};
> +
> static const struct panel_delay delay_100_500_e200 = {
> .hpd_absent = 100,
> .unprepare = 500,
> @@ -1982,6 +1989,7 @@ static const struct edp_panel_entry edp_panels[] = {
> EDP_PANEL_ENTRY('C', 'S', 'W', 0x1103, &delay_200_500_e80_d50,
> "MNB601LS1-3"),
> EDP_PANEL_ENTRY('C', 'S', 'W', 0x1104, &delay_200_500_e50,
> "MNB601LS1-4"),
> EDP_PANEL_ENTRY('C', 'S', 'W', 0x1448, &delay_200_500_e50,
> "MNE007QS3-7"),
> + EDP_PANEL_ENTRY('C', 'S', 'W', 0x1457, &delay_80_500_e80_p2e200,
> "MNE007QS3-8"),
I always get a little nervous when I see a low `hpd_absent` delay, but
this isn't the first one that's 80ms. As long as you're sure these
numbers are all correct then this looks fine to me.
Reviewed-by: Douglas Anderson <diand...@chromium.org>
