Hi,
2018-02-08 18:48 GMT+01:00 Sean Paul <seanpaul@xxxxxxxxxxxx>:
> Convert the sharp lq123p1jx31 from using a fixed mode to specifying a
> display timing with min/typ/max values. This allows us to capture the
> timings set forth in the datasheet as well as the additional values that
> we've cleared with the display vendor to avoid interference with the
> digitizer on the Samsung Chromebook Plus (kevin).
>
> A follow-on patch will specify the override mode for kevin devices.
>
> Changes in v2:
> - None
> Changes in v3:
> - None
>
> Cc: Doug Anderson <dianders@xxxxxxxxxxxx>
> Cc: Eric Anholt <eric@xxxxxxxxxx>
> Cc: Heiko Stuebner <heiko@xxxxxxxxx>
> Cc: Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> Cc: Rob Herring <robh+dt@xxxxxxxxxx>
> Cc: Stéphane Marchesin <marcheu@xxxxxxxxxxxx>
> Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: linux-rockchip@xxxxxxxxxxxxxxxxxxx
> Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx>
> ---
> drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++--------------
> 1 file changed, 13 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 87488392bca1..6de9c39bc182 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -1806,23 +1806,22 @@ static const struct panel_desc sharp_lq101k1ly04 = {
> .bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA,
> };
>
> -static const struct drm_display_mode sharp_lq123p1jx31_mode = {
> - .clock = 252750,
> - .hdisplay = 2400,
> - .hsync_start = 2400 + 48,
> - .hsync_end = 2400 + 48 + 32,
> - .htotal = 2400 + 48 + 32 + 80,
> - .vdisplay = 1600,
> - .vsync_start = 1600 + 3,
> - .vsync_end = 1600 + 3 + 10,
> - .vtotal = 1600 + 3 + 10 + 33,
> - .vrefresh = 60,
> - .flags = DRM_MODE_FLAG_NVSYNC | DRM_MODE_FLAG_NHSYNC,
> +static const struct display_timing sharp_lq123p1jx31_timing = {
> + .pixelclock = { 252750000, 252750000, 266604720 },
> + .hactive = { 2400, 2400, 2400 },
> + .hfront_porch = { 48, 48, 48 },
> + .hback_porch = { 80, 80, 84 },
> + .hsync_len = { 32, 32, 32 },
> + .vactive = { 1600, 1600, 1600 },
> + .vfront_porch = { 3, 3, 3 },
> + .vback_porch = { 33, 33, 120 },
> + .vsync_len = { 10, 10, 10 },
> + .flags = DISPLAY_FLAGS_VSYNC_LOW | DISPLAY_FLAGS_HSYNC_LOW,
> };
>
> static const struct panel_desc sharp_lq123p1jx31 = {
> - .modes = &sharp_lq123p1jx31_mode,
> - .num_modes = 1,
> + .timings = &sharp_lq123p1jx31_timing,
> + .num_timings = 1,
> .bpc = 8,
> .size = {
> .width = 259,
> --
Tested on top of linux-next on a Samsung Chromebook Plus.
Tested-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> 2.16.0.rc1.238.g530d649a79-goog
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
