From: Krzysztof Kozlowski <krzk@kernel.org> To: Daniel Golle <daniel@makrotopia.org> Cc: netdev@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Russell King" <linux@armlinux.org.uk>, "Heiner Kallweit" <hkallweit1@gmail.com>, "Lorenzo Bianconi" <lorenzo@kernel.org>, "Mark Lee" <Mark-MC.Lee@mediatek.com>, "John Crispin" <john@phrozen.org>, "Felix Fietkau" <nbd@nbd.name>, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, "Matthias Brugger" <matthias.bgg@gmail.com>, "DENG Qingfang" <dqfext@gmail.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "Sean Wang" <sean.wang@mediatek.com>, "Paolo Abeni" <pabeni@redhat.com>, "Jakub Kicinski" <kuba@kernel.org>, "Eric Dumazet" <edumazet@google.com>, "David S. Miller" <davem@davemloft.net>, "Vladimir Oltean" <olteanv@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Andrew Lunn" <andrew@lunn.ch>, "Jianhui Zhao" <zhaojh329@gmail.com>, "Bjørn Mork" <bjorn@mork.no> Subject: Re: [PATCH v2 03/11] dt-bindings: arm: mediatek: add 'mediatek,pn_swap' property Date: Wed, 8 Feb 2023 10:32:53 +0100 [thread overview] Message-ID: <b6d782ef-b375-1e73-a384-1ff37c1548a7@kernel.org> (raw) In-Reply-To: <Y+KR26aepqlfsjYG@makrotopia.org> On 07/02/2023 19:00, Daniel Golle wrote: > Hi Krzysztof, > > On Tue, Feb 07, 2023 at 06:38:23PM +0100, Krzysztof Kozlowski wrote: >> On 07/02/2023 15:19, Daniel Golle wrote: >>> Add documentation for the newly introduced 'mediatek,pn_swap' property >>> to mediatek,sgmiisys.txt. >>> >> >> Please use scripts/get_maintainers.pl to get a list of necessary people >> and lists to CC. It might happen, that command when run on an older >> kernel, gives you outdated entries. Therefore please be sure you base >> your patches on recent Linux kernel. >> >>> Signed-off-by: Daniel Golle <daniel@makrotopia.org> >>> --- >>> .../devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> index d2c24c277514..b38dd0fde21d 100644 >>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> @@ -14,6 +14,10 @@ Required Properties: >>> - "mediatek,mt7986-sgmiisys_1", "syscon" >>> - #clock-cells: Must be 1 >>> >>> +Optional Properties: >>> + >>> +- mediatek,pn_swap: Invert polarity of the SGMII data lanes. >> >> No: >> 1. No new properties for TXT bindings, > > So I'll have to convert the bindings to YAML, right? Yes, please. > >> 2. Underscore is not allowed. > > Ack, will change the name of the property. > >> 3. Does not look like property of this node. This is a clock controller >> or system controller, not SGMII/phy etc. > > The register range referred to by this node *does* represent also an > SGMII phy. These sgmiisys nodes also carry the 'syscon' compatible, and > are referenced in the node of the Ethernet core, and then used by > drivers/net/ethernet/mediatek/mtk_sgmii.c using syscon_node_to_regmap. > (This is the current situation already, and not related to the patchset > now adding only a new property to support hardware which needs that) Just because a register is located in syscon block, does not mean that SGMII configuration is a property of this device. > > So: Should I introduce a new binding for the same compatible strings > related to the SGMII PHY features? Or is it fine in this case to add > this property to the existing binding? The user of syscon should configure it. I don't think you need new binding. You just have to update the user of this syscon. Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org> To: Daniel Golle <daniel@makrotopia.org> Cc: netdev@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Russell King" <linux@armlinux.org.uk>, "Heiner Kallweit" <hkallweit1@gmail.com>, "Lorenzo Bianconi" <lorenzo@kernel.org>, "Mark Lee" <Mark-MC.Lee@mediatek.com>, "John Crispin" <john@phrozen.org>, "Felix Fietkau" <nbd@nbd.name>, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@collabora.com>, "Matthias Brugger" <matthias.bgg@gmail.com>, "DENG Qingfang" <dqfext@gmail.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "Sean Wang" <sean.wang@mediatek.com>, "Paolo Abeni" <pabeni@redhat.com>, "Jakub Kicinski" <kuba@kernel.org>, "Eric Dumazet" <edumazet@google.com>, "David S. Miller" <davem@davemloft.net>, "Vladimir Oltean" <olteanv@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Andrew Lunn" <andrew@lunn.ch>, "Jianhui Zhao" <zhaojh329@gmail.com>, "Bjørn Mork" <bjorn@mork.no> Subject: Re: [PATCH v2 03/11] dt-bindings: arm: mediatek: add 'mediatek,pn_swap' property Date: Wed, 8 Feb 2023 10:32:53 +0100 [thread overview] Message-ID: <b6d782ef-b375-1e73-a384-1ff37c1548a7@kernel.org> (raw) In-Reply-To: <Y+KR26aepqlfsjYG@makrotopia.org> On 07/02/2023 19:00, Daniel Golle wrote: > Hi Krzysztof, > > On Tue, Feb 07, 2023 at 06:38:23PM +0100, Krzysztof Kozlowski wrote: >> On 07/02/2023 15:19, Daniel Golle wrote: >>> Add documentation for the newly introduced 'mediatek,pn_swap' property >>> to mediatek,sgmiisys.txt. >>> >> >> Please use scripts/get_maintainers.pl to get a list of necessary people >> and lists to CC. It might happen, that command when run on an older >> kernel, gives you outdated entries. Therefore please be sure you base >> your patches on recent Linux kernel. >> >>> Signed-off-by: Daniel Golle <daniel@makrotopia.org> >>> --- >>> .../devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> index d2c24c277514..b38dd0fde21d 100644 >>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt >>> @@ -14,6 +14,10 @@ Required Properties: >>> - "mediatek,mt7986-sgmiisys_1", "syscon" >>> - #clock-cells: Must be 1 >>> >>> +Optional Properties: >>> + >>> +- mediatek,pn_swap: Invert polarity of the SGMII data lanes. >> >> No: >> 1. No new properties for TXT bindings, > > So I'll have to convert the bindings to YAML, right? Yes, please. > >> 2. Underscore is not allowed. > > Ack, will change the name of the property. > >> 3. Does not look like property of this node. This is a clock controller >> or system controller, not SGMII/phy etc. > > The register range referred to by this node *does* represent also an > SGMII phy. These sgmiisys nodes also carry the 'syscon' compatible, and > are referenced in the node of the Ethernet core, and then used by > drivers/net/ethernet/mediatek/mtk_sgmii.c using syscon_node_to_regmap. > (This is the current situation already, and not related to the patchset > now adding only a new property to support hardware which needs that) Just because a register is located in syscon block, does not mean that SGMII configuration is a property of this device. > > So: Should I introduce a new binding for the same compatible strings > related to the SGMII PHY features? Or is it fine in this case to add > this property to the existing binding? The user of syscon should configure it. I don't think you need new binding. You just have to update the user of this syscon. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-08 9:33 UTC|newest] Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-07 14:18 [PATCH v2 00/11] net: ethernet: mtk_eth_soc: various enhancements Daniel Golle 2023-02-07 14:18 ` Daniel Golle 2023-02-07 14:18 ` [PATCH v2 01/11] net: ethernet: mtk_eth_soc: add support for MT7981 SoC Daniel Golle 2023-02-07 14:18 ` Daniel Golle 2023-02-07 14:19 ` [PATCH v2 02/11] dt-bindings: net: mediatek,net: add mt7981-eth binding Daniel Golle 2023-02-07 14:19 ` Daniel Golle 2023-02-07 17:36 ` Krzysztof Kozlowski 2023-02-07 17:36 ` Krzysztof Kozlowski 2023-02-07 14:19 ` [PATCH v2 03/11] dt-bindings: arm: mediatek: add 'mediatek,pn_swap' property Daniel Golle 2023-02-07 14:19 ` Daniel Golle 2023-02-07 17:38 ` Krzysztof Kozlowski 2023-02-07 17:38 ` Krzysztof Kozlowski 2023-02-07 18:00 ` Daniel Golle 2023-02-07 18:00 ` Daniel Golle 2023-02-08 9:32 ` Krzysztof Kozlowski [this message] 2023-02-08 9:32 ` Krzysztof Kozlowski 2023-02-08 13:51 ` Daniel Golle 2023-02-08 13:51 ` Daniel Golle 2023-02-08 20:08 ` Krzysztof Kozlowski 2023-02-08 20:08 ` Krzysztof Kozlowski 2023-02-08 22:30 ` Daniel Golle 2023-02-08 22:30 ` Daniel Golle 2023-02-09 11:30 ` Krzysztof Kozlowski 2023-02-09 11:30 ` Krzysztof Kozlowski 2023-02-10 10:34 ` Russell King (Oracle) 2023-02-10 10:34 ` Russell King (Oracle) 2023-02-10 12:23 ` Russell King (Oracle) 2023-02-10 12:23 ` Russell King (Oracle) 2023-02-10 12:35 ` Krzysztof Kozlowski 2023-02-10 12:35 ` Krzysztof Kozlowski 2023-02-10 10:30 ` Russell King (Oracle) 2023-02-10 10:30 ` Russell King (Oracle) 2023-02-07 14:20 ` [PATCH v2 04/11] net: ethernet: mtk_eth_soc: set MDIO bus clock frequency Daniel Golle 2023-02-07 14:20 ` Daniel Golle 2023-02-07 17:07 ` Andrew Lunn 2023-02-07 17:07 ` Andrew Lunn 2023-02-07 14:21 ` [PATCH v2 05/11] net: ethernet: mtk_eth_soc: reset PCS state Daniel Golle 2023-02-07 14:21 ` Daniel Golle 2023-02-10 10:36 ` Russell King (Oracle) 2023-02-10 10:36 ` Russell King (Oracle) 2023-02-07 14:21 ` [PATCH v2 06/11] net: ethernet: mtk_eth_soc: only write values if needed Daniel Golle 2023-02-07 14:21 ` Daniel Golle 2023-02-10 10:38 ` Russell King (Oracle) 2023-02-10 10:38 ` Russell King (Oracle) 2023-02-07 14:22 ` [PATCH v2 07/11] net: ethernet: mtk_eth_soc: fix RX data corruption issue Daniel Golle 2023-02-07 14:22 ` Daniel Golle 2023-02-07 14:22 ` [PATCH v2 08/11] net: ethernet: mtk_eth_soc: ppe: add support for flow accounting Daniel Golle 2023-02-07 14:22 ` Daniel Golle 2023-02-07 14:23 ` [PATCH v2 09/11] net: pcs: add driver for MediaTek SGMII PCS Daniel Golle 2023-02-07 14:23 ` Daniel Golle 2023-02-10 10:50 ` Russell King (Oracle) 2023-02-10 10:50 ` Russell King (Oracle) 2023-02-07 14:23 ` [PATCH v2 10/11] net: ethernet: mtk_eth_soc: switch to external PCS driver Daniel Golle 2023-02-07 14:23 ` Daniel Golle 2023-02-07 14:30 ` Daniel Golle 2023-02-07 14:30 ` Daniel Golle 2023-02-07 14:24 ` [PATCH v2 11/11] net: dsa: mt7530: use " Daniel Golle 2023-02-07 14:24 ` Daniel Golle 2023-02-10 10:56 ` Russell King (Oracle) 2023-02-10 10:56 ` Russell King (Oracle) 2023-02-10 12:52 ` Daniel Golle 2023-02-10 12:52 ` Daniel Golle 2023-02-10 22:17 ` Daniel Golle 2023-02-10 22:17 ` Daniel Golle
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=b6d782ef-b375-1e73-a384-1ff37c1548a7@kernel.org \ --to=krzk@kernel.org \ --cc=Landen.Chao@mediatek.com \ --cc=Mark-MC.Lee@mediatek.com \ --cc=andrew@lunn.ch \ --cc=angelogioacchino.delregno@collabora.com \ --cc=bjorn@mork.no \ --cc=daniel@makrotopia.org \ --cc=davem@davemloft.net \ --cc=dqfext@gmail.com \ --cc=edumazet@google.com \ --cc=f.fainelli@gmail.com \ --cc=hkallweit1@gmail.com \ --cc=john@phrozen.org \ --cc=kuba@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux@armlinux.org.uk \ --cc=lorenzo@kernel.org \ --cc=matthias.bgg@gmail.com \ --cc=nbd@nbd.name \ --cc=netdev@vger.kernel.org \ --cc=olteanv@gmail.com \ --cc=pabeni@redhat.com \ --cc=sean.wang@mediatek.com \ --cc=zhaojh329@gmail.com \ /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: linkBe 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.