RE: [EXT] Re: [PATCH 2/2] add dts file to enable support for ls1046afrwy board.

[Vabhav Sharma <vabhav.sharma@nxp.com>]

Hi Fabio,

> -----Original Message-----
> From: Fabio Estevam <festevam@gmail.com>
> Sent: Wednesday, May 8, 2019 7:30 PM
> To: Pramod Kumar <pramod.kumar_1@nxp.com>
> Cc: robh+dt@kernel.org; mark.rutland@arm.com; shawnguo@kernel.org;
> manivannan.sadhasivam@linaro.org; Aisheng Dong
> <aisheng.dong@nxp.com>; Michal.Vokac@ysoft.com;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; Vabhav Sharma
> <vabhav.sharma@nxp.com>
> Subject: [EXT] Re: [PATCH 2/2] add dts file to enable support for ls1046afrwy
> board.
> 
> Caution: EXT Email
> 
> Hi Pramod,
> 
> On Wed, May 8, 2019 at 10:56 AM Pramod Kumar
> <pramod.kumar_1@nxp.com> wrote:
> 
> > +&fman0 {
> > +       ethernet@e0000 {
> 
> You have passed @e0000 without a corresponfing reg entry.
> 
> This causes dtc build warnings with W=1.
> 
> Please make sure you don't introduce new W=1 warnings.
ethernet node is included with fsl-ls1046-post.dtsi file and no warning is introduced.
node content can be found in  arch/arm64/boot/dts/freescale/qoriq-fman3-0-1g-0.dtsi:
        ethernet@e0000 {
                cell-index = <0>;
                compatible = "fsl,fman-memac";
                reg = <0xe0000 0x1000>;
                fsl,fman-ports = <&fman0_rx_0x08 &fman0_tx_0x28>;
                ptp-timer = <&ptp_timer0>;
                pcsphy-handle = <&pcsphy0>;
        };

Regards,
Vabhav