panel: simple: Fix size and bpp of BOE NV133FHM-N61
authorDouglas Anderson <dianders@chromium.org>
Fri, 8 May 2020 22:59:00 +0000 (15:59 -0700)
committerSam Ravnborg <sam@ravnborg.org>
Sat, 9 May 2020 19:57:02 +0000 (21:57 +0200)
The BOE NV133FHM-N61 is documented in the original commit to be a
13.3" panel, but the size listed in our struct doesn't match.
Specifically:

  math.sqrt(30.0 * 30.0 + 18.7 * 18.7) / 2.54 ==> 13.92

Searching around on the Internet shows that the size that was in the
structure was the "Outline Size", not the "Display Area".  Let's fix
it.

Also the Internet says that this panel supports 262K colors.  That's
6bpp, not 8bpp.

Fixes: b0c664cc80e8 ("panel: simple: Add BOE NV133FHM-N61")
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20200508155859.1.I4d29651c0837b4095fb4951253f44036a371732f@changeid
drivers/gpu/drm/panel/panel-simple.c

index d6c2954..7219436 100644 (file)
@@ -1189,10 +1189,10 @@ static const struct drm_display_mode boe_nv133fhm_n61_modes = {
 static const struct panel_desc boe_nv133fhm_n61 = {
        .modes = &boe_nv133fhm_n61_modes,
        .num_modes = 1,
-       .bpc = 8,
+       .bpc = 6,
        .size = {
-               .width = 300,
-               .height = 187,
+               .width = 294,
+               .height = 165,
        },
        .delay = {
                .hpd_absent_delay = 200,