[lkml]   [2002]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH]: 2.5.38uc1 (MMU-less support)
    Hi Matthew,

    Matthew Wilcox wrote:
    > Thanks for splitting the patch up, makes it easier to see what's going on.
    > Let's have another go at making this better...


    > Motorola 5272 ethernet driver:
    > * In, let's conditionalise it on CONFIG_PPC or something

    The FEC ethernet core has been used by Motorola in both PowerPC parts
    (the MPC860) and the ColdFire (5272). But it could be conditional
    on either of these.

    > * Can you use module_init() so it doesn't need an entry in Space.c?

    Easy done :-)

    > * You're defining CONFIG_* variables in the .c file. I don't know whether
    > this is something we're still trying to avoid doing ... Greg, you seem
    > to be CodingStyle enforcer, what's the word?

    To be honest I don't know why all of these are not always included.
    The extra code size is not that significant (tehse rae just PHY

    Some history:
    I didn't write this driver. It is Dan Malek's driver for the ethernet
    core in the PowerPC 860. I took that code and generalizd it for use
    on the ColdFire 5272 which also uses the same ethernet core design.

    By and large I have tried to leave the original drivers functionalty
    "as it was", conditionalizing the 5272 support as required (there are
    a few key differences to be dealt with).

    I approached Dan in the past about merging the code, and he was not

    > * Why do you need to EXPORT_SYMBOL fec_register_ph and fec_unregister_ph?

    I can't see any use for this...

    > * There's an awful lot of stuff conditionalised on CONFIG_M5272. In general,
    > having #ifdefs within functions is frowned upon.

    Yeah, there is a bit of that. All due to trying to handle the
    silicon implementation differences on the 860 and 5272.
    I'll see if I can clean it up some.

    > Motorola 68328 and ColdFire serial drivers:
    > * Move to drivers/serial
    > * Lose this change from the Makefile:
    > - selection.o sonypi.o sysrq.o tty_io.o tty_ioctl.o
    > + selection.o sonypi.o sysrq.o tty_io.o tty_ioctl.o \
    > * Drop the custom MIN() definition.
    > * Port to new serial driver framework.

    This is on my todo list :-)

    > MTD driver patches for uClinux supported platforms:
    > I don't see any problems. Submit to Linus via Dave Woodhouse, I guess.

    Already done. They have just not hit the mailline tree yet.
    But David has it in his CVS now.

    > Motorola 68328 framebuffer:
    > Don't see any problems here either.
    > uClinux FLAT file format exe loader:
    > * Drop the MAX() macro.
    > * +#include "../lib/inflate2.c". Er. You seem to have missed inflate2.c
    > from your patch, and this really isn't the right way to do it anyway.
    > Can't you share inflate.c these days?
    > * I'm also a little unsure about your per-arch #defines. Could you put
    > comments by each saying why they're necessary?

    I'll look at cleaning this up too. This code has been hacked
    on by so many people, I am still cleaning the chewy bits of it :-)

    > I haven't reveiwed the other two patches.

    mmnommu is probably the most contenious, and effects the most
    files. I am still looking at merging the separated mmnommu
    directory at the top level back into the mm directory. Just
    thinking about the cleanest way to do that.

    The m68knommu is purely architecture support, doesn't mess
    with anything else in the kernel.

    Thanks for the feedback, much appreciated!


    Greg Ungerer -- Chief Software Wizard EMAIL:
    Snapgear Pty Ltd PHONE: +61 7 3279 1822
    825 Stanley St, FAX: +61 7 3279 1820
    Woolloongabba, QLD, 4102, Australia WEB:

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:29    [W:0.024 / U:36.680 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site