[U-Boot] [PATCH] tsec: report when there is no vendor specific PHY support.

Wolfgang Denk wd at denx.de
Wed Feb 18 00:30:51 CET 2009


Dear Ben,

In message <29f2f2fe7e8a340b4bad9db14d2f6d3844cf8964.1228951215.git.paul.gortmaker at windriver.com>
Paul Gortmaker wrote:
> Commit af1c2b84 added a generic phy support, with an ID of zero
> and a 32 bit mask; meaning that it will match on any PHY ID.
> 
> The problem is that there is a test that checked if a matching
> PHY was found, and if not, it printed the non-matching ID.
> But since there will always be a match (on the generic PHY,
> worst case), this test will never trip.
> 
> In the case of a misconfigured PHY address, or of a PHY that
> isn't explicitly supported outside of the generic support,
> you will never see the ID of 0xffffffff, or the ID of the
> real (but unsupported) chip.  It will silently fall through
> onto the generic support.
> 
> This change makes that test useful again, and ensures that
> the selection of generic PHY support doesn't happen without
> some sort of notice.  It also makes it explicitly clear that
> the generic PHY must be last in the PHY table.
> 
> Signed-off-by: Paul Gortmaker <paul.gortmaker at windriver.com>
> ---
>  drivers/net/tsec.c |    7 +++----
>  1 files changed, 3 insertions(+), 4 deletions(-)

This patch was ACKed by Andy, but I don't think it was applied.

Was this intentional or by accident?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Monday is an awful way to spend one seventh of your life.


More information about the U-Boot mailing list