* [PATCH] dt-bindings: net: mediatek: support MT7621 SoC
@ 2021-04-19 3:42 Ilya Lipnitskiy
2021-04-19 6:24 ` Bjørn Mork
2021-04-21 21:19 ` Rob Herring
0 siblings, 2 replies; 5+ messages in thread
From: Ilya Lipnitskiy @ 2021-04-19 3:42 UTC (permalink / raw)
To: David S. Miller, Jakub Kicinski, Rob Herring, Matthias Brugger,
Greg Ungerer, Florian Fainelli, Sean Wang, Bjørn Mork,
netdev, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek
Cc: Ilya Lipnitskiy
Add missing binding documentation for SoC support that has been in place
since v5.1
Fixes: 889bcbdeee57 ("net: ethernet: mediatek: support MT7621 SoC ethernet hardware")
Cc: Bjørn Mork <bjorn@mork.no>
Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
---
Documentation/devicetree/bindings/net/mediatek-net.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/net/mediatek-net.txt b/Documentation/devicetree/bindings/net/mediatek-net.txt
index 72d03e07cf7c..950ef6af20b1 100644
--- a/Documentation/devicetree/bindings/net/mediatek-net.txt
+++ b/Documentation/devicetree/bindings/net/mediatek-net.txt
@@ -10,6 +10,7 @@ Required properties:
- compatible: Should be
"mediatek,mt2701-eth": for MT2701 SoC
"mediatek,mt7623-eth", "mediatek,mt2701-eth": for MT7623 SoC
+ "mediatek,mt7621-eth": for MT7621 SoC
"mediatek,mt7622-eth": for MT7622 SoC
"mediatek,mt7629-eth": for MT7629 SoC
"ralink,rt5350-eth": for Ralink Rt5350F and MT7628/88 SoC
--
2.31.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: net: mediatek: support MT7621 SoC
2021-04-19 3:42 [PATCH] dt-bindings: net: mediatek: support MT7621 SoC Ilya Lipnitskiy
@ 2021-04-19 6:24 ` Bjørn Mork
2021-04-19 15:12 ` Ilya Lipnitskiy
2021-04-20 2:44 ` Ilya Lipnitskiy
2021-04-21 21:19 ` Rob Herring
1 sibling, 2 replies; 5+ messages in thread
From: Bjørn Mork @ 2021-04-19 6:24 UTC (permalink / raw)
To: Ilya Lipnitskiy
Cc: David S. Miller, Jakub Kicinski, Rob Herring, Matthias Brugger,
Greg Ungerer, Florian Fainelli, Sean Wang, netdev, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com> writes:
> Add missing binding documentation for SoC support that has been in place
> since v5.1
>
> Fixes: 889bcbdeee57 ("net: ethernet: mediatek: support MT7621 SoC ethernet hardware")
> Cc: Bjørn Mork <bjorn@mork.no>
> Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
> ---
> Documentation/devicetree/bindings/net/mediatek-net.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/net/mediatek-net.txt b/Documentation/devicetree/bindings/net/mediatek-net.txt
> index 72d03e07cf7c..950ef6af20b1 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-net.txt
> +++ b/Documentation/devicetree/bindings/net/mediatek-net.txt
> @@ -10,6 +10,7 @@ Required properties:
> - compatible: Should be
> "mediatek,mt2701-eth": for MT2701 SoC
> "mediatek,mt7623-eth", "mediatek,mt2701-eth": for MT7623 SoC
> + "mediatek,mt7621-eth": for MT7621 SoC
> "mediatek,mt7622-eth": for MT7622 SoC
> "mediatek,mt7629-eth": for MT7629 SoC
> "ralink,rt5350-eth": for Ralink Rt5350F and MT7628/88 SoC
Thanks for taking care of this!
Note, however, that this compatible value is defined in
Documentation/devicetree/bindings/net/ralink,rt2880-net.txt
I believe that file should go away. These two files are both documenting
the same compatible property AFAICS.
Bjørn
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: net: mediatek: support MT7621 SoC
2021-04-19 6:24 ` Bjørn Mork
@ 2021-04-19 15:12 ` Ilya Lipnitskiy
2021-04-20 2:44 ` Ilya Lipnitskiy
1 sibling, 0 replies; 5+ messages in thread
From: Ilya Lipnitskiy @ 2021-04-19 15:12 UTC (permalink / raw)
To: Bjørn Mork
Cc: David S. Miller, Jakub Kicinski, Rob Herring, Matthias Brugger,
Greg Ungerer, Florian Fainelli, Sean Wang, netdev,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Linux Kernel Mailing List, linux-arm-kernel, linux-mediatek
On Sun, Apr 18, 2021 at 11:24 PM Bjørn Mork <bjorn@mork.no> wrote:
>
> Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com> writes:
>
> > Add missing binding documentation for SoC support that has been in place
> > since v5.1
> >
> > Fixes: 889bcbdeee57 ("net: ethernet: mediatek: support MT7621 SoC ethernet hardware")
> > Cc: Bjørn Mork <bjorn@mork.no>
> > Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
> > ---
> > Documentation/devicetree/bindings/net/mediatek-net.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/mediatek-net.txt b/Documentation/devicetree/bindings/net/mediatek-net.txt
> > index 72d03e07cf7c..950ef6af20b1 100644
> > --- a/Documentation/devicetree/bindings/net/mediatek-net.txt
> > +++ b/Documentation/devicetree/bindings/net/mediatek-net.txt
> > @@ -10,6 +10,7 @@ Required properties:
> > - compatible: Should be
> > "mediatek,mt2701-eth": for MT2701 SoC
> > "mediatek,mt7623-eth", "mediatek,mt2701-eth": for MT7623 SoC
> > + "mediatek,mt7621-eth": for MT7621 SoC
> > "mediatek,mt7622-eth": for MT7622 SoC
> > "mediatek,mt7629-eth": for MT7629 SoC
> > "ralink,rt5350-eth": for Ralink Rt5350F and MT7628/88 SoC
>
>
> Thanks for taking care of this!
>
> Note, however, that this compatible value is defined in
> Documentation/devicetree/bindings/net/ralink,rt2880-net.txt
Good point. I don't think there is a driver in-tree for that binding.
It looks like commit 663148e48a66 ("Documentation: DT: net: add docs
for ralink/mediatek SoC ethernet binding") should just be reverted and
the three documents (mediatek,mt7620-gsw.txt; ralink,rt2880-net.txt;
ralink,rt3050-esw.txt) removed. Any objections?
Ilya
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: net: mediatek: support MT7621 SoC
2021-04-19 6:24 ` Bjørn Mork
2021-04-19 15:12 ` Ilya Lipnitskiy
@ 2021-04-20 2:44 ` Ilya Lipnitskiy
1 sibling, 0 replies; 5+ messages in thread
From: Ilya Lipnitskiy @ 2021-04-20 2:44 UTC (permalink / raw)
To: Bjørn Mork
Cc: David S. Miller, Jakub Kicinski, Rob Herring, Matthias Brugger,
Greg Ungerer, Florian Fainelli, Sean Wang, netdev,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Linux Kernel Mailing List, linux-arm-kernel, linux-mediatek
On Sun, Apr 18, 2021 at 11:24 PM Bjørn Mork <bjorn@mork.no> wrote:
>
> Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com> writes:
>
> > Add missing binding documentation for SoC support that has been in place
> > since v5.1
> >
> > Fixes: 889bcbdeee57 ("net: ethernet: mediatek: support MT7621 SoC ethernet hardware")
> > Cc: Bjørn Mork <bjorn@mork.no>
> > Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
> > ---
> > Documentation/devicetree/bindings/net/mediatek-net.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/mediatek-net.txt b/Documentation/devicetree/bindings/net/mediatek-net.txt
> > index 72d03e07cf7c..950ef6af20b1 100644
> > --- a/Documentation/devicetree/bindings/net/mediatek-net.txt
> > +++ b/Documentation/devicetree/bindings/net/mediatek-net.txt
> > @@ -10,6 +10,7 @@ Required properties:
> > - compatible: Should be
> > "mediatek,mt2701-eth": for MT2701 SoC
> > "mediatek,mt7623-eth", "mediatek,mt2701-eth": for MT7623 SoC
> > + "mediatek,mt7621-eth": for MT7621 SoC
> > "mediatek,mt7622-eth": for MT7622 SoC
> > "mediatek,mt7629-eth": for MT7629 SoC
> > "ralink,rt5350-eth": for Ralink Rt5350F and MT7628/88 SoC
>
>
> Thanks for taking care of this!
>
> Note, however, that this compatible value is defined in
> Documentation/devicetree/bindings/net/ralink,rt2880-net.txt
>
> I believe that file should go away. These two files are both documenting
> the same compatible property AFAICS.
Removed along with two others in
https://lore.kernel.org/lkml/20210420024222.101615-1-ilya.lipnitskiy@gmail.com/T/#u
Ilya
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: net: mediatek: support MT7621 SoC
2021-04-19 3:42 [PATCH] dt-bindings: net: mediatek: support MT7621 SoC Ilya Lipnitskiy
2021-04-19 6:24 ` Bjørn Mork
@ 2021-04-21 21:19 ` Rob Herring
1 sibling, 0 replies; 5+ messages in thread
From: Rob Herring @ 2021-04-21 21:19 UTC (permalink / raw)
To: Ilya Lipnitskiy
Cc: Jakub Kicinski, Rob Herring, Greg Ungerer, Florian Fainelli,
linux-arm-kernel, netdev, devicetree, linux-kernel,
linux-mediatek, Bjørn Mork, David S. Miller,
Matthias Brugger, Sean Wang
On Sun, 18 Apr 2021 20:42:53 -0700, Ilya Lipnitskiy wrote:
> Add missing binding documentation for SoC support that has been in place
> since v5.1
>
> Fixes: 889bcbdeee57 ("net: ethernet: mediatek: support MT7621 SoC ethernet hardware")
> Cc: Bjørn Mork <bjorn@mork.no>
> Signed-off-by: Ilya Lipnitskiy <ilya.lipnitskiy@gmail.com>
> ---
> Documentation/devicetree/bindings/net/mediatek-net.txt | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-04-21 21:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-19 3:42 [PATCH] dt-bindings: net: mediatek: support MT7621 SoC Ilya Lipnitskiy
2021-04-19 6:24 ` Bjørn Mork
2021-04-19 15:12 ` Ilya Lipnitskiy
2021-04-20 2:44 ` Ilya Lipnitskiy
2021-04-21 21:19 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).