From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leonard Crestez Subject: Re: [PATCH 1/2] ARM: dts: imx6ul-14x14-evk: Add ksz8081 phy properties Date: Wed, 31 May 2017 01:20:16 +0300 Message-ID: <1496182816.19669.18.camel@nxp.com> References: <46bde649-c9ce-5a87-d4c4-1c1ebc5a8923@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Johan Hovold , Alexandre Belloni , Philipp Zabel , Fabio Estevam , , , To: Florian Fainelli , Shawn Guo , Andrew Lunn , Andy Duan Return-path: In-Reply-To: <46bde649-c9ce-5a87-d4c4-1c1ebc5a8923@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, 2017-05-30 at 11:10 -0700, Florian Fainelli wrote: > On 05/30/2017 10:34 AM, Leonard Crestez wrote: > > Right now mach-imx6ul registers a fixup for the ksz8081 phy. The same > > register values can be set through the micrel phy driver by using dts > > properties. > > > > This seems preferable and allows cleanly fixing suspend/resume. > > > > Signed-off-by: Leonard Crestez > > Should you have a Fixes: tag for that? Sounds like something you'd want > to backport to stable trees as well, no? I don't know if suspend over nfs ever worked on this board in upstream so there is no commit that this patch "Fixes: ". It would make sense to CC stable.