Hi, On Fri, Jul 24, 2026 at 4:55 AM Weiji Wang <[email protected]> wrote: > > Add timing for BOE NE140QDM-NX2. > > edid-decode (hex): > > 00 ff ff ff ff ff ff 00 09 e5 f2 0a 00 00 00 00 > 20 1f 01 04 a5 1e 13 78 03 ee 96 a3 54 4c 99 26 > 0f 4e 51 00 00 00 01 01 01 01 01 01 01 01 01 01 > 01 01 01 01 01 01 60 d2 00 a0 a0 40 32 60 30 20 > 35 00 2e bd 10 00 00 18 c8 9d 00 a0 a0 40 32 60 > 30 20 35 00 2e bd 10 00 00 18 00 00 00 fd 00 30 > 78 c6 c6 36 01 0a 20 20 20 20 20 20 00 00 00 fe > 00 4e 45 31 34 30 51 44 4d 2d 4e 58 32 20 01 f8 > > 70 13 79 00 00 03 01 14 30 69 00 05 ff 09 9f 00 > 2f 00 1f 00 3f 06 31 00 02 00 04 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 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 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 00 00 00 00 00 00 00 00 00 4d 90 > > Signed-off-by: Weiji Wang <[email protected]> > --- > drivers/gpu/drm/panel/panel-edp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/panel/panel-edp.c > b/drivers/gpu/drm/panel/panel-edp.c > index df7286679446..9e0981ecbc1c 100644 > --- a/drivers/gpu/drm/panel/panel-edp.c > +++ b/drivers/gpu/drm/panel/panel-edp.c > @@ -2031,6 +2031,7 @@ static const struct edp_panel_entry edp_panels[] = { > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0ae8, &delay_200_500_e50_p2e80, > "NV140WUM-N41"), > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0b09, &delay_200_500_e50_po2e200, > "NV140FHM-NZ"), > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0b1e, &delay_200_500_e80, > "NE140QDM-N6A"), > + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0af2, &delay_200_500_e80, > "NE140QDM-NX2"), > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0b34, &delay_200_500_e80_d50, > "NV122WUM-N41"),
This is not sorted properly. 0x0af2 does not belong between 0x0b1e and 0x0b34. ...I could fix that when applying, but I also have a question: do you have access to the datasheet of this panel, or are you just guessing the right timing by looking at the nearby panels? I assume you have this panel yourself for testing and confirm it seems to work OK. It's fine if you don't have the datasheet and are making guesses, since acquiring the datasheet is really hard for end-users. ...but in that case, I like to at least document that fact in the commit message so if someone later comes along and proposes different timings (and they have the datasheet) then we can figure out how to proceed. :-) -Doug
