From: Andrew Lunn <andrew@lunn.ch>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: Phil Reid <preid@electromag.com.au>,
netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
dongsheng.wang@hxt-semitech.com, cphealy@gmail.com,
clemens.gruber@pqgruber.com, hkallweit1@gmail.com, nbd@nbd.name,
harini.katakam@xilinx.com
Subject: Re: regression from: net: phy: marvell: Avoid unnecessary soft reset
Date: Mon, 18 Mar 2019 18:15:56 +0100 [thread overview]
Message-ID: <20190318171556.GC23030@lunn.ch> (raw)
In-Reply-To: <e4ccd3c6-62af-adf0-cbf3-5d25ad7be89b@gmail.com>
> Thanks, you mentioned a mv88e6085 but that chip is a 10/100 switch, did
> you mean that the mv88e6085 compatible string is used in Device Tree to
> designate that chip? While the 88E1540 driver is picked up, that driver
> is originally for external PHY packages (AFAICT), so there could be some
> switch-specific integration of this PHY that makes it behave
> differently, having the OUI helps narrow down what might be necessary to
> accomplish.
>
> FWIW, the changes were originally tested with a 88e6352.
Hi Florian
Some of the older Marvell switches use the 1540 for its internal
PHYs. Same OUI as the external PHY. I don't remember which
switches. Maybe the mv88e6161 in the RDU1?
Andrew
next prev parent reply other threads:[~2019-03-18 17:16 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-25 18:28 [PATCH net-next 0/2] net: phy: Eliminate unnecessary soft Florian Fainelli
2018-09-25 18:28 ` [PATCH net-next 1/2] net: phy: Stop with excessive soft reset Florian Fainelli
2018-09-25 18:28 ` [PATCH net-next 2/2] net: phy: marvell: Avoid unnecessary " Florian Fainelli
2019-03-15 8:52 ` regression from: " Phil Reid
2019-03-15 21:58 ` Florian Fainelli
2019-03-18 2:11 ` Phil Reid
2019-03-18 17:09 ` Florian Fainelli
2019-03-18 17:15 ` Andrew Lunn [this message]
2019-03-18 17:18 ` Chris Healy
2019-03-18 17:53 ` Andrew Lunn
2019-03-19 1:32 ` Phil Reid
2019-03-19 16:53 ` Florian Fainelli
2019-03-20 1:33 ` Phil Reid
2019-03-20 2:34 ` liweihang
2019-03-20 3:37 ` Florian Fainelli
2019-03-20 5:16 ` Phil Reid
2019-03-20 6:39 ` Heiner Kallweit
2019-03-20 7:08 ` Phil Reid
2019-03-20 12:22 ` liweihang
2019-03-20 18:15 ` Heiner Kallweit
2019-03-20 20:35 ` Maxim Uvarov
2019-03-21 6:16 ` liweihang
2019-03-21 8:07 ` Phil Reid
2018-09-26 3:27 ` [PATCH net-next 0/2] net: phy: Eliminate unnecessary soft 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=20190318171556.GC23030@lunn.ch \
--to=andrew@lunn.ch \
--cc=clemens.gruber@pqgruber.com \
--cc=cphealy@gmail.com \
--cc=davem@davemloft.net \
--cc=dongsheng.wang@hxt-semitech.com \
--cc=f.fainelli@gmail.com \
--cc=harini.katakam@xilinx.com \
--cc=hkallweit1@gmail.com \
--cc=nbd@nbd.name \
--cc=netdev@vger.kernel.org \
--cc=preid@electromag.com.au \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.