On Sat, May 14, 2011 at 12:16:31PM +0200, Jonas Gorski wrote: > On 13 May 2011 11:59, Arend van Spriel <arend@xxxxxxxxxxxx> wrote: > > From: Roland Vossen <rvossen@xxxxxxxxxxxx> > > > > Code cleanup. Broadcom MIPS chips in big endian mode have a memory window > > in which sdram contents is byte swapped. Code in hnddmac.c makes use of > > this window. Reworked code so the #ifdef __mips__ is moved from .c to > > .h file. > > Shouldn't this be guarded with e.g. CONFIG_BCM63XX instead of > __mips__? Not every big endian mips is a broadcom ;-). I don't know if > there are further Broadcom ones, the only other I know is bcm47xx and > that one is little endian. Good point, now dropped. greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel