Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: dRZ3i9yPCPKlUIA5Mot0eXmQylmgiLS2hVU6fjZ02c5iCNLsX1aisq/liSpv43d+CTiZ4J6nKA
 CFTOvmal/juw==
IronPort-SDR: 3SM/1HtmnKjGEbL6xQWfrCcdERiBjAS3tQkv4R3rwpXJ6poechIgJbIZ9F/2KxyJeo2FVSgq7e
 zVcbD6FMxbcg==
From:   "Voon, Weifeng" <weifeng.voon@intel.com>
To:     Andrew Lunn <andrew@lunn.ch>,
        "Ling, Pei Lee" <pei.lee.ling@intel.com>
CC:     Giuseppe Cavallaro <peppe.cavallaro@st.com>,
        Alexandre Torgue <alexandre.torgue@st.com>,
        Jose Abreu <joabreu@synopsys.com>,
        "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        "Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        "Ong, Boon Leong" <boon.leong.ong@intel.com>,
        Wong Vee Khee <vee.khee.wong@linux.intel.com>,
        "Wong, Vee Khee" <vee.khee.wong@intel.com>,
        "Tan, Tee Min" <tee.min.tan@intel.com>,
        "Sit, Michael Wei Hong" <michael.wei.hong.sit@intel.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-stm32@st-md-mailman.stormreply.com" 
        <linux-stm32@st-md-mailman.stormreply.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH net-next V1 3/4] net: stmmac: Reconfigure the PHY WOL
 settings in stmmac_resume()
Thread-Topic: [PATCH net-next V1 3/4] net: stmmac: Reconfigure the PHY WOL
 settings in stmmac_resume()
Thread-Index: AQHXZoJFBtZ0uXgSn0S7qboFjm+Joqseb0UAgALh+bA=
Date:   Wed, 23 Jun 2021 10:06:44 +0000
Message-ID: <CH0PR11MB53806E2DC74B2B9BE8F84D7088089@CH0PR11MB5380.namprd11.prod.outlook.com>
References: <20210621094536.387442-1-pei.lee.ling@intel.com>
 <20210621094536.387442-4-pei.lee.ling@intel.com> <YNCOqGCDgSOy/yTP@lunn.ch>
In-Reply-To: <YNCOqGCDgSOy/yTP@lunn.ch>
Accept-Language: en-US
Content-Language: en-US
dlp-version: 11.5.1.3
dlp-product: dlpe-windows
dlp-reaction: no-action
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?DK+nx7/H13CbYTeNW8Aw4v59kmCustKb6HPd3HCLrETo5j7fK37e8YKbyTq1?=
 =?us-ascii?Q?tNZir77d6uPwpgZXol4UxjV9qrtyQ4rNulVRKzeNMOJPSFfW0BPgYx0tqCJ6?=
 =?us-ascii?Q?x+pI31nRLmbHEnlAqHx/y86qIZeons83OYEc2cYb2ABfqy/rbyfIiN5oMWf1?=
 =?us-ascii?Q?8RLqXVeQ0MGWhvOoyU8+Nu641zRF4KtN8NzOYzK0hD3j0QjLKGggvPiZWxM8?=
 =?us-ascii?Q?e+QXmNKBD0nrb0Bv8Zi7dEh9XgmOmngWFefWaxa/3A6Qf0Ugp3l4lUim6/CD?=
 =?us-ascii?Q?IssuKDtyz9cWqMX4De24Hzr0aIZaIi0ldSZo3S+7V0NFA9HltQWSBc2XlR3Y?=
 =?us-ascii?Q?YZDeUdi2OeOkONqUbv2fHpPJLIAH5xzQ0eEMEY8zIlWeIw3vA4EMMP+isodg?=
 =?us-ascii?Q?Fx2VVuVw67q+XQpNoTachArFMBj6IAWo1Yu4TyU6FMfS/ykB046pEVqKrAvo?=
 =?us-ascii?Q?o1qGdNVMS2B4OiwA4JApDCzGayNy2VjC8PuB8nJht43RT9YNrW0DWBCqUnVC?=
 =?us-ascii?Q?Xire1xCxftma6kRkPsyXc2IFieDLzLqSSsC89hHRiFb/VfYw5x6yNausE2lx?=
 =?us-ascii?Q?m8s8ZprAN2bWBlkY9YNqBLewdxNmSCntwKtApiA5ZcgtkZTrYU0ZnZFqBj2D?=
 =?us-ascii?Q?HG1gDrAzXG7xdLHsnDJs1JGyN40jSMwm1Z3OZQE+H/4esq4OkY2Byz2aOWGx?=
 =?us-ascii?Q?T2fSRbEO2LqqVpMXCvLPSwgX3zxuco6Xu0C0uOFahrOvn8xJK7FRJAST/ItT?=
 =?us-ascii?Q?qnE6LA28/pBKZDN7xeP2zSsTnbqV8ouNMyF9X6/z0k25/6tChwpRwCZgR3i1?=
 =?us-ascii?Q?b3azAO+mjx5xghCWbUIeGcs76YwUqboSOZOBfNZ04iYh33er8kKhPyP4zy48?=
 =?us-ascii?Q?GluVr64GO4wpIfJhix1TakOxoiQDedJUmyNSasYRl7wR8oRt5QRCyNK62dEV?=
 =?us-ascii?Q?qB93UJNP6XbISxoll0pyUhXlcgHpbGSCKFwKtV2RE9ltJlj+9qBjjtoWf1YP?=
 =?us-ascii?Q?mTlCgCy1dGTk81R+718YsMa/g9Hn3jTnkdoKKTOj+Olnmk1XBfTiAkb/Lnpc?=
 =?us-ascii?Q?lpSoR9vBzfgec4pXx192l8A0/0X/NIiRZdrd2O8n/s8153kCLZCgFe+a47NS?=
 =?us-ascii?Q?DbZxThJ0kFAkRhHrUa/dzCrb9Oe7yEHHdJeU5NprXkFsd5IcRyNq66Npe8sc?=
 =?us-ascii?Q?GoPjVpujK6HIJSkmhpjqhS7OzFnGPNQy+osYD9n1qE1gEehAn6l+AdZ1Ft5+?=
 =?us-ascii?Q?fWhsOP6OwfeD9a4QGCLKFGqN2BcqEnytTX+erghTmKFI2Njnl2cLbxD4xqtm?=
 =?us-ascii?Q?UfCmZHXFL+MS86e1QVzkCQqEYhOovtZpkJhuiKbg9mFD62638GTvwplqfjcI?=
 =?us-ascii?Q?ZDOFrlccHidXipiU4Ik/d2vJ+lUM?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH0PR11MB5380.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 038a88e1-dd07-4d24-d8b4-08d9362e9b48
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Jun 2021 10:06:44.5578
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: FXvAdfbUqxg5jYf8z6OpRInGXzeEJ9+CqMJJPc8K88erjSsMGppQtt2X0GU5N6qBYS44GLF1sKiQgAZo0/3Gvg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5315
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> > From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com>
> >
> > After PHY received a magic packet, the PHY WOL event will be triggered
> > then PHY WOL event interrupt will be disarmed.
> > Ethtool settings will remain with WOL enabled after a S3/S4 suspend
> > resume cycle as expected. Hence,the driver should reconfigure the PHY
> > settings to reenable/disable WOL depending on the ethtool WOL settings
> > in the resume flow.
>=20
> Please could you explain this a bit more? I'm wondering if you have a
> PHY driver bug. PHY WOL should remain enabled until it is explicitly
> disabled.
>=20
> 	Andrew

Let's take Marvell 1510 as example.=20

As explained in driver/net/phy/marvell.c
1773 >------->-------/* If WOL event happened once, the LED[2] interrupt pi=
n
1774 >------->------- * will not be cleared unless we reading the interrupt=
 status
1775 >------->------- * register.=20

The WOL event will not able trigger again if the driver does not clear
the interrupt status.
Are we expecting PHY driver will automatically clears the interrupt
status rather than trigger from the MAC driver?

After scanning through all the PHY drivers, the drivers only touches=20
the WOL settings in the get|set_wol() callbacks. Hence, I think that=20
currently there are no PHY drivers that clear the WOL status.
Unless the PHY able to self-clear the WOL event status, the PHY WOL
would not able to remain enabled after resume from S3/S4.
Therefore, we implemented it in the MAC driver to reconfigure the PHY
WOL during the MAC resume() flow.     =20

Weifeng
=20