public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] Add support for Generic PHY in macb
Date: Mon, 25 Feb 2008 20:30:13 -0500	[thread overview]
Message-ID: <47C36BA5.5090204@gmail.com> (raw)
In-Reply-To: <718910.92209.qm@web26215.mail.ukl.yahoo.com>

Michael Trimarchi wrote:
> Hi,
> ----- Messaggio originale -----
>   
>> Da: Haavard Skinnemoen <hskinnemoen@atmel.com>
>> A: Michael Trimarchi <trimarchimichael@yahoo.it>
>> Cc: Michael Schwingen <rincewind@discworld.dascon.de>; u-boot-users at lists.sourceforge.net
>> Inviato: Venerd? 22 febbraio 2008, 17:08:56
>> Oggetto: Re: [U-Boot-Users] [PATCH] Add support for Generic PHY in macb
>>
>> On Fri, 22 Feb 2008 16:02:13 +0000 (GMT)
>> Michael Trimarchi  wrote:
>>
>>     
>>>> In any case, the dm9161a PHY in question does implement MDIO, so it
>>>> really should work without any modifications.
>>>>   
>>>>         
>>> Maybe we can add the for cycle like in linux to check the phy addr?
>>>       
>> Yes, we could do that. But it comes with its own share of problems --
>> some boards have a too weak pull-up (or none at all) on the MDIO line,
>> so the probing code sometimes thinks it has found a whole bunch of
>> PHYs...
>>
>> Haavard
>>
>>     
> But the u-boot user may to choice to use the automatic detection, based on
> phy_id mask and stop the search on the first one, or somenthing like this.
>
> Michael
>   
I don't follow why you'd want to do this.  As Andy, Haavard and others 
have pointed out, you can't probe for PHY addresses because quite often 
multiple ports will share an MDIO bus, and also it's unreliable at 
best.  Anybody who writes U-boot code needs to know the details of his 
or her board, and PHY addresses are always fixed in hardware.  There may 
be some reason to make them dynamic, but I can't imagine why.

regards,
Ben

  reply	other threads:[~2008-02-26  1:30 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-22 16:19 [U-Boot-Users] [PATCH] Add support for Generic PHY in macb Michael Trimarchi
2008-02-26  1:30 ` Ben Warren [this message]
  -- strict thread matches above, loose matches on Subject: below --
2008-02-22 16:02 Michael Trimarchi
2008-02-22 16:08 ` Haavard Skinnemoen
2008-02-22 16:22   ` Michael Schwingen
2008-02-23 12:23   ` michael
2008-02-24  9:10     ` Michael Schwingen
2008-02-24 13:30       ` trimarchi at gandalf.sssup.it
2008-02-24 16:36         ` Michael Schwingen
2008-02-24 17:48           ` trimarchi at gandalf.sssup.it
2008-02-24 18:24         ` Andy Fleming
2008-02-22 14:07 Michael Trimarchi
2008-02-22 12:29 Michael Trimarchi
2008-02-22 12:27 Michael Trimarchi
2008-02-22 12:52 ` Haavard Skinnemoen
2008-02-22 14:05   ` michael
2008-02-22 14:39     ` Haavard Skinnemoen
2008-02-22 15:05   ` Michael Schwingen
2008-02-22 15:34     ` Haavard Skinnemoen
2008-02-22 15:46       ` Michael Schwingen
2008-02-26  1:23       ` Ben Warren
2008-02-22 11:41 michael
2008-02-22 12:16 ` Haavard Skinnemoen

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=47C36BA5.5090204@gmail.com \
    --to=biggerbadderben@gmail.com \
    --cc=u-boot@lists.denx.de \
    /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