* [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 2:26 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 2:26 UTC (permalink / raw)
To: cphealy, andrew, hkallweit1, linux, davem, neil.armstrong,
khilman, netdev, linux-arm-kernel, linux-amlogic, linux-kernel,
jeremy.wang
Cc: Chris Healy
From: Chris Healy <healych@amazon.com>
The Meson G12A Internal PHY does not support standard IEEE MMD extended
register access, therefore add generic dummy stubs to fail the read and
write MMD calls. This is necessary to prevent the core PHY code from
erroneously believing that EEE is supported by this PHY even though this
PHY does not support EEE, as MMD register access returns all FFFFs.
Signed-off-by: Chris Healy <healych@amazon.com>
---
drivers/net/phy/meson-gxl.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
index c49062ad72c6..5e41658b1e2f 100644
--- a/drivers/net/phy/meson-gxl.c
+++ b/drivers/net/phy/meson-gxl.c
@@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
.handle_interrupt = meson_gxl_handle_interrupt,
.suspend = genphy_suspend,
.resume = genphy_resume,
+ .read_mmd = genphy_read_mmd_unsupported,
+ .write_mmd = genphy_write_mmd_unsupported,
},
};
--
2.39.1
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 2:26 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 2:26 UTC (permalink / raw)
To: cphealy, andrew, hkallweit1, linux, davem, neil.armstrong,
khilman, netdev, linux-arm-kernel, linux-amlogic, linux-kernel,
jeremy.wang
Cc: Chris Healy
From: Chris Healy <healych@amazon.com>
The Meson G12A Internal PHY does not support standard IEEE MMD extended
register access, therefore add generic dummy stubs to fail the read and
write MMD calls. This is necessary to prevent the core PHY code from
erroneously believing that EEE is supported by this PHY even though this
PHY does not support EEE, as MMD register access returns all FFFFs.
Signed-off-by: Chris Healy <healych@amazon.com>
---
drivers/net/phy/meson-gxl.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
index c49062ad72c6..5e41658b1e2f 100644
--- a/drivers/net/phy/meson-gxl.c
+++ b/drivers/net/phy/meson-gxl.c
@@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
.handle_interrupt = meson_gxl_handle_interrupt,
.suspend = genphy_suspend,
.resume = genphy_resume,
+ .read_mmd = genphy_read_mmd_unsupported,
+ .write_mmd = genphy_write_mmd_unsupported,
},
};
--
2.39.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
2023-01-29 2:26 ` Chris Healy
(?)
@ 2023-01-29 10:26 ` Heiner Kallweit
-1 siblings, 0 replies; 18+ messages in thread
From: Heiner Kallweit @ 2023-01-29 10:26 UTC (permalink / raw)
To: Chris Healy, andrew, linux, davem, neil.armstrong, khilman,
netdev, linux-arm-kernel, linux-amlogic, linux-kernel,
jeremy.wang
Cc: Chris Healy
On 29.01.2023 03:26, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
Hi Chris,
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Signed-off-by: Chris Healy <healych@amazon.com>
> ---
> drivers/net/phy/meson-gxl.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> index c49062ad72c6..5e41658b1e2f 100644
> --- a/drivers/net/phy/meson-gxl.c
> +++ b/drivers/net/phy/meson-gxl.c
> @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> .handle_interrupt = meson_gxl_handle_interrupt,
> .suspend = genphy_suspend,
> .resume = genphy_resume,
> + .read_mmd = genphy_read_mmd_unsupported,
> + .write_mmd = genphy_write_mmd_unsupported,
> },
> };
>
thanks for catching this. The same issue we may have for the GXL-internal PHY.
Did you check this?
One result of the issue is the invalid ethtool --show-eee output given below.
Therefore the patch should go to stable, please annotate it as [PATCH net].
Fixes tag should be:
5c3407abb338 ("net: phy: meson-gxl: add g12a support")
EEE settings for eth0:
EEE status: enabled - active
Tx LPI: 1000000 (us)
Supported EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Link partner advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 10:26 ` Heiner Kallweit
0 siblings, 0 replies; 18+ messages in thread
From: Heiner Kallweit @ 2023-01-29 10:26 UTC (permalink / raw)
To: Chris Healy, andrew, linux, davem, neil.armstrong, khilman,
netdev, linux-arm-kernel, linux-amlogic, linux-kernel,
jeremy.wang
Cc: Chris Healy
On 29.01.2023 03:26, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
Hi Chris,
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Signed-off-by: Chris Healy <healych@amazon.com>
> ---
> drivers/net/phy/meson-gxl.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> index c49062ad72c6..5e41658b1e2f 100644
> --- a/drivers/net/phy/meson-gxl.c
> +++ b/drivers/net/phy/meson-gxl.c
> @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> .handle_interrupt = meson_gxl_handle_interrupt,
> .suspend = genphy_suspend,
> .resume = genphy_resume,
> + .read_mmd = genphy_read_mmd_unsupported,
> + .write_mmd = genphy_write_mmd_unsupported,
> },
> };
>
thanks for catching this. The same issue we may have for the GXL-internal PHY.
Did you check this?
One result of the issue is the invalid ethtool --show-eee output given below.
Therefore the patch should go to stable, please annotate it as [PATCH net].
Fixes tag should be:
5c3407abb338 ("net: phy: meson-gxl: add g12a support")
EEE settings for eth0:
EEE status: enabled - active
Tx LPI: 1000000 (us)
Supported EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Link partner advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 10:26 ` Heiner Kallweit
0 siblings, 0 replies; 18+ messages in thread
From: Heiner Kallweit @ 2023-01-29 10:26 UTC (permalink / raw)
To: Chris Healy, andrew, linux, davem, neil.armstrong, khilman,
netdev, linux-arm-kernel, linux-amlogic, linux-kernel,
jeremy.wang
Cc: Chris Healy
On 29.01.2023 03:26, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
Hi Chris,
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Signed-off-by: Chris Healy <healych@amazon.com>
> ---
> drivers/net/phy/meson-gxl.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> index c49062ad72c6..5e41658b1e2f 100644
> --- a/drivers/net/phy/meson-gxl.c
> +++ b/drivers/net/phy/meson-gxl.c
> @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> .handle_interrupt = meson_gxl_handle_interrupt,
> .suspend = genphy_suspend,
> .resume = genphy_resume,
> + .read_mmd = genphy_read_mmd_unsupported,
> + .write_mmd = genphy_write_mmd_unsupported,
> },
> };
>
thanks for catching this. The same issue we may have for the GXL-internal PHY.
Did you check this?
One result of the issue is the invalid ethtool --show-eee output given below.
Therefore the patch should go to stable, please annotate it as [PATCH net].
Fixes tag should be:
5c3407abb338 ("net: phy: meson-gxl: add g12a support")
EEE settings for eth0:
EEE status: enabled - active
Tx LPI: 1000000 (us)
Supported EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
Link partner advertised EEE link modes: 100baseT/Full
1000baseT/Full
10000baseT/Full
1000baseKX/Full
10000baseKX4/Full
10000baseKR/Full
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
2023-01-29 10:26 ` Heiner Kallweit
(?)
@ 2023-01-29 21:25 ` Chris Healy
-1 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:25 UTC (permalink / raw)
To: Heiner Kallweit
Cc: andrew, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 2:26 AM Heiner Kallweit <hkallweit1@gmail.com> wrote:
>
> On 29.01.2023 03:26, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> Hi Chris,
>
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
> >
> > Signed-off-by: Chris Healy <healych@amazon.com>
> > ---
> > drivers/net/phy/meson-gxl.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> > index c49062ad72c6..5e41658b1e2f 100644
> > --- a/drivers/net/phy/meson-gxl.c
> > +++ b/drivers/net/phy/meson-gxl.c
> > @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> > .handle_interrupt = meson_gxl_handle_interrupt,
> > .suspend = genphy_suspend,
> > .resume = genphy_resume,
> > + .read_mmd = genphy_read_mmd_unsupported,
> > + .write_mmd = genphy_write_mmd_unsupported,
> > },
> > };
> >
>
> thanks for catching this. The same issue we may have for the GXL-internal PHY.
> Did you check this?
I do not have HW with GXL-internal PHY so I cannot test it and don't
feel comfortable making the change for that PHY ID without someone
providing the necessary feedback. If someone can confirm the same all
FFFFs with the MMD registers, I'd be happy to add the read/write MMD
callbacks for that PHY in v2.
>
> One result of the issue is the invalid ethtool --show-eee output given below.
> Therefore the patch should go to stable, please annotate it as [PATCH net].
>
> Fixes tag should be:
> 5c3407abb338 ("net: phy: meson-gxl: add g12a support")
Yep, looking at the ethtool --show-eee output is what got me looking
at this in the first place. I was hoping to enable EEE... ;-)
I'll add the fixes tag in v2.
>
>
> EEE settings for eth0:
> EEE status: enabled - active
> Tx LPI: 1000000 (us)
> Supported EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Link partner advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
>
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:25 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:25 UTC (permalink / raw)
To: Heiner Kallweit
Cc: andrew, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 2:26 AM Heiner Kallweit <hkallweit1@gmail.com> wrote:
>
> On 29.01.2023 03:26, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> Hi Chris,
>
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
> >
> > Signed-off-by: Chris Healy <healych@amazon.com>
> > ---
> > drivers/net/phy/meson-gxl.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> > index c49062ad72c6..5e41658b1e2f 100644
> > --- a/drivers/net/phy/meson-gxl.c
> > +++ b/drivers/net/phy/meson-gxl.c
> > @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> > .handle_interrupt = meson_gxl_handle_interrupt,
> > .suspend = genphy_suspend,
> > .resume = genphy_resume,
> > + .read_mmd = genphy_read_mmd_unsupported,
> > + .write_mmd = genphy_write_mmd_unsupported,
> > },
> > };
> >
>
> thanks for catching this. The same issue we may have for the GXL-internal PHY.
> Did you check this?
I do not have HW with GXL-internal PHY so I cannot test it and don't
feel comfortable making the change for that PHY ID without someone
providing the necessary feedback. If someone can confirm the same all
FFFFs with the MMD registers, I'd be happy to add the read/write MMD
callbacks for that PHY in v2.
>
> One result of the issue is the invalid ethtool --show-eee output given below.
> Therefore the patch should go to stable, please annotate it as [PATCH net].
>
> Fixes tag should be:
> 5c3407abb338 ("net: phy: meson-gxl: add g12a support")
Yep, looking at the ethtool --show-eee output is what got me looking
at this in the first place. I was hoping to enable EEE... ;-)
I'll add the fixes tag in v2.
>
>
> EEE settings for eth0:
> EEE status: enabled - active
> Tx LPI: 1000000 (us)
> Supported EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Link partner advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
>
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:25 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:25 UTC (permalink / raw)
To: Heiner Kallweit
Cc: andrew, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 2:26 AM Heiner Kallweit <hkallweit1@gmail.com> wrote:
>
> On 29.01.2023 03:26, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> Hi Chris,
>
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
> >
> > Signed-off-by: Chris Healy <healych@amazon.com>
> > ---
> > drivers/net/phy/meson-gxl.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/net/phy/meson-gxl.c b/drivers/net/phy/meson-gxl.c
> > index c49062ad72c6..5e41658b1e2f 100644
> > --- a/drivers/net/phy/meson-gxl.c
> > +++ b/drivers/net/phy/meson-gxl.c
> > @@ -271,6 +271,8 @@ static struct phy_driver meson_gxl_phy[] = {
> > .handle_interrupt = meson_gxl_handle_interrupt,
> > .suspend = genphy_suspend,
> > .resume = genphy_resume,
> > + .read_mmd = genphy_read_mmd_unsupported,
> > + .write_mmd = genphy_write_mmd_unsupported,
> > },
> > };
> >
>
> thanks for catching this. The same issue we may have for the GXL-internal PHY.
> Did you check this?
I do not have HW with GXL-internal PHY so I cannot test it and don't
feel comfortable making the change for that PHY ID without someone
providing the necessary feedback. If someone can confirm the same all
FFFFs with the MMD registers, I'd be happy to add the read/write MMD
callbacks for that PHY in v2.
>
> One result of the issue is the invalid ethtool --show-eee output given below.
> Therefore the patch should go to stable, please annotate it as [PATCH net].
>
> Fixes tag should be:
> 5c3407abb338 ("net: phy: meson-gxl: add g12a support")
Yep, looking at the ethtool --show-eee output is what got me looking
at this in the first place. I was hoping to enable EEE... ;-)
I'll add the fixes tag in v2.
>
>
> EEE settings for eth0:
> EEE status: enabled - active
> Tx LPI: 1000000 (us)
> Supported EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
> Link partner advertised EEE link modes: 100baseT/Full
> 1000baseT/Full
> 10000baseT/Full
> 1000baseKX/Full
> 10000baseKX4/Full
> 10000baseKR/Full
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
2023-01-29 2:26 ` Chris Healy
(?)
@ 2023-01-29 16:51 ` Andrew Lunn
-1 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 16:51 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
Hi Chris
This change in itself makes sense. But i wounder if we should also
change phy_init_eee(). It reads the EEE Ability register. The 2018
version of the standard indicates the top two bits are reserved and
should be zero. We also don't have any PHY which supports 100GBase-R
through to 100Base-TX. So a read of 0xffff suggests the PHY does not
support EEE and returning -EPROTONOSUPPORT would be good.
Andrew
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 16:51 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 16:51 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
Hi Chris
This change in itself makes sense. But i wounder if we should also
change phy_init_eee(). It reads the EEE Ability register. The 2018
version of the standard indicates the top two bits are reserved and
should be zero. We also don't have any PHY which supports 100GBase-R
through to 100Base-TX. So a read of 0xffff suggests the PHY does not
support EEE and returning -EPROTONOSUPPORT would be good.
Andrew
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 16:51 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 16:51 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> From: Chris Healy <healych@amazon.com>
>
> The Meson G12A Internal PHY does not support standard IEEE MMD extended
> register access, therefore add generic dummy stubs to fail the read and
> write MMD calls. This is necessary to prevent the core PHY code from
> erroneously believing that EEE is supported by this PHY even though this
> PHY does not support EEE, as MMD register access returns all FFFFs.
Hi Chris
This change in itself makes sense. But i wounder if we should also
change phy_init_eee(). It reads the EEE Ability register. The 2018
version of the standard indicates the top two bits are reserved and
should be zero. We also don't have any PHY which supports 100GBase-R
through to 100Base-TX. So a read of 0xffff suggests the PHY does not
support EEE and returning -EPROTONOSUPPORT would be good.
Andrew
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
2023-01-29 16:51 ` Andrew Lunn
(?)
@ 2023-01-29 21:28 ` Chris Healy
-1 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:28 UTC (permalink / raw)
To: Andrew Lunn
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 8:52 AM Andrew Lunn <andrew@lunn.ch> wrote:
>
> On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Hi Chris
>
> This change in itself makes sense. But i wounder if we should also
> change phy_init_eee(). It reads the EEE Ability register. The 2018
> version of the standard indicates the top two bits are reserved and
> should be zero. We also don't have any PHY which supports 100GBase-R
> through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> support EEE and returning -EPROTONOSUPPORT would be good.
>
This seems like a good change to make in phy_init_eee(). I should be
able to do this. Can this be done in a subsequent patch or does this
need to be present to land this meson specific PHY patch?
> Andrew
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:28 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:28 UTC (permalink / raw)
To: Andrew Lunn
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 8:52 AM Andrew Lunn <andrew@lunn.ch> wrote:
>
> On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Hi Chris
>
> This change in itself makes sense. But i wounder if we should also
> change phy_init_eee(). It reads the EEE Ability register. The 2018
> version of the standard indicates the top two bits are reserved and
> should be zero. We also don't have any PHY which supports 100GBase-R
> through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> support EEE and returning -EPROTONOSUPPORT would be good.
>
This seems like a good change to make in phy_init_eee(). I should be
able to do this. Can this be done in a subsequent patch or does this
need to be present to land this meson specific PHY patch?
> Andrew
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:28 ` Chris Healy
0 siblings, 0 replies; 18+ messages in thread
From: Chris Healy @ 2023-01-29 21:28 UTC (permalink / raw)
To: Andrew Lunn
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
On Sun, Jan 29, 2023 at 8:52 AM Andrew Lunn <andrew@lunn.ch> wrote:
>
> On Sat, Jan 28, 2023 at 06:26:15PM -0800, Chris Healy wrote:
> > From: Chris Healy <healych@amazon.com>
> >
> > The Meson G12A Internal PHY does not support standard IEEE MMD extended
> > register access, therefore add generic dummy stubs to fail the read and
> > write MMD calls. This is necessary to prevent the core PHY code from
> > erroneously believing that EEE is supported by this PHY even though this
> > PHY does not support EEE, as MMD register access returns all FFFFs.
>
> Hi Chris
>
> This change in itself makes sense. But i wounder if we should also
> change phy_init_eee(). It reads the EEE Ability register. The 2018
> version of the standard indicates the top two bits are reserved and
> should be zero. We also don't have any PHY which supports 100GBase-R
> through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> support EEE and returning -EPROTONOSUPPORT would be good.
>
This seems like a good change to make in phy_init_eee(). I should be
able to do this. Can this be done in a subsequent patch or does this
need to be present to land this meson specific PHY patch?
> Andrew
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
2023-01-29 21:28 ` Chris Healy
(?)
@ 2023-01-29 21:53 ` Andrew Lunn
-1 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 21:53 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
> > Hi Chris
> >
> > This change in itself makes sense. But i wounder if we should also
> > change phy_init_eee(). It reads the EEE Ability register. The 2018
> > version of the standard indicates the top two bits are reserved and
> > should be zero. We also don't have any PHY which supports 100GBase-R
> > through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> > support EEE and returning -EPROTONOSUPPORT would be good.
> >
> This seems like a good change to make in phy_init_eee(). I should be
> able to do this. Can this be done in a subsequent patch or does this
> need to be present to land this meson specific PHY patch?
They are independent, submit them in whatever order you want. But
doing it first will make it easier to test it actually works.
Andrew
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:53 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 21:53 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
> > Hi Chris
> >
> > This change in itself makes sense. But i wounder if we should also
> > change phy_init_eee(). It reads the EEE Ability register. The 2018
> > version of the standard indicates the top two bits are reserved and
> > should be zero. We also don't have any PHY which supports 100GBase-R
> > through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> > support EEE and returning -EPROTONOSUPPORT would be good.
> >
> This seems like a good change to make in phy_init_eee(). I should be
> able to do this. Can this be done in a subsequent patch or does this
> need to be present to land this meson specific PHY patch?
They are independent, submit them in whatever order you want. But
doing it first will make it easier to test it actually works.
Andrew
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH 1/1] net: phy: meson-gxl: Add generic dummy stubs for MMD register access
@ 2023-01-29 21:53 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2023-01-29 21:53 UTC (permalink / raw)
To: Chris Healy
Cc: hkallweit1, linux, davem, neil.armstrong, khilman, netdev,
linux-arm-kernel, linux-amlogic, linux-kernel, jeremy.wang,
Chris Healy
> > Hi Chris
> >
> > This change in itself makes sense. But i wounder if we should also
> > change phy_init_eee(). It reads the EEE Ability register. The 2018
> > version of the standard indicates the top two bits are reserved and
> > should be zero. We also don't have any PHY which supports 100GBase-R
> > through to 100Base-TX. So a read of 0xffff suggests the PHY does not
> > support EEE and returning -EPROTONOSUPPORT would be good.
> >
> This seems like a good change to make in phy_init_eee(). I should be
> able to do this. Can this be done in a subsequent patch or does this
> need to be present to land this meson specific PHY patch?
They are independent, submit them in whatever order you want. But
doing it first will make it easier to test it actually works.
Andrew
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread