On Fri, 21 Oct 2011 12:00:26 -0700, Jesse Barnes
wrote:
> Yeah works much better with this applied. Makes me want to change the
> PLL sharing code a bit though; this should be factored out into a
> separate function and I should probably just add a tiny PLL allocator
> for the pipes.
That woul
On Thu, 20 Oct 2011 23:18:06 -0700
Keith Packard wrote:
> On Wed, 19 Oct 2011 08:12:08 -0700, Jesse Barnes
> wrote:
>
> > /* PCH eDP needs FDI, but CPU eDP does not */
> > - if (!has_edp_encoder ||
> > intel_encoder_is_pch_edp(&has_edp_encoder->base)) {
> > + if (!intel_crtc->no_pll &
On Thu, 20 Oct 2011 23:18:06 -0700
Keith Packard wrote:
> On Wed, 19 Oct 2011 08:12:08 -0700, Jesse Barnes
> wrote:
>
> > /* PCH eDP needs FDI, but CPU eDP does not */
> > - if (!has_edp_encoder ||
> > intel_encoder_is_pch_edp(&has_edp_encoder->base)) {
> > + if (!intel_crtc->no_pll &
On Wed, 19 Oct 2011 08:12:08 -0700, Jesse Barnes
wrote:
> /* PCH eDP needs FDI, but CPU eDP does not */
> - if (!has_edp_encoder ||
> intel_encoder_is_pch_edp(&has_edp_encoder->base)) {
> + if (!intel_crtc->no_pll &&
> + (!has_edp_encoder ||
> + intel_encoder_is_p
Add two new fields to the intel_crtc struct for 3 pipe support: no_pll
and use_pll_a. The no_pll field is only set on the 3rd pipe to indicate
that it doesn't have a PLL of its own and so shouldn't try to write the
main PLL regs. The use_pll_a field controls which PLL pipe 3 will
share, A or B.
