On 9 June 2011 16:44, Roland Vossen <rvossen@xxxxxxxxxxxx> wrote: > From: Arend van Spriel <arend@xxxxxxxxxxxx> > > The source file dma.c used a macro definition that indicated a big endian > platform. The linux tree has its own macro determined by architecture and/or > kernel configuration. This byteorder macro is __BIG_ENDIAN. This is now used > in dma.c. While this replacement is technically correct, the check itself is wrong and should be fixed instead. Currently it assumes that __mips__ means Broadcom MIPS; but if it isn't, then there's likely no SI_SDRAM_SWAPPED and any access there will in best case do nothing (but more like hang the system). Use either CONFIG_BCM47XX or CONFIG_BCM63XX to check for the BMIPS platforms this is probably intended for (I'm not 100% sure if BCM63XX has the swapped memory space - better check with your xDSL people ;-). Jonas _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
