mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: External PHY in Linux smc91x driver
       [not found] <4765ECCD.8030309@systemsolve.net>
@ 2007-12-17 18:50 ` Shaun Jackman
  0 siblings, 0 replies; only message in thread
From: Shaun Jackman @ 2007-12-17 18:50 UTC (permalink / raw)
  To: David Keeffe; +Cc: LKML

Hello David, (cc'ed to LKML)

The patch, `smc91x: Add support for an external PHY' from 2006-07-19,
worked for me. It simply set the extphy bit of the smc91x to disable
the internal PHY and use the external MII interface. As you can see
from the patch, it doesn't support configuring the external PHY in any
way. Most PHYs power up to a default functional state. In further
work, someone could add the smc91x MII configuration interface to the
PHY abstraction layer. Incidentally, the PHY I was using (an unmanaged
switch chip) didn't even support configuration over the MII interface.
Apply this patch first as a minimum first measure. Test the patch, if
it works for you, sign off on it and repost it to the LKML.

Cheers,
Shaun

On Dec 16, 2007 8:28 PM, David Keeffe <david@systemsolve.net> wrote:
> Hi Shaun
>
> I've been asked to look at supporting an external (LXT) PHY for the
> SMC91x driver, and I noticed that you have published a patch for this.
> Is that all that's needed to make the SMC chip work with an external
> PHY? Presumably there are also parts which actually do the phy setup (if
> it's not an SMC component).
>
> I've been considering changing the driver to use the PHY abstraction
> layer, but that seems a big job if the problem has already been solved!
>
> Would you care to share what you've achieved?
>
> David Keeffe

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-12-17 18:50 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <4765ECCD.8030309@systemsolve.net>
2007-12-17 18:50 ` External PHY in Linux smc91x driver Shaun Jackman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®