Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Mime-Version: 1.0
Subject: Re: [PATCH v2 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy
Reply-To: wangseok.lee@samsung.com
Sender: Wangseok Lee <wangseok.lee@samsung.com>
From:   Wangseok Lee <wangseok.lee@samsung.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Wangseok Lee <wangseok.lee@samsung.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzk+dt@kernel.org" <krzk+dt@kernel.org>,
        "kishon@ti.com" <kishon@ti.com>,
        "vkoul@kernel.org" <vkoul@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "jesper.nilsson@axis.com" <jesper.nilsson@axis.com>,
        "lars.persson@axis.com" <lars.persson@axis.com>
CC:     "bhelgaas@google.com" <bhelgaas@google.com>,
        "linux-phy@lists.infradead.org" <linux-phy@lists.infradead.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "lorenzo.pieralisi@arm.com" <lorenzo.pieralisi@arm.com>,
        "kw@linux.com" <kw@linux.com>,
        "linux-arm-kernel@axis.com" <linux-arm-kernel@axis.com>,
        "kernel@axis.com" <kernel@axis.com>,
        Moon-Ki Jun <moonki.jun@samsung.com>,
        Sang Min Kim <hypmean.kim@samsung.com>,
        Dongjin Yang <dj76.yang@samsung.com>,
        Yeeun Kim <yeeun119.kim@samsung.com>
In-Reply-To: <45aa26be-625e-47eb-c21f-92497e60b6dd@linaro.org>
Message-ID: <20220608033040epcms2p5807e0bf6528ba9f6b24863e9f2244c84@epcms2p5>
Date:   Wed, 08 Jun 2022 12:30:40 +0900
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 102P
DLP-Filter: Pass
References: <45aa26be-625e-47eb-c21f-92497e60b6dd@linaro.org>
        <20220603015431epcms2p6203908cebe6a320854136559a32b54cb@epcms2p6>
        <20220603023158epcms2p6949f6cd93926b9e811e1b4d52fbef91b@epcms2p6>
        <CGME20220603015431epcms2p6203908cebe6a320854136559a32b54cb@epcms2p5>
Precedence: bulk

On=C2=A006/06/2022=C2=A019:16,=C2=A0Krzysztof=C2=A0Kozlowski=C2=A0wrote:=0D=
=0A>=C2=A0On=C2=A003/06/2022=C2=A004:31,=C2=A0Wangseok=C2=A0Lee=C2=A0wrote:=
=0D=0A>>=C2=A0Add=C2=A0description=C2=A0to=C2=A0support=C2=A0Axis,=C2=A0ART=
PEC-8=C2=A0SoC.=0D=0A>>=C2=A0ARTPEC-8=C2=A0is=C2=A0the=C2=A0SoC=C2=A0platfo=
rm=C2=A0of=C2=A0Axis=C2=A0Communications=0D=0A>>=C2=A0and=C2=A0PCIe=C2=A0ph=
y=C2=A0is=C2=A0designed=C2=A0based=C2=A0on=C2=A0SAMSUNG=C2=A0PHY.=0D=0A>=C2=
=A0=0D=0A>=C2=A0This=C2=A0does=C2=A0not=C2=A0look=C2=A0like=C2=A0wrapped=C2=
=A0in=C2=A0Linux=C2=A0commit=C2=A0style.=0D=0A>=C2=A0https://protect2.firee=
ye.com/v1/url?k=3Dc73ae309-a6470b4e-c73b6846-74fe485fff30-d00b7249c2c0a970&=
q=3D1&e=3Db8b33895-af3d-42ce-80ac-4835057078e7&u=3Dhttps%3A%2F%2Felixir.boo=
tlin.com%2Flinux%2Fv5.18-rc4%2Fsource%2FDocumentation%2Fprocess%2Fsubmittin=
g-patches.rst%23L586=0D=0A>=C2=A0=0D=0A=C2=A0=0D=0AOk,=C2=A0i=C2=A0will=C2=
=A0fix=C2=A0it.=0D=0A=C2=A0=0D=0A>>=C2=A0=0D=0A>>=C2=A0changes=C2=A0since=
=C2=A0v1=C2=A0:=0D=0A>>=C2=A0-'make=C2=A0dt_binding_check'=C2=A0result=C2=
=A0improvement=0D=0A>>=C2=A0-Add=C2=A0the=C2=A0missing=C2=A0property=C2=A0l=
ist=0D=0A>>=C2=A0-Align=C2=A0the=C2=A0indentation=C2=A0of=C2=A0continued=C2=
=A0lines/entries=0D=0A>>=C2=A0=0D=0A>>=C2=A0Signed-off-by:=C2=A0Wangseok=C2=
=A0Lee=C2=A0<wangseok.lee=40samsung.com>=0D=0A>>=C2=A0---=0D=0A>>=C2=A0=C2=
=A0.../bindings/phy/axis,artpec8-pcie-phy.yaml=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=7C=C2=A070=C2=A0++++++++++++++++++++++=0D=0A>>=C2=A0=
=C2=A01=C2=A0file=C2=A0changed,=C2=A070=C2=A0insertions(+)=0D=0A>>=C2=A0=C2=
=A0create=C2=A0mode=C2=A0100644=C2=A0Documentation/devicetree/bindings/phy/=
axis,artpec8-pcie-phy.yaml=0D=0A>>=C2=A0=0D=0A>>=C2=A0diff=C2=A0--git=C2=A0=
a/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml=C2=A0b/D=
ocumentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml=0D=0A>>=C2=
=A0new=C2=A0file=C2=A0mode=C2=A0100644=0D=0A>>=C2=A0index=C2=A00000000..ab9=
766f=0D=0A>>=C2=A0---=C2=A0/dev/null=0D=0A>>=C2=A0+++=C2=A0b/Documentation/=
devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml=0D=0A>>=C2=A0=40=40=C2=
=A0-0,0=C2=A0+1,70=C2=A0=40=40=0D=0A>>=C2=A0+=23=C2=A0SPDX-License-Identifi=
er:=C2=A0GPL-2.0-only=C2=A0OR=C2=A0BSD-2-Clause=0D=0A>>=C2=A0+%YAML=C2=A01.=
2=0D=0A>>=C2=A0+---=0D=0A>>=C2=A0+=24id:=C2=A0https://protect2.fireeye.com/=
v1/url?k=3D8784225c-e6f9ca1b-8785a913-74fe485fff30-b4af51b9b3670f43&q=3D1&e=
=3Db8b33895-af3d-42ce-80ac-4835057078e7&u=3Dhttp%3A%2F%2Fdevicetree.org%2Fs=
chemas%2Fphy%2Faxis%2Cartpec8-pcie-phy.yaml%23=0D=0A>>=C2=A0+=24schema:=C2=
=A0https://protect2.fireeye.com/v1/url?k=3Df78efaf3-96f312b4-f78f71bc-74fe4=
85fff30-583950d45c073877&q=3D1&e=3Db8b33895-af3d-42ce-80ac-4835057078e7&u=
=3Dhttp%3A%2F%2Fdevicetree.org%2Fmeta-schemas%2Fcore.yaml%23=0D=0A>>=C2=A0+=
=0D=0A>>=C2=A0+title:=C2=A0ARTPEC-8=C2=A0SoC=C2=A0PCIe=C2=A0PHY=C2=A0Device=
=C2=A0Tree=C2=A0Bindings=0D=0A>=C2=A0=0D=0A>=C2=A0Same=C2=A0comment=C2=A0as=
=C2=A0patch=C2=A0=231.=0D=0A>=C2=A0=0D=0A=C2=A0=0D=0AI=C2=A0will=C2=A0remov=
e=C2=A0'Device=C2=A0Tree=C2=A0Bindings'.=0D=0A=C2=A0=0D=0A>>=C2=A0+=0D=0A>>=
=C2=A0+maintainers:=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0Jesper=C2=A0Nilsson=C2=
=A0<jesper.nilsson=40axis.com>=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+properties:=0D=
=0A>>=C2=A0+=C2=A0=C2=A0compatible:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0c=
onst:=C2=A0axis,artpec8-pcie-phy=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=C2=A0r=
eg:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0items:=0D=0A>>=C2=A0+=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0-=C2=A0description:=C2=A0PHY=C2=A0registers.=0D=0A>=
>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0-=C2=A0description:=C2=A0PHY=C2=
=A0coding=C2=A0sublayer=C2=A0registers.=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=
=C2=A0reg-names:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0items:=0D=0A>>=C2=A0=
+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0-=C2=A0const:=C2=A0phy=0D=0A>>=C2=A0+=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0-=C2=A0const:=C2=A0pcs=0D=0A>>=C2=A0+=
=0D=0A>>=C2=A0+=C2=A0=C2=A0=22=23phy-cells=22:=0D=0A>>=C2=A0+=C2=A0=C2=A0=
=C2=A0=C2=A0const:=C2=A00=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=C2=A0clocks:=
=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0items:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0-=C2=A0description:=C2=A0PCIe=C2=A0PHY=C2=A0reference=
=C2=A0clock=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=C2=A0clock-names:=0D=0A>>=
=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0items:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0-=C2=A0const:=C2=A0ref_clk=0D=0A>=C2=A0=0D=0A>=C2=A0Same=C2=
=A0comment=C2=A0as=C2=A0patch=C2=A0=231.=0D=0A>=C2=A0=0D=0A=C2=A0=0D=0AOk,=
=C2=A0remove=C2=A0_clk.=0D=0A=C2=A0=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=C2=
=A0num-lanes:=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0const:=C2=A02=0D=0A>>=
=C2=A0+=0D=0A>>=C2=A0+required:=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0compatible=
=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0reg=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0reg-=
names=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0=22=23phy-cells=22=0D=0A>>=C2=A0+=C2=
=A0=C2=A0-=C2=A0clocks=0D=0A>>=C2=A0+=C2=A0=C2=A0-=C2=A0clock-names=0D=0A>>=
=C2=A0+=C2=A0=C2=A0-=C2=A0num-lanes=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+additional=
Properties:=C2=A0false=0D=0A>>=C2=A0+=0D=0A>>=C2=A0+examples:=0D=0A>>=C2=A0=
+=C2=A0=C2=A0-=C2=A0=7C=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=23include=C2=
=A0<dt-bindings/interrupt-controller/irq.h>=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=
=A0=C2=A0=23include=C2=A0<dt-bindings/interrupt-controller/arm-gic.h>=0D=0A=
>>=C2=A0+=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0artpec8=C2=A0=7B=0D=0A>=C2=
=A0=0D=0A>=C2=A0Same=C2=A0comment=C2=A0as=C2=A0patch=C2=A0=231.=0D=0A>=C2=
=A0=0D=0A>=C2=A0=0D=0A=C2=A0=0D=0AOk,=C2=A0modity=C2=A0to=C2=A0'soc'.=0D=0A=
=C2=A0=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=23add=
ress-cells=C2=A0=3D=C2=A0<2>;=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=23size-cells=C2=A0=3D=C2=A0<2>;=0D=0A>>=C2=A0+=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0pcie_phy:=C2=A0pcie-phy=4016c80000=
=C2=A0=7B=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0compatible=C2=A0=3D=C2=A0=22axis,artpec8-pcie-phy=22;=
=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0reg=C2=A0=3D=C2=A0<0x0=C2=A00x16c80000=C2=A00x0=C2=A00x2000>,=
=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0<0x0=C2=A00x16c90000=C2=A00=
x0=C2=A00x1000>;=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0reg-names=C2=A0=3D=C2=A0=22phy=22,=C2=A0=22pc=
s=22;=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=23phy-cells=C2=A0=3D=C2=A0<0>;=0D=0A>>=C2=A0+=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clocks=C2=A0=
=3D=C2=A0<&clock_cmu_fsys=C2=A053>;=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clock-names=C2=A0=3D=C2=A0=
=22ref_clk=22;=0D=0A>>=C2=A0+=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0num-lanes=C2=A0=3D=C2=A0<2>;=0D=0A>>=C2=A0+=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=7D;=0D=0A>>=C2=A0+=C2=A0=C2=
=A0=C2=A0=C2=A0=7D;=0D=0A>>=C2=A0+...=0D=0A>=C2=A0=0D=0A>=C2=A0=0D=0A>=C2=
=A0Best=C2=A0regards,=0D=0A>=C2=A0Krzysztof=0D=0A=0D=0AThank=20you=20for=20=
kindness=20reivew.=0D=0A=0D=0ABest=20regards,=0D=0AWangseok=20Lee