[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4434574.6rqfZtZq1D@diego>
Date: Fri, 29 Mar 2019 20:17:57 +0100
From: Heiko Stübner <heiko@...ech.de>
To: Douglas Anderson <dianders@...omium.org>
Cc: Thierry Reding <thierry.reding@...il.com>,
Sean Paul <seanpaul@...omium.org>,
linux-rockchip@...ts.infradead.org,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
dri-devel@...ts.freedesktop.org,
Boris Brezillon <boris.brezillon@...labora.com>,
Ezequiel Garcia <ezequiel@...labora.com>,
Enric Balletbò <enric.balletbo@...labora.com>,
Rob Herring <robh+dt@...nel.org>, mka@...omium.org,
David Airlie <airlied@...ux.ie>, linux-kernel@...r.kernel.org,
Daniel Vetter <daniel@...ll.ch>
Subject: Re: [PATCH v4 4/7] drm/panel: simple: Use display_timing for Innolux n116bge
Am Donnerstag, 28. März 2019, 18:17:07 CET schrieb Douglas Anderson:
> Convert the Innolux n116bge from using a fixed mode to specifying a
> display timing with min/typ/max values.
>
> Note that the n116bge's datasheet doesn't fit too well into DRM's way
> of specifying things. Specifically the panel's datasheet just
> specifies the vertical blanking period and horizontal blanking period
> and doesn't break things out. For now we'll leave everything as a
> fixed value but just allow adjusting the pixel clock. I've added a
> comment on what the datasheet claims so someone could later expand
> things to fit their needs if they wanted to test other blanking
> periods.
>
> The goal here is to be able to specify the panel timings in the device
> tree for several rk3288 Chromebooks (like rk3288-veryon-jerry). These
> Chromebooks have all been running in the downstream kernel with the
> standard porches and sync lengths but just with a slightly slower
> pixel clock because the 76.42 MHz clock is not achievable from the
> fixed PLL that was available. These Chromebooks only achieve a
> refresh rate of ~58 Hz. While it's probable that we could adjust the
> timings to achieve 60 Hz it's probably wisest to match what's been
> running on these devices all these years.
>
> I'll note that though the upstream kernel has always tried to achieve
> 76.42 MHz, it has actually been running at 74.25 MHz also since the
> video processor is parented off the same fixed PLL.
>
> Changes in v4:
> - display_timing for Innolux n116bge new for v4.
>
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
on a rk3288-jerry and rk3288-pinky
Tested-by: Heiko Stuebner <heiko@...ech.de>
Powered by blists - more mailing lists