[Mesa-dev] [PATCH] radeonsi: Don't write to the PA_SC_RASTER_CONFIG register.

[Michel Dänzer]

On Mon, 2012-08-27 at 14:54 +0200, Christian König wrote: 
> On 27.08.2012 14:46, Michel Dänzer wrote:
> > On Fre, 2012-08-24 at 12:18 -0400, Alex Deucher wrote:
> >> On Fri, Aug 24, 2012 at 11:37 AM, Michel Dänzer <michel at daenzer.net> wrote:
> >>> From: Michel Dänzer <michel.daenzer at amd.com>
> >>>
> >>> It should be initialized by the kernel as necessary.
> >> Does this patch work ok after powering off the system to make sure
> >> there is no state in the register from a previous run?
> > Yes. AFAICT the VBIOS initializes the register to the 'golden' value,
> > then the kernel to something else. We should probably fix the latter one
> > way or another, but meanwhile I really don't think it makes sense for
> > the radeonsi driver to write 0 to it. :)
> 
> I also played around with that a bit. One interesting thing we should 
> keep in mind is that writing 0 actually does seem to work.

Yeah, AFAICT this register can only affect performance, not correctness.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer