| To: | Atsushi Nemoto <anemo@mba.ocn.ne.jp> |
|---|---|
| Subject: | Re: [PATCH] TXx9: Add TX4939 RNG support |
| From: | Matt Mackall <mpm@selenic.com> |
| Date: | Sun, 31 May 2009 12:32:50 -0500 |
| Cc: | linux-mips@linux-mips.org, ralf@linux-mips.org, linux-kernel@vger.kernel.org, herbert@gondor.apana.org.au |
| In-reply-to: | <20090601.022335.200392387.anemo@mba.ocn.ne.jp> |
| Original-recipient: | rfc822;linux-mips@linux-mips.org |
| References: | <1243350141-883-2-git-send-email-anemo@mba.ocn.ne.jp> <20090601.015755.21367568.anemo@mba.ocn.ne.jp> <1243789289.22069.25.camel@calx> <20090601.022335.200392387.anemo@mba.ocn.ne.jp> |
| Sender: | linux-mips-bounce@linux-mips.org |
On Mon, 2009-06-01 at 02:23 +0900, Atsushi Nemoto wrote: > On Sun, 31 May 2009 12:01:29 -0500, Matt Mackall <mpm@selenic.com> wrote: > > > > Not clear to me how this works when this is a module? > > > > > > This patch add a registration of a platform device for RNG to vmlinux. > > > And the other patch add a driver module for the RNG. This strategy is > > > fairly common for SoCs or embedded platforms. > > > > If your driver is built as a module (which your patch allows), the above > > won't work, right? > > No, the rng driver can be used regardless of module or built-in, as > like as other platform drivers. Any special issue for hw_rng? I found the source of my confusion: you've given the init function in both files exactly the same name. So when I saw .._init at the bottom of the second patch, I assumed it was referring to the possibly not loaded driver's init code. -- http://selenic.com : development and support for Mercurial and Linux |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] TXx9: Add TX4939 RNG support, Atsushi Nemoto |
|---|---|
| Next by Date: | [PATCH 0/10] Updated bcm63xx support, Florian Fainelli |
| Previous by Thread: | Re: [PATCH] TXx9: Add TX4939 RNG support, Atsushi Nemoto |
| Next by Thread: | [PATCH] hwrng: Add TX4939 RNG driver, Atsushi Nemoto |
| Indexes: | [Date] [Thread] [Top] [All Lists] |