Messages in this thread | | | Date | Mon, 22 Nov 2004 11:49:33 -0700 | From | "Mark A. Greer" <> | Subject | Re: [PATCH][PPC32] Marvell host bridge support (mv64x60) |
| |
Christoph Hellwig wrote:
>Just looking through this and you should share some more code with mips, >e.g. the mavell register layout should move from asm-mips/marvell.h and >your file to linux/marvell.h or something, especially as another ppc >plattform (pegasosII) needs it aswell. >
Yes, and there is already include/linux/mv643xx.h that has a bunch of 64340 definitions which share offset with the 64360 (and some with the 64260). However, to be correct, there needs to be a lot of munging. mv643xx.h should be renamed to mv64xxx.h and all of the MV64340 macros need to be renamed as well. This ripples into the mips code which I didn't want to battle with right now. I would prefer that this patch go in and then submit a separate patch that fixes and combines the hdr files & macros they contain. Doing it that way, there is one patch for the ppc bridge support and one patch for the hdr file clean up. That's cleaner than mixing the two (IMHO). It is on the todo list.
Mark
- 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/
| |