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;
Date:   Tue, 19 Oct 2021 14:29:01 +0200
From:   Andrew Lunn <andrew@lunn.ch>
To:     Jie Luo <quic_luoj@quicinc.com>
Cc:     Luo Jie <luoj@codeaurora.org>, hkallweit1@gmail.com,
        linux@armlinux.org.uk, davem@davemloft.net, kuba@kernel.org,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        sricharan@codeaurora.org
Subject: Re: [PATCH v3 03/13] net: phy: at803x: improve the WOL feature
Message-ID: <YW66DXOIt8GrR2IQ@lunn.ch>
References: <20211018033333.17677-1-luoj@codeaurora.org>
 <20211018033333.17677-4-luoj@codeaurora.org>
 <YW2/wck2NPhgwjuL@lunn.ch>
 <0ba3022d-9879-bf85-251d-3f48b9cff93b@quicinc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <0ba3022d-9879-bf85-251d-3f48b9cff93b@quicinc.com>
Precedence: bulk

> Hi Andrew,
> 
> when this register AT803X_INTR_STATUS bits are cleared after read, we can't
> clear only WOL interrupt here.

O.K. But you do have the value of the interrupt status register. So
you could call phy_trigger_machine(phydev) if there are any other
interrupt pending. They won't get lost that way.

	  Andrew