Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII;
 format=flowed
Content-Transfer-Encoding: 7bit
Date:   Sat, 22 Feb 2020 14:19:57 +0100
From:   Michael Walle <michael@walle.cc>
To:     Vladimir Oltean <olteanv@gmail.com>
Cc:     Andrew Lunn <andrew@lunn.ch>,
        "David S. Miller" <davem@davemloft.net>,
        devicetree@vger.kernel.org,
        Florian Fainelli <f.fainelli@gmail.com>,
        lkml <linux-kernel@vger.kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        netdev <netdev@vger.kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Vivien Didelot <vivien.didelot@gmail.com>
Subject: Re: [PATCH v2 net-next/devicetree 4/5] arm64: dts: fsl: ls1028a: add
 node for Felix switch
In-Reply-To: <CA+h21hpCBjo18zHc-SvMj5Y=C+e=rna5MUgp7SW1u0btma+wfg@mail.gmail.com>
References: <20200219151259.14273-5-olteanv@gmail.com>
 <20200222113829.32431-1-michael@walle.cc>
 <CA+h21hpCBjo18zHc-SvMj5Y=C+e=rna5MUgp7SW1u0btma+wfg@mail.gmail.com>
Message-ID: <c02160323fafd2ec621561e7e527de45@walle.cc>
User-Agent: Roundcube Webmail/1.3.10
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Vladimir,

Am 2020-02-22 13:25, schrieb Vladimir Oltean:
> Hi Michael,
> 
> On Sat, 22 Feb 2020 at 13:38, Michael Walle <michael@walle.cc> wrote:
>> 
>> Hi,
>> 
> 
>> > +
>> > +                     enetc_port2: ethernet@0,2 {
>> > +                             compatible = "fsl,enetc";
>> > +                             reg = <0x000200 0 0 0 0>;
>> > +                             phy-mode = "gmii";
>> Can we disable this port by default in this dtsi? As mentioned in the 
>> other
>> mail, I'd prefer to have all ports disabled because it doesn't make 
>> sense
>> to have this port while having all the external ports disabled.
>> 
> 
> Ok. What would you want to happen with the "ethernet" property? Do you
> want the board dts to set that too?

That's something I've also thought about. And now that you've mention
this, I think it makes more sense to have that in the board too. Because
if you have the freedom to use either eno2/swp4 or eno3/swp5, then if I
choose the second one I'd have to delete the ethernet property from the
first, correct? I actually thought about adding the ethernet property
to both; but (1) I don't know if that is even possible (given that one
is always disabled) and (2) if one want to use the second port as an
additional link to the switch you'd have to remove the ethernet property
on that port. correct?


>> > +                                     /* Internal port with DSA tagging */
>> > +                                     mscc_felix_port4: port@4 {
>> > +                                             reg = <4>;
>> > +                                             phy-mode = "internal";
>> > +                                             ethernet = <&enetc_port2>;
>> Likewise, I'd prefer to have this disabled.
>> 
> 
> Ok.
> 
>> > +                     enetc_port3: ethernet@0,6 {
>> > +                             compatible = "fsl,enetc";
>> > +                             reg = <0x000600 0 0 0 0>;
>> > +                             status = "disabled";
>> > +                             phy-mode = "gmii";
>> shouldn't the status be after the phy-mode property?
> 
> Why?

I thought that would be a rule. I just had a quick look on some other 
device
trees before and they all has the status property as the last property 
(before
any subnodes). I might be mistaken. If so, you could do it for 
consistency
reasons ;) all status property in the ls1028a.dtsi are the last ones.

-michael