| To: | Ralf Baechle <ralf@linux-mips.org> |
|---|---|
| Subject: | Re: [2.6 patch] malta_mtd.c: add MODULE_LICENSE |
| From: | Dmitri Vorobiev <dmitri.vorobiev@movial.fi> |
| Date: | Tue, 17 Jun 2008 11:52:38 +0300 |
| Cc: | Adrian Bunk <bunk@kernel.org>, linux-mips@linux-mips.org |
| In-reply-to: | <20080616143700.GA28471@linux-mips.org> |
| Organization: | Movial Creative Technologies |
| Original-recipient: | rfc822;linux-mips@linux-mips.org |
| References: | <20080615161321.GB7865@cs181133002.pp.htv.fi> <20080616133049.GA24056@linux-mips.org> <48567A12.3050206@movial.fi> <20080616143700.GA28471@linux-mips.org> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Icedove 1.5.0.14eol (X11/20080509) |
This is a multi-part message in MIME format. Ralf Baechle wrote: > On Mon, Jun 16, 2008 at 05:34:58PM +0300, Dmitri Vorobiev wrote: > >> Every time I come across this file I wonder whether this needs to be >> modularized at all. Isn't you chainsaw itching for this as well? :) > > The reason why it was implemented as a module was that due to a MTD > "feature" the code wouldn't compile in 2.6.20 and earlier. So yes, I > chainsawed it into the right shape, patch is on the way to Linus. Looks like Linus has applied the patch. I think that module_init() needs to be replaced by device_initcall() in malta_mtd.c. Not that it would make any difference at all, but just some nitpicking - I simply get a bit nervous and irritated when looking at the `module' word in the core kernel code. :) Patch attached. Dmitri > > Ralf > |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 1/1] [MIPS] Get rid of module_init() in core kernel code, Dmitri Vorobiev |
|---|---|
| Next by Date: | Linux Boot RAM Determination, Sean Parker |
| Previous by Thread: | Re: [2.6 patch] malta_mtd.c: add MODULE_LICENSE, Dmitri Vorobiev |
| Next by Thread: | Re: [2.6 patch] malta_mtd.c: add MODULE_LICENSE, Dmitri Vorobiev |
| Indexes: | [Date] [Thread] [Top] [All Lists] |