lkml.org 
[lkml]   [2004]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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/

\
 
 \ /
  Last update: 2005-03-22 14:08    [W:0.063 / U:0.972 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site