Message-ID: <1490699810.3546.10.camel@nxp.com>
Subject: Re: [PATCH] ARM: imx_v6_v7_defconfig: Select SMSC_PHY
From: Leonard Crestez <leonard.crestez@nxp.com>
To: Dong Aisheng <dongas86@gmail.com>
CC: Shawn Guo <shawnguo@kernel.org>, Sascha Hauer <kernel@pengutronix.de>,
        <linux-arm-kernel@lists.infradead.org>,
        Fabio Estevam <fabio.estevam@nxp.com>,
        Fugang Duan <fugang.duan@nxp.com>,
        "Octavian Purdila" <octavian.purdila@nxp.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        <linux-kernel@vger.kernel.org>
Date: Tue, 28 Mar 2017 14:16:50 +0300
In-Reply-To: <20170324064318.GD12604@b29396-OptiPlex-7040>
References: <82d0aeeaed87ae5c9390b889cef4c54d715cde61.1490191972.git.leonard.crestez@nxp.com>
         <20170324064318.GD12604@b29396-OptiPlex-7040>
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Mar 2017 11:16:54.4348
 (UTC)
X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR0301MB1957
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1557
Lines: 44

On Fri, 2017-03-24 at 14:43 +0800, Dong Aisheng wrote:
> Hi Leonard,
> 
> On Wed, Mar 22, 2017 at 04:27:37PM +0200, Leonard Crestez wrote:
> > 
> > The imx6sl-evk board has a LAN8720A ethernet phy supported by SMSC_PHY.
> > Add this driver to the default imx config since the device is present on
> > one of the evaluation boards.
> Upstream currently no evaluation board support.
> 
> This may leave to Shawn to decide whether apply it.

Really? But the dts for this eval board is included in upstream. Shouldn't the
default config try to work with the default imx*.dts files included?

> > This used to work mostly fine with the generic phy driver but since
> > commit 0878fff1f42c18e448ab5b8b4f6a3eb32365b5b6 that driver no longer
> I guess you should get a checkpatch error here.

Because I didn't reference that other commit right? Sorry about this.

Should I submit again with an improved commit message?

> > performs a soft reset on startup. This causes netboot to sometimes
> > timeout on DHCP because RX doesn't work right. DHCP is eventually retried
> > and
> Probably another checkpatch warning.

Why, line too long?

> > This was generated with "make savedefconfig" and it includes a few
> > additional minor cleanups.
> > 
> It is a bit strange, after apply your patch, i still get a lot difference
> as follows when savedefconfig:
> 
> I don't know what's wrong. Toolchains difference?

Maybe, kconfig acts strangely sometimes. It's not clear option ordering is
guaranteed to be stable. I don't think it matters though.

--
Regards,
Leonard