[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a4da45c6-7a04-c649-b841-ee1567c531ea@intel.com>
Date: Tue, 17 Jan 2023 17:08:22 -0800
From: Jesse Brandeburg <jesse.brandeburg@...el.com>
To: Michael Walle <michael@...le.cc>,
Heiner Kallweit <hkallweit1@...il.com>,
Russell King <linux@...linux.org.uk>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>, Felix Fietkau <nbd@....name>,
John Crispin <john@...ozen.org>,
Sean Wang <sean.wang@...iatek.com>,
Mark Lee <Mark-MC.Lee@...iatek.com>,
"Lorenzo Bianconi" <lorenzo@...nel.org>,
Matthias Brugger <matthias.bgg@...il.com>,
Bryan Whitehead <bryan.whitehead@...rochip.com>,
<UNGLinuxDriver@...rochip.com>,
Giuseppe Cavallaro <peppe.cavallaro@...com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Jose Abreu <joabreu@...opsys.com>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
"Joel Stanley" <joel@....id.au>, Andrew Jeffery <andrew@...id.au>
CC: <netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-mediatek@...ts.infradead.org>,
<linux-stm32@...md-mailman.stormreply.com>,
<linux-aspeed@...ts.ozlabs.org>, Andrew Lunn <andrew@...n.ch>
Subject: Re: [PATCH net-next 6/6] net: phy: Remove probe_capabilities
On 1/16/2023 4:55 AM, Michael Walle wrote:
> From: Andrew Lunn <andrew@...n.ch>
>
> Deciding if to probe of PHYs using C45 is now determine by if the bus
> provides the C45 read method. This makes probe_capabilities redundant
> so remove it.
>
> Signed-off-by: Andrew Lunn <andrew@...n.ch>
> Signed-off-by: Michael Walle <michael@...le.cc>
...
> diff --git a/include/linux/phy.h b/include/linux/phy.h
> index fceaac0fb319..fbeba4fee8d4 100644
> --- a/include/linux/phy.h
> +++ b/include/linux/phy.h
> @@ -419,14 +419,6 @@ struct mii_bus {
> /** @reset_gpiod: Reset GPIO descriptor pointer */
> struct gpio_desc *reset_gpiod;
>
> - /** @probe_capabilities: bus capabilities, used for probing */
> - enum {
> - MDIOBUS_NO_CAP = 0,
> - MDIOBUS_C22,
> - MDIOBUS_C45,
> - MDIOBUS_C22_C45,
> - } probe_capabilities;
> -
I'm a little surprised there is no Documentation regarding this stuff,
that needs to be removed or fixed up, but since there isn't...
Reviewed-by: Jesse Brandeburg <jesse.brandeburg@...el.com>
Powered by blists - more mailing lists