On Mon, Mar 07, 2011 at 06:15:46PM +0000, Maarten Maathuis wrote: > On Fri, Mar 4, 2011 at 4:50 PM, Marcin Slusarz <[email protected]> > wrote: > > On Tue, Feb 22, 2011 at 06:32:40PM +0100, Marcin Slusarz wrote: > >> Addresses: > >> https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-nouveau/+bug/723012 > >> > >> Reported-and-tested-by: Alan Pope > >> --- > >> src/gallium/drivers/nv50/nv50_reg.h | 4 ++++ > >> src/gallium/drivers/nv50/nv50_screen.c | 3 +++ > >> 2 files changed, 7 insertions(+), 0 deletions(-) > >> > >> diff --git a/src/gallium/drivers/nv50/nv50_reg.h > >> b/src/gallium/drivers/nv50/nv50_reg.h > >> index 949838b..90d77e5 100644 > >> --- a/src/gallium/drivers/nv50/nv50_reg.h > >> +++ b/src/gallium/drivers/nv50/nv50_reg.h > >> @@ -1685,6 +1685,10 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN > >> THE SOFTWARE. > >> > >> > >> > >> +#define NVAFTCL > >> 0x00008697 > >> + > >> + > >> + > >> #define NV50_COMPUTE > >> 0x000050c0 > >> > >> #define NV50_COMPUTE_NOP > >> 0x00000100 > >> diff --git a/src/gallium/drivers/nv50/nv50_screen.c > >> b/src/gallium/drivers/nv50/nv50_screen.c > >> index edc3d54..8069509 100644 > >> --- a/src/gallium/drivers/nv50/nv50_screen.c > >> +++ b/src/gallium/drivers/nv50/nv50_screen.c > >> @@ -389,6 +389,9 @@ nv50_screen_create(struct pipe_winsys *ws, struct > >> nouveau_device *dev) > >> case 0xac: > >> tesla_class = NVA0TCL; > >> break; > >> + case 0xaf: > >> + tesla_class = NVAFTCL; > >> + break; > >> default: > >> tesla_class = NVA8TCL; > >> break; > >> -- > >> 1.7.4.rc3 > >> > > > > ping > > > > if it's wrong for some reason, just say so... > > > > The patch seems ok to me, the nitpicker in me would tell you to > separate the reported by and tested by tags and include Alan Pope's > email address :) >
Christoph already fixed it in his big "nv50: replace most of it with nvc0 driver ported to nv50" commit. Thanks for a review anyway :), Marcin _______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
