Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> writes: > On Fri, Jan 27, 2017 at 03:44:27PM +0100, Arthur Heymans wrote: > > That indeed is what the docs say. > > The code is tantalizingly close to the 915gm code now, so maybe > we could share it with a simple > > if (IS_915GM(dev_priv)) > return 320000; > else > return 333333; > Agreed but it's the other way around ;) > Now if someone could figure out where to dig up the DDR and FSB clocks > we could also fix up the 190 vs. 200 MHz case... > >> + default: >> + case GC_DISPLAY_CLOCK_190_200_MHZ: >> + return 200000; >> + } >> + } >> +} >> + Hmm that seems to be 915gm specific (always 200 on 945gm). According to "Mobile Intel® 915/910 Express Chipset: Datasheet", the only fsb/dram combo that has 190MHz is FSB: 533MHz, DDR333. All the other supported combos have 200Mhz set by that configuration. -- Arthur Heymans _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
