xf86-video-intel: src/i810_reg.h src/i830_driver.c

[Jesse Barnes]

On Thursday, May 29, 2008 9:23 am Alan Hourihane wrote:
> > You're right, we should be doing a bandwidth calculation based on the
> > current mode for each plane.  But I still don't have access to the
> > spreadsheet that documents how the FIFO should be split, so until then
> > we'll have to hope that stealing all of display plane C's FIFO entries is
> > enough. :)  And at least now we'll know when underruns happen, since I
> > added some debug code.
>
> The way I understand the split to be configured is as so....
>
> e.g. FIFO RAM entries = 96, so...
>
> HDISPLAY for PIPEA = 1280
> HDISPLAY for PIPEB = 1024
>
> Therefore the split is 1280*(96/(1280+1024)) and 1024*(96/(1280+1024))
> giving a value of approx 53 and 43, so DSPARB would be set as....
>
> OUTREG(DSPARB, (95 << DSPARB_CSTART_SHIFT) |
> 		  (52 << DSPARB_BSTART_SHIFT));
>
> Things would need adjusting for FIFO RAM which is 128 or 256 wide,
> obviously.

According to the chipset guys it's a little more complicated than that, though 
what you suggest would probably be a much better default behavior than what 
we have now.  I'm still talking with them, but if I don't get something 
useful back from them in the next few days, I'll go ahead and hack together 
something like you suggest.

Thanks,
Jesse