Re: [Intel-gfx] [PATCH 1/7] drm/i915/cnl: Remove spurious central_freq.

2017-11-14 Thread Manasi Navare
Verified this from the Bspec that central frequency should be left at default at 8400MHz which is value 3 in cfgcr1.central_freq. Looks good to me. Reviewed-by: Manasi Navare On Tue, Nov 14, 2017 at 11:47:53AM -0800, Rodrigo Vivi wrote: > "Display software must leave this field at the default va

[Intel-gfx] [PATCH 1/7] drm/i915/cnl: Remove spurious central_freq.

2017-11-14 Thread Rodrigo Vivi
"Display software must leave this field at the default value. It no longer needs to be configured as part of PLL programming." We respect this already and we are setting up the default one line below: "DPLL_CFGCR1_CENTRAL_FREQ". Also we don't touch anywhere else this central_freq for cnl. So let'