Re: [PATCH] Several patches agains linuxppc_devel-2.4.19-pre8

[Tom Rini <trini@kernel.crashing.org>]

On Thu, May 16, 2002 at 11:26:33AM +0200, "David M?ller (ELSOFT AG)" wrote:
> Tom Rini schrieb:
> >On Wed, May 15, 2002 at 01:35:55PM +0200, "David M?ller (ELSOFT AG)" wrote:
> >
> >
> >>The attached patch fixes the following issues:
> >
> >
> >Looks good, except:
> >Don't #ifdef externs, if they aren't used it's OK.
> >Since this code only has to work in recent'ish kernels (in reality only
> >'current' kernels) don't bother with the
> >#ifdef MODULE_LICENSE
> >...
> >#endif
> >
> >Just use it.
> >
>
> I have copied the MODULE_LICENSE stuff from another driver.
> Does anybody knows if this feature is tagged to be a remove candidate in
> future kernels and/or in the modutils?

Unfortunately, no.  I think other drivers do it for backwards
compatibility, which we don't have to worry about.

> Another question:
>
> I've noticed that (at least) the following files have their executable
> bit set, although they are simple C source files.
> ./include/asm-ppc/ppc405_dma.h
> ./arch/ppc/kernel/ppc405_dma.c
> ./arch/ppc/kernel/ppc405_pci.c
> ./arch/ppc/kernel/ppc4xx_setup.c
> ./arch/ppc/platforms/walnut.c
> ./arch/ppc/platforms/cpc700_pic.c
>
> On the other hand, the PPCBoot kernel packer script in
> ./arch/ppc/boot/utils/mkimage.wrapper have the exec bit cleared.
>
> Is it a bit task to fix this?

I assume you ment big.  And no, I've just fixed these.  Thanks.

--
Tom Rini (TR1265)
http://gate.crashing.org/~trini/

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/