[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <7960623.ZFqGeXkhir@diego>
Date: Fri, 29 Mar 2019 20:20:00 +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,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Mark Rutland <mark.rutland@....com>,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v4 6/7] ARM: dts: rockchip: Specify rk3288-veyron-jerry's display timings
Am Donnerstag, 28. März 2019, 18:17:09 CET schrieb Douglas Anderson:
> Let's document the display timings that jerry has been using out in
> the field. This uses the standard blankings but a slightly slower
> clock rate, thus getting a refresh rate 58.3 Hz.
>
> NOTE: this won't really do anything except cause DRM to properly
> report the refresh rate since vop_crtc_mode_fixup() was rounding the
> pixel clock to 74.25 MHz anyway. Apparently the adjusted rate isn't
> exposed to userspace so it's important that the rate we're trying to
> achieve is mostly right.
>
> For the downstream kernel change related to this see See
> https://crrev.com/c/324558.
>
> NOTE: minnie will be fixed up in a future patch, so for now we'll just
> delete the panel timings there.
>
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
> ---
>
> Changes in v4:
> - rk3288-veyron-jerry patch new for v4.
>
> Changes in v3: None
> Changes in v2: None
>
> arch/arm/boot/dts/rk3288-veyron-chromebook.dtsi | 14 ++++++++++++++
hmm, the commit message explicitly mentions jerry, but this is the
general panel definition for most veyron-chromebooks (jerry, pinky, jaq,...)?
It does work on both pinky and jerry for me, so I guess just the commit
message needs a bit adapting?
Heiko
Powered by blists - more mailing lists