Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-17 Thread Benjamin Herrenschmidt
For example, there are two copies of register aperture one with low-endian translation and one with big-endian. That isn't necessary. I don't think there is overhead using the endian-swapping versions of load store instructions, and you need specific macros anyways since you absolutely need

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-17 Thread Benjamin Herrenschmidt
On Fri, 2005-01-14 at 12:23 -0500, Michel Dänzer wrote: On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in server/radeon_macros.h have to do endian swapping. Moreover the swapping

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-17 Thread Michel Dänzer
On Tue, 2005-01-18 at 14:22 +1100, Benjamin Herrenschmidt wrote: On Fri, 2005-01-14 at 12:23 -0500, Michel Dnzer wrote: On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-15 Thread Jerome Glisse
On Fri, 14 Jan 2005 20:04:48 -0500 (EST), Vladimir Dergachev [EMAIL PROTECTED] wrote: On Fri, 14 Jan 2005, Jerome Glisse wrote: On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-15 Thread Jerome Glisse
On Sat, 15 Jan 2005 12:22:24 -0500 (EST), Vladimir Dergachev [EMAIL PROTECTED] wrote: From radeon_reg.h: #define RADEON_HOST_PATH_CNTL 0x0130 # define RADEON_HDP_SOFT_RESET(1 26) You also need to add the following to the header: /*set this to map second

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-15 Thread Michel Dänzer
On Sat, 2005-01-15 at 18:55 +0100, Jerome Glisse wrote: On Sat, 15 Jan 2005 12:22:24 -0500 (EST), Vladimir Dergachev [EMAIL PROTECTED] wrote: #define R300_SURFACE_CNTL 0xb00 # define R300_SURFACE_TRANSLATION_DISABLE (18) /* this is default */ # define

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-14 Thread Jerome Glisse
On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in server/radeon_macros.h have to do endian swapping. Moreover the swapping is only needed for r300, isn't it ? So do

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-14 Thread Michel Dänzer
On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in server/radeon_macros.h have to do endian swapping. Moreover the swapping is only needed for r300, isn't it ? So do we need to have our

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-14 Thread Michel Dänzer
On Fri, 2005-01-14 at 18:48 +0100, Jerome Glisse wrote: On Fri, 14 Jan 2005 12:23:50 -0500, Michel Dnzer [EMAIL PROTECTED] wrote: On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-14 Thread Michel Dänzer
On Fri, 2005-01-14 at 19:51 +0100, Jerome Glisse wrote: I will try your patch but i bet it works. I am agree too that this should not be in driver specifique but how to handle the r300 case properly on ppc ? There's no 'r300 case'. This will be the same for all Radeons and

Re: [Mesa3d-dev] Fixing r300 on PPC / How to conditionnaly have endian swap macro

2005-01-14 Thread Vladimir Dergachev
On Fri, 14 Jan 2005, Jerome Glisse wrote: On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: Anyway i wanted to ask mesa folks how to make a real proper patch. The fact is that INREG OUTREG in server/radeon_macros.h have to do endian swapping. Moreover the swapping is only needed for r300,