On 1/19/23 7:39 AM, Yoshihiro Shimoda wrote:
> Since this driver enables the interrupt by RIC2_QFE1, this driver
> should clear the interrupt flag if it happens. Otherwise, the interrupt
> causes to hang the system.
>
> Fixes: a0d2f20650e8 ("Renesas Ethernet AVB PTP clock driver")
No, it's actually c156633f1353 ("Renesas Ethernet AVB driver proper|)!
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
[...]
> diff --git a/drivers/net/ethernet/renesas/ravb_main.c b/drivers/net/ethernet/renesas/ravb_main.c
> index 3f61100c02f4..bcbb62f90fb7 100644
> --- a/drivers/net/ethernet/renesas/ravb_main.c
> +++ b/drivers/net/ethernet/renesas/ravb_main.c
> @@ -1101,7 +1101,7 @@ static void ravb_error_interrupt(struct net_device *ndev)
> ravb_write(ndev, ~(EIS_QFS | EIS_RESERVED), EIS);
> if (eis & EIS_QFS) {
> ris2 = ravb_read(ndev, RIS2);
> - ravb_write(ndev, ~(RIS2_QFF0 | RIS2_RFFF | RIS2_RESERVED),
> + ravb_write(ndev, ~(RIS2_QFF0 | RIS2_QFF1 | RIS2_RFFF | RIS2_RESERVED),
Might as well fix the QFF1 comment indentation below...
[...]
MBR, Sergey
