On Wed, 2007-10-03 at 14:08 -0700, Ian Romanick wrote: > > diff --git a/linux-core/xgi_cmdlist.c b/linux-core/xgi_cmdlist.c > index 261f4e1..35f7e1b 100644 > --- a/linux-core/xgi_cmdlist.c > +++ b/linux-core/xgi_cmdlist.c > @@ -45,7 +45,7 @@ static inline void dwWriteReg(struct drm > DRM_INFO("mmio_map->handle = 0x%p, addr = 0x%x, data = 0x%x\n", > map->handle, addr, data); > #endif > - DRM_WRITE32(map, addr, data); > + DRM_WRITE32(map, addr, cpu_to_le32(data));
These look odd - DRM_READ/WRITE* already swap bytes. Does the device's MMIO aperture swap bytes on its own? If so, and you can't or don't want to disable that, maybe we need new macros using __raw_read/write* on Linux. -- Earthling Michel Dänzer | http://tungstengraphics.com Libre software enthusiast | Debian, X and DRI developer ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel