Hi, On Mon, Mar 20, 2017 at 1:01 PM, St?phane Marchesin <marcheu at chromium.org> wrote: >> 1. We have validated with the panel manufacturer that 266.667 MHz is >> valid and within spec. The panel's datasheet itself says something >> like "if you want to try other values you can, but they might not >> work", so technically the only values "known" to work are those that >> were in the original patch and the values here in Sean's patch. > > > So why don't you add 2 modes, and let the drivers pick the clock they > prefer? Yeah, that's what I did originally in my RFC patch at <http://crosreview.com/354165> but I personally didn't have a good way for the driver to figure out how to pick the right clock. When the patch actually landed I guess nobody else had a good way either and thus only one clock mode was there. -Doug
