MIME-Version: 1.0
References: <20230319234710.1749455-1-cphealy@gmail.com> <CAOMZO5Cfc2cYEYvZxm4v1teCgZJjbY08_gWWDxmp4YDCmbewhw@mail.gmail.com>
In-Reply-To: <CAOMZO5Cfc2cYEYvZxm4v1teCgZJjbY08_gWWDxmp4YDCmbewhw@mail.gmail.com>
From:   Chris Healy <cphealy@gmail.com>
Date:   Mon, 20 Mar 2023 07:00:58 -0700
Message-ID: <CAFXsbZpHOx6D_tQJ09u2gNd1a_nQLaf32F2y=gSTTwqtnHyhTQ@mail.gmail.com>
Subject: Re: [PATCH 1/1] ARM: imx_v6_v7_defconfig: Enable SMSC Ethernet PHY
To:     Fabio Estevam <festevam@gmail.com>
Cc:     linux@armlinux.org.uk, shawnguo@kernel.org, s.hauer@pengutronix.de,
        kernel@pengutronix.de, linux-imx@nxp.com, arnd@arndb.de,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        andrew@lunn.ch, Chris Healy <healych@amazon.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Sun, Mar 19, 2023 at 8:07=E2=80=AFPM Fabio Estevam <festevam@gmail.com> =
wrote:
>
> Hi Chris,
>
> On Sun, Mar 19, 2023 at 8:47=E2=80=AFPM Chris Healy <cphealy@gmail.com> w=
rote:
> >
> > From: Chris Healy <healych@amazon.com>
> >
> > The imx53 base imx53qsb uses an SMSC Ethernet PHY.  Enable it's driver.
> >
> > Signed-off-by: Chris Healy <healych@amazon.com>
> > ---
> >  arch/arm/configs/imx_v6_v7_defconfig | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/im=
x_v6_v7_defconfig
> > index 6dc6fed12af8..a12d45e99782 100644
> > --- a/arch/arm/configs/imx_v6_v7_defconfig
> > +++ b/arch/arm/configs/imx_v6_v7_defconfig
> > @@ -135,6 +135,7 @@ CONFIG_SMSC911X=3Dy
> >  # CONFIG_NET_VENDOR_STMICRO is not set
> >  CONFIG_MICREL_PHY=3Dy
> >  CONFIG_AT803X_PHY=3Dy
> > +CONFIG_SMSC_PHY=3Dy
>
> Running 'make imx_v6_v7_defconfig' causes CONFIG_SMSC_PHY=3Dy to be
> selected by default
> because USB_NET_SMSC95XX is already enabled.
>
> It seems that we don't need to explicitly select CONFIG_SMSC_PHY=3Dy as
> done in this patch.

I see what you are saying.  I agree this patch is no longer necessary.  Tnx=
.