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;
From:   "Sriranjani P" <sriranjani.p@samsung.com>
To:     "'Krzysztof Kozlowski'" <krzysztof.kozlowski@linaro.org>,
        <davem@davemloft.net>, <edumazet@google.com>, <kuba@kernel.org>,
        <pabeni@redhat.com>, <robh+dt@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>,
        <alexandre.torgue@foss.st.com>, <peppe.cavallaro@st.com>,
        <joabreu@synopsys.com>, <mcoquelin.stm32@gmail.com>
Cc:     <netdev@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <pankaj.dubey@samsung.com>,
        <alim.akhtar@samsung.com>, <ravi.patel@samsung.com>,
        "'Jayati Sahu'" <jayati.sahu@samsung.com>, <swathi.ks@samsung.com>
In-Reply-To: <7d74a96e-e3a5-118c-04a9-e8ed95fffa54@linaro.org>
Subject: RE: [PATCH v2 4/4] arm64: dts: fsd: Add Ethernet support for PERIC
 Block of FSD SoC
Date:   Mon, 14 Aug 2023 15:57:29 +0530
Message-ID: <000101d9ce99$f454da20$dcfe8e60$@samsung.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Language: en-in
Thread-Index: AQGL0eLTHe4L9QyPOdGPE73PwLZVZAHe9r7UAmSEgRUC2omxnrBMnmnw
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 105P
DLP-Filter: Pass
References: <20230111075422.107173-1-sriranjani.p@samsung.com>
        <CGME20230111075455epcas5p1951d1981f15d252e09281621ef5fbf15@epcas5p1.samsung.com>
        <20230111075422.107173-5-sriranjani.p@samsung.com>
        <7d74a96e-e3a5-118c-04a9-e8ed95fffa54@linaro.org>
Precedence: bulk


> -----Original Message-----
> From: Krzysztof Kozlowski =5Bmailto:krzysztof.kozlowski=40linaro.org=5D
> Sent: 12 January 2023 15:07
> To: Sriranjani P <sriranjani.p=40samsung.com>; davem=40davemloft.net;
> edumazet=40google.com; kuba=40kernel.org; pabeni=40redhat.com;
> robh+dt=40kernel.org; krzysztof.kozlowski+dt=40linaro.org;
> alexandre.torgue=40foss.st.com; peppe.cavallaro=40st.com;
> joabreu=40synopsys.com; mcoquelin.stm32=40gmail.com
> Cc: netdev=40vger.kernel.org; devicetree=40vger.kernel.org; linux-
> kernel=40vger.kernel.org; pankaj.dubey=40samsung.com;
> alim.akhtar=40samsung.com; ravi.patel=40samsung.com; Jayati Sahu
> <jayati.sahu=40samsung.com>
> Subject: Re: =5BPATCH v2 4/4=5D arm64: dts: fsd: Add Ethernet support for=
 PERIC
> Block of FSD SoC
>=20
> On 11/01/2023 08:54, Sriranjani P wrote:
> > The FSD SoC contains two instances of Synopsys DWC QoS Ethernet IP,
> > one in
> > FSYS0 block and other in PERIC block.
> >
> > Adds device tree node for Ethernet in PERIC Block and enables the same
> > for FSD platform.
> >
> > Signed-off-by: Pankaj Dubey <pankaj.dubey=40samsung.com>
> > Signed-off-by: Jayati Sahu <jayati.sahu=40samsung.com>
> > Signed-off-by: Sriranjani P <sriranjani.p=40samsung.com>
> > ---
> >  arch/arm64/boot/dts/tesla/fsd-evb.dts      =7C  9 ++++
> >  arch/arm64/boot/dts/tesla/fsd-pinctrl.dtsi =7C 56
> ++++++++++++++++++++++
> >  arch/arm64/boot/dts/tesla/fsd.dtsi         =7C 29 +++++++++++
> >  3 files changed, 94 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/tesla/fsd-evb.dts
> > b/arch/arm64/boot/dts/tesla/fsd-evb.dts
> > index ca0c1a28d562..2c0cbe775e04 100644
> > --- a/arch/arm64/boot/dts/tesla/fsd-evb.dts
> > +++ b/arch/arm64/boot/dts/tesla/fsd-evb.dts
> > =40=40 -39,6 +39,15 =40=40
> >  	=7D;
> >  =7D;
> >
> > +&ethernet_1 =7B
> > +	status =3D =22okay=22;
> > +
> > +	fixed-link =7B
> > +		speed =3D <1000>;
> > +		full-duplex;
> > +	=7D;
> > +=7D;
> > +
> >  &fin_pll =7B
> >  	clock-frequency =3D <24000000>;
> >  =7D;
> > diff --git a/arch/arm64/boot/dts/tesla/fsd-pinctrl.dtsi
> > b/arch/arm64/boot/dts/tesla/fsd-pinctrl.dtsi
> > index 7ccc0738a149..c955bf159786 100644
> > --- a/arch/arm64/boot/dts/tesla/fsd-pinctrl.dtsi
> > +++ b/arch/arm64/boot/dts/tesla/fsd-pinctrl.dtsi
> > =40=40 -395,6 +395,62 =40=40
> >  		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> >  		samsung,pin-drv =3D <FSD_PIN_DRV_LV1>;
> >  	=7D;
> > +
> > +	eth1_tx_clk: eth1-tx-clk-pins =7B
> > +		samsung,pins =3D =22gpf2-0=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_DOWN>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_tx_data: eth1-tx-data-pins =7B
> > +		samsung,pins =3D =22gpf2-1=22, =22gpf2-2=22, =22gpf2-3=22, =22gpf2-4=
=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_tx_ctrl: eth1-tx-ctrl-pins =7B
> > +		samsung,pins =3D =22gpf2-5=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_phy_intr: eth1-phy-intr-pins =7B
> > +		samsung,pins =3D =22gpf2-6=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV4>;
> > +	=7D;
> > +
> > +	eth1_rx_clk: eth1-rx-clk-pins =7B
> > +		samsung,pins =3D =22gpf3-0=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_rx_data: eth1-rx-data-pins =7B
> > +		samsung,pins =3D =22gpf3-1=22, =22gpf3-2=22, =22gpf3-3=22, =22gpf3-4=
=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_rx_ctrl: eth1-rx-ctrl-pins =7B
> > +		samsung,pins =3D =22gpf3-5=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV6>;
> > +	=7D;
> > +
> > +	eth1_mdio: eth1-mdio-pins =7B
> > +		samsung,pins =3D =22gpf3-6=22, =22gpf3-7=22;
> > +		samsung,pin-function =3D <FSD_PIN_FUNC_2>;
> > +		samsung,pin-pud =3D <FSD_PIN_PULL_UP>;
> > +		samsung,pin-drv =3D <FSD_PIN_DRV_LV4>;
> > +	=7D;
> >  =7D;
> >
> >  &pinctrl_pmu =7B
> > diff --git a/arch/arm64/boot/dts/tesla/fsd.dtsi
> > b/arch/arm64/boot/dts/tesla/fsd.dtsi
> > index ade707cc646b..8807055807dd 100644
> > --- a/arch/arm64/boot/dts/tesla/fsd.dtsi
> > +++ b/arch/arm64/boot/dts/tesla/fsd.dtsi
> > =40=40 -33,6 +33,7 =40=40
> >  		spi1 =3D &spi_1;
> >  		spi2 =3D &spi_2;
> >  		eth0 =3D &ethernet_0;
> > +		eth1 =3D &ethernet_1;
>=20
> Nope for the reasons I explained last time.
Sorry somehow I'm not able to find my previous mail chain.
I had replied to your comment in the previous version of the mail.
In this case alias id is used to differentiate between Ethernet instance 0 =
and 1 in the driver code.
>=20
> >  	=7D;
> >
> >  	cpus =7B
> > =40=40 -882,6 +883,34 =40=40
> >  			phy-mode =3D =22rgmii=22;
> >  			status =3D =22disabled=22;
> >  		=7D;
> > +
> > +		ethernet_1: ethernet=4014300000 =7B
>=20
> Do not add nodes to the end.
If you mean to say that the nodes are supposed to be in the increasing orde=
r of physical address,
I can see that the existing file is not arranged in that format. If that is=
 required I can first fix the
existing file and post the revised patch for the same.=20
>=20
> Best regards,
> Krzysztof