On Fri, 3 May 2013 15:34:41 -0700 Jesse Barnes <jbarnes at virtuousgeek.org> wrote: > We need this for comparing modes between configuration changes. > > v2: try harder to calulate non-simple pixel clocks (Daniel) > call get_clock after getting the encoder config, needed for pixel multiply > (Jesse) > In reply to danvet's questions: - I can move the pixel multiplier into get_config; I thought some of our encoder regs had a bit that could be used - if I do that, I don't need to call get_clock separately; it needs the pixel multiplier to calculate m/n correctly, so that's why I had to put it after the encoder get_config I added pipe config checks for the new mode flags, and I get warnings now, but I don't want to add one for the clock just yet. Maybe after everything is in and working we can add one... -- Jesse Barnes, Intel Open Source Technology Center