lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 29 Mar 2017 10:57:44 +0800 From: Shawn Guo <shawnguo@...nel.org> To: Leonard Crestez <leonard.crestez@....com> Cc: Dong Aisheng <dongas86@...il.com>, Sascha Hauer <kernel@...gutronix.de>, linux-arm-kernel@...ts.infradead.org, Fabio Estevam <fabio.estevam@....com>, Fugang Duan <fugang.duan@....com>, Octavian Purdila <octavian.purdila@....com>, Florian Fainelli <f.fainelli@...il.com>, linux-kernel@...r.kernel.org Subject: Re: [PATCH] ARM: imx_v6_v7_defconfig: Select SMSC_PHY On Tue, Mar 28, 2017 at 02:16:50PM +0300, Leonard Crestez wrote: > 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? Yes, please, and make sure that checkpatch.pl is happy about the patch. Shawn
Powered by blists - more mailing lists