Messages in this thread | | | Date | Tue, 5 Apr 2005 08:21:04 +0100 | From | Christoph Hellwig <> | Subject | Re: iomapping a big endian area |
| |
On Mon, Apr 04, 2005 at 04:17:45PM -0500, James Bottomley wrote: > OK, I sent the patch off to Andrew. To complete the original problem, > the attached is the patch that uses it in the parisc lasi driver > (although, actually, it sets up 53c700 to work everywhere including BE > on a LE system). > > I changed some of the flags around to reflect the fact that we now have > generic BE support in the driver (rather than the more limited > force_le_on_be flag).
What I really don't like is that you still need ifdefs and wrappers to support BE and LE wired chips in the same driver. Shouldn't you set the BE or LE flag at iomap() time and always use the same accessors?
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |