* [ECOS] devs/eth/phy/current/src/eth_phy.c @ 2004-05-26 21:10 John Newlin 2004-05-26 21:28 ` Gary Thomas 0 siblings, 1 reply; 7+ messages in thread From: John Newlin @ 2004-05-26 21:10 UTC (permalink / raw) To: ecos-discuss I see there looks that there was some attempt at making a generic phy interface, but I don't see that it is used anywhere. Is this a work in progress, or is there an example of this in use in some other code? Thanks, -john -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 21:10 [ECOS] devs/eth/phy/current/src/eth_phy.c John Newlin @ 2004-05-26 21:28 ` Gary Thomas 2004-05-26 21:30 ` Gary Thomas 0 siblings, 1 reply; 7+ messages in thread From: Gary Thomas @ 2004-05-26 21:28 UTC (permalink / raw) To: John Newlin; +Cc: ecos-discuss On Wed, 2004-05-26 at 15:03, John Newlin wrote: > I see there looks that there was some attempt at making a generic phy > interface, but I don't see that it is used anywhere. Is this a work in > progress, or is there an example of this in use in some other code? It *is* used - look at the rattler and MOAB packages. -- Gary Thomas <gary@mlbassoc.com> MLB Associates -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 21:28 ` Gary Thomas @ 2004-05-26 21:30 ` Gary Thomas 2004-05-26 22:08 ` John Newlin 0 siblings, 1 reply; 7+ messages in thread From: Gary Thomas @ 2004-05-26 21:30 UTC (permalink / raw) To: John Newlin; +Cc: ecos-discuss On Wed, 2004-05-26 at 15:07, Gary Thomas wrote: > On Wed, 2004-05-26 at 15:03, John Newlin wrote: > > I see there looks that there was some attempt at making a generic phy > > interface, but I don't see that it is used anywhere. Is this a work in > > progress, or is there an example of this in use in some other code? > > It *is* used - look at the rattler and MOAB packages. Also, look at the common ethernet drivers they use devs/eth/powerpc/{fec,fcc,ppc405} to see how the actual PHY interfaces work. -- Gary Thomas <gary@mlbassoc.com> MLB Associates -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 21:30 ` Gary Thomas @ 2004-05-26 22:08 ` John Newlin 2004-05-26 22:11 ` Gary Thomas 0 siblings, 1 reply; 7+ messages in thread From: John Newlin @ 2004-05-26 22:08 UTC (permalink / raw) To: Gary Thomas; +Cc: ecos-discuss > On Wed, 2004-05-26 at 15:07, Gary Thomas wrote: > > On Wed, 2004-05-26 at 15:03, John Newlin wrote: > > > I see there looks that there was some attempt at making a generic phy > > > interface, but I don't see that it is used anywhere. Is this a work in > > > progress, or is there an example of this in use in some other code? > > > > It *is* used - look at the rattler and MOAB packages. > > Also, look at the common ethernet drivers they use > devs/eth/powerpc/{fec,fcc,ppc405} > to see how the actual PHY interfaces work. > Thanks Gary! -john -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 22:08 ` John Newlin @ 2004-05-26 22:11 ` Gary Thomas 2004-05-26 22:30 ` John Newlin 0 siblings, 1 reply; 7+ messages in thread From: Gary Thomas @ 2004-05-26 22:11 UTC (permalink / raw) To: John Newlin; +Cc: ecos-discuss On Wed, 2004-05-26 at 15:28, John Newlin wrote: > > On Wed, 2004-05-26 at 15:07, Gary Thomas wrote: > > > On Wed, 2004-05-26 at 15:03, John Newlin wrote: > > > > I see there looks that there was some attempt at making a generic phy > > > > interface, but I don't see that it is used anywhere. Is this a work in > > > > progress, or is there an example of this in use in some other code? > > > > > > It *is* used - look at the rattler and MOAB packages. > > > > Also, look at the common ethernet drivers they use > > devs/eth/powerpc/{fec,fcc,ppc405} > > to see how the actual PHY interfaces work. > > > > Thanks Gary! No problem. Of course, I'm very interested in any feedback you have on these interfaces [since, so far, I'm the only one that has used them] -- Gary Thomas <gary@mlbassoc.com> MLB Associates -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 22:11 ` Gary Thomas @ 2004-05-26 22:30 ` John Newlin 2004-05-26 22:38 ` Gary Thomas 0 siblings, 1 reply; 7+ messages in thread From: John Newlin @ 2004-05-26 22:30 UTC (permalink / raw) To: Gary Thomas; +Cc: ecos-discuss > On Wed, 2004-05-26 at 15:28, John Newlin wrote: > > > On Wed, 2004-05-26 at 15:07, Gary Thomas wrote: > > > > On Wed, 2004-05-26 at 15:03, John Newlin wrote: > > > > > I see there looks that there was some attempt at making a generic phy > > > > > interface, but I don't see that it is used anywhere. Is this a work in > > > > > progress, or is there an example of this in use in some other code? > > > > > > > > It *is* used - look at the rattler and MOAB packages. > > > > > > Also, look at the common ethernet drivers they use > > > devs/eth/powerpc/{fec,fcc,ppc405} > > > to see how the actual PHY interfaces work. > > > > > > > Thanks Gary! > > No problem. Of course, I'm very interested in any feedback you have on > these interfaces [since, so far, I'm the only one that has used them] One comment. _eth_phy_init walks through the 32 possible PHY addresses to locate the PHY. For a chip with a single MAC interface this is fine. The chip I am working with has more than 1 MAC, but a single MII Management interface is used to configure multiple external PHYS. Maybe instead, have the driver that is supplying eth_phy_access_t, supply a valid PHY address, and it would just look at that PHY address for the PHY. Or it could pass a special value, like -1, to signify that the code should scan for the PHY. Alternatively, if the function pointers supplied in eth_phy_access_t took, eth_phy_access_t as the first parameters, you could stuff some extra data at the end of that structure, and the called functions could do the address check. Do you have another suggestion, surely there are other chips that have multiple MAC interfaces. I thought powerquicc had more than 1 mac. -john -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ECOS] devs/eth/phy/current/src/eth_phy.c 2004-05-26 22:30 ` John Newlin @ 2004-05-26 22:38 ` Gary Thomas 0 siblings, 0 replies; 7+ messages in thread From: Gary Thomas @ 2004-05-26 22:38 UTC (permalink / raw) To: John Newlin; +Cc: ecos-discuss On Wed, 2004-05-26 at 16:08, John Newlin wrote: > > On Wed, 2004-05-26 at 15:28, John Newlin wrote: > > > > On Wed, 2004-05-26 at 15:07, Gary Thomas wrote: > > > > > On Wed, 2004-05-26 at 15:03, John Newlin wrote: > > > > > > I see there looks that there was some attempt at making a generic phy > > > > > > interface, but I don't see that it is used anywhere. Is this a work in > > > > > > progress, or is there an example of this in use in some other code? > > > > > > > > > > It *is* used - look at the rattler and MOAB packages. > > > > > > > > Also, look at the common ethernet drivers they use > > > > devs/eth/powerpc/{fec,fcc,ppc405} > > > > to see how the actual PHY interfaces work. > > > > > > > > > > Thanks Gary! > > > > No problem. Of course, I'm very interested in any feedback you have on > > these interfaces [since, so far, I'm the only one that has used them] > > One comment. _eth_phy_init walks through the 32 possible PHY addresses to > locate the PHY. For a chip with a single MAC interface this is fine. > The chip I am working with has more than 1 MAC, but a single MII > Management interface is used to configure multiple external PHYS. > > Maybe instead, have the driver that is supplying eth_phy_access_t, supply > a valid PHY address, and it would just look at that PHY address for the > PHY. Or it could pass a special value, like -1, to signify that the code > should scan for the PHY. > > Alternatively, if the function pointers supplied in eth_phy_access_t took, > eth_phy_access_t as the first parameters, you could stuff some extra data > at the end of that structure, and the called functions could do the > address check. > > Do you have another suggestion, surely there are other chips that have > multiple MAC interfaces. I thought powerquicc had more than 1 mac. Indeed, but multiple networks does not always imply more than one MII bus or vice-versa. I actually have already solved this by letting the device instance set the PHY address if it knows it (which is normally the case). The changes just haven't made it into the public tree yet. -- Gary Thomas <gary@mlbassoc.com> MLB Associates -- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2004-05-26 22:11 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2004-05-26 21:10 [ECOS] devs/eth/phy/current/src/eth_phy.c John Newlin 2004-05-26 21:28 ` Gary Thomas 2004-05-26 21:30 ` Gary Thomas 2004-05-26 22:08 ` John Newlin 2004-05-26 22:11 ` Gary Thomas 2004-05-26 22:30 ` John Newlin 2004-05-26 22:38 ` Gary Thomas
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 read-only IMAP folder(s) and NNTP newsgroup(s).