On Wed, Feb 20, 2019 at 2:33 AM Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: > > On Tue, Feb 19, 2019 at 07:44:56AM -0800, Vasily Khoruzhick wrote: > > On Tue, Feb 19, 2019 at 12:56 AM Maxime Ripard > > Maxime, thanks for your patch but it doesn't work for me. Pinebook > > needs 1% tolerance. Having it as a module parameter means that no > > distro will be able to boot on Pinebook out of the box. > > I don't really know what to tell you, the VESA spec defines everywhere > that tolerance, and if we're not able to provide that, then we're not > compliant and I don't want us to not be compliant just because one > panel needed to be a bit more flexible, and especially since what > could work on one panel might fail on another one. > > If you want alternate solutions, then please answer to: > http://lists.infradead.org/pipermail/linux-arm-kernel/2019-February/630441.html > > or provide the EDID blob. Uh, I spoke to early, my bad. I made a mistake when I calculated rate deviation for 768p last time - I took 73MHz as dotclock, but actual requested rate 72.3 MHz and it gets 72.296296 MHz, so it's within 0.5% tolerance. 1080p pinebook needs 138.5 MHz and gets 138.461538 MHz - also within 0.5%. However your patch has an issue, I'll respond to your previous email. > Maxime > > -- > Maxime Ripard, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com