RE: [PATCH net] arm64: dts: imx8dxl: Disable hibernation mode of AR8031 for EQOS

Wei Fang <wei.fang@xxxxxxx> · Fri, 16 Dec 2022 01:06:05 +0000

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Sent: 2022年12月15日 18:13
> To: Wei Fang <wei.fang@xxxxxxx>; robh+dt@xxxxxxxxxx;
> krzysztof.kozlowski+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx;
> s.hauer@xxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx;
> dl-linux-imx <linux-imx@xxxxxxx>; Frank Li <frank.li@xxxxxxx>; Shenwei
> Wang <shenwei.wang@xxxxxxx>; Clark Wang <xiaoning.wang@xxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH net] arm64: dts: imx8dxl: Disable hibernation mode of
> AR8031 for EQOS
> 
> On 15/12/2022 10:36, wei.fang@xxxxxxx wrote:
> > From: Wei Fang <wei.fang@xxxxxxx>
> >
> > The hibernation mode of AR8031 PHY defaults to be enabled after
> > hardware reset. When the cable is unplugged, the PHY will enter
> > hibernation mode after about 10 senconds and the PHY clocks will be
> > stopped to save power. However, due to the design of EQOS, the mac
> > needs the RX_CLK of PHY for software reset to complete. Otherwise the
> > software reset of EQOS will be failed and do not work correctly. The
> > only way is to disable hibernation mode of AR8031 PHY for EQOS, the
> "qca,disable-hibernation-mode"
> > property is used for this purpose and has already been submitted to
> > the upstream, for more details please refer to the below link:
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore
> > .kernel.org%2Fnetdev%2F20220818030054.1010660-2-wei.fang%40nxp.co
> m%2F&
> >
> amp;data=05%7C01%7Cwei.fang%40nxp.com%7C47ca8408b3124cdc6abf08d
> ade84f8
> >
> 1b%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6380669599293
> 37076%7CU
> >
> nknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6
> Ik1ha
> >
> WwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=92M0yZ%2FmcqFr
> U9pGrkbn8Tfo
> > %2BG5MVgjyexiNRKD2nic%3D&amp;reserved=0
> 
> This is not a patch for net-next as you indicate in the subject. DTS patches go
> via SoC platform maintainers.
> 

Okay, I'll remove "net" from the subject.