netdev - Re: [PATCHv2 4/4] dt-bindings: net: dsa: add new MT7530 binding to support MT7621

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <91475ef4-2ca1-7341-7b0a-08a8c7e6ee9c@kernel.org>
Date:   Tue, 15 Jan 2019 15:07:33 +1000
From:   Greg Ungerer <gerg@...nel.org>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     sean.wang@...iatek.com, bjorn@...k.no,
        vivien.didelot@...oirfairelinux.com, f.fainelli@...il.com,
        netdev@...r.kernel.org, rene@...rst.com, john@...ozen.org,
        neil@...wn.name
Subject: Re: [PATCHv2 4/4] dt-bindings: net: dsa: add new MT7530 binding to
 support MT7621

Hi Andrew,

On 15/1/19 12:07 am, Andrew Lunn wrote:
> On Mon, Jan 14, 2019 at 05:03:34PM +1000, gerg@...nel.org wrote:
>> From: Greg Ungerer <gerg@...nel.org>
>>
>> Add devicetree binding to support the compatible mt7530 switch as used
>> in the MediaTek MT7621 SoC.
> 
> Hi Gerg
> 
> It gets messy, but could you try to indicate that core-supply and
> io-supply are not required for your new device. Currently the binding
> indicates they are required for all devices.

Yep, good point. Are you thinking something like this?

--- a/Documentation/devicetree/bindings/net/dsa/mt7530.txt
+++ b/Documentation/devicetree/bindings/net/dsa/mt7530.txt
@@ -3,16 +3,19 @@ Mediatek MT7530 Ethernet switch

  Required properties:

-- compatible: Must be compatible = "mediatek,mt7530";
+- compatible: may be compatible = "mediatek,mt7530"
+       or compatible = "mediatek,mt7621"
  - #address-cells: Must be 1.
  - #size-cells: Must be 0.
  - mediatek,mcm: Boolean; if defined, indicates that either MT7530 is the part
         on multi-chip module belong to MT7623A has or the remotely standalone
         chip as the function MT7623N reference board provided for.
  - core-supply: Phandle to the regulator node necessary for the core power.
+       (not required on mediatek,mt7621)
  - io-supply: Phandle to the regulator node necessary for the I/O power.
         See Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
         for details for the regulator setup on these boards.
+       (not required on mediatek,mt7621)

  If the property mediatek,mcm isn't defined, following property is required


Regards
Greg