Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230509200032.308934-1-liviu@dudau.co.uk> <a5fdd37e-6f42-2f37-357f-ad5bb082fa7d@arinc9.com>
 <ZFvaxVwVOnzVofrU@bart.dudau.co.uk>
In-Reply-To: <ZFvaxVwVOnzVofrU@bart.dudau.co.uk>
From:   Sergio Paracuellos <sergio.paracuellos@gmail.com>
Date:   Thu, 11 May 2023 06:13:41 +0200
Message-ID: <CAMhs-H8yu9HFte0x53u_Mq8vFzS1ZOKxxR7H=tszfijhtVrU=A@mail.gmail.com>
Subject: Re: [PATCH] mips: dts: ralink: Clarify usage of MT7621 ethernet phy arguments
To:     Liviu Dudau <liviu@dudau.co.uk>
Cc:     =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= <arinc.unal@arinc9.com>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        Paul Burton <paulburton@kernel.org>,
        Rob Herring <robh+dt@kernel.org>, linux-mips@vger.kernel.org,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Liviu,

On Wed, May 10, 2023 at 7:56=E2=80=AFPM Liviu Dudau <liviu@dudau.co.uk> wro=
te:
>
> Hi Ar=C4=B1n=C3=A7,
>
> On Wed, May 10, 2023 at 02:59:44PM +0200, Ar=C4=B1n=C3=A7 =C3=9CNAL wrote=
:
> > On 9.05.2023 22:00, Liviu Dudau wrote:
> > > The device tree uses numbers as arguments to the phys property that a=
re
> > > confusing for newcomers. Define names for the values and use them in =
the
> > > device tree.
> > >
> > > Signed-off-by: Liviu Dudau <liviu@dudau.co.uk>
> >
> > You should document this on
> > instead of
> > doing this. Under the phys property, add 'description:' and explain thi=
s.
>
> There is already some sort of explanation under
> Documentation/devicetree/bindings/phy/mediatek,mt7621-pci-phy.yaml, so I'=
m
> not sure what I'm improving by adding new text in the /pci/ section.
>
> Maybe I haven't explained properly in the commit message, this is meant t=
o
> give a name to the 1 and 0 values used in the device tree, not to clarify
> any perceived missing documentation.

What is that useful for if the bindings are well documented? The
description in the
'Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml' file
for the '#phy-cells'
property is already telling you what that cell is used for. It is
obvious that zero means not dual ported and one means dual ported.
So for me there is no need to add anything extra but in case you want
to clarify anything you should modify binding documentation not the
device tree file at all.

Thanks,
    Sergio Paracuellos
>
> >
> > Ar=C4=B1n=C3=A7
>
> Best regards,
> Liviu
>
> --
> Everyone who uses computers frequently has had, from time to time,
> a mad desire to attack the precocious abacus with an axe.
>                                   -- John D. Clark, Ignition!