From: David Woodhouse <dwmw2@infradead.org>
To: Richard Cochran <richardcochran@gmail.com>
Cc: netdev@vger.kernel.org
Subject: Re: [PATCH] phylib: Add autoload support for the LXT973 phy.
Date: Wed, 23 Jun 2010 10:00:12 +0100 [thread overview]
Message-ID: <1277283612.16262.195.camel@macbook.infradead.org> (raw)
In-Reply-To: <20100623053723.GA3404@riccoc20.at.omicron.at>
On Wed, 2010-06-23 at 07:37 +0200, Richard Cochran wrote:
>
> Question about the whole PHY MODULE_DEVICE_TABLE system:
>
> I recently posted a phy driver for the National Semiconductor
> DP83640. During development, I used drivers/net/arm/ixp4xx_eth.c as
> the MAC driver, which was linked into the kernel (not a module). I
> noticed that the phy driver's probe function only gets called if the
> phy driver is also statically linked, but not when it is loaded as a
> module.
>
> Is this the correct behavior?
Hm, that seems like the _expected_ behaviour, certainly. The MAC driver
will probe its device at boot time, and will issue a request_module() to
load the a specific PHY driver if there is one. When no such module
turns up (which it won't if you have no file system mounted yet), it'll
just fall back to the generic PHY support.
--
David Woodhouse Open Source Technology Centre
David.Woodhouse@intel.com Intel Corporation
next prev parent reply other threads:[~2010-06-23 9:00 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-31 13:09 [PATCH] phylib: Add support for the LXT973 phy Richard Cochran
2010-06-01 22:39 ` Andy Fleming
2010-06-02 12:55 ` Richard Cochran
2010-06-02 13:07 ` Richard Cochran
2010-06-02 13:50 ` David Miller
2010-06-02 15:08 ` Richard Cochran
2010-06-02 15:15 ` David Miller
2010-06-03 11:28 ` Richard Cochran
2010-06-02 19:32 ` Andy Fleming
2010-06-05 14:00 ` Richard Cochran
2010-06-07 8:18 ` David Miller
2010-06-07 15:39 ` Andy Fleming
2010-06-07 15:39 ` Richard Cochran
2010-06-09 23:17 ` David Miller
2010-06-22 12:38 ` [PATCH] phylib: Add autoload " David Woodhouse
2010-06-23 5:37 ` Richard Cochran
2010-06-23 9:00 ` David Woodhouse [this message]
2010-06-27 5:16 ` David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1277283612.16262.195.camel@macbook.infradead.org \
--to=dwmw2@infradead.org \
--cc=netdev@vger.kernel.org \
--cc=richardcochran@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).