On Wed, Mar 03, 2021 at 09:31:36AM +0100, Marc Kleine-Budde wrote:
> > Looking at the flow in m_can_rx_handler, it looks as though
> > m_can_handle_bus_errors -> m_can_handle_lost_msg already handles message
> > loss properly, and issues a netdev_err. I wonder whether we can remove
> > the warning from m_can_do_rx_poll entirely:
> >
> > diff --git a/drivers/net/can/m_can/m_can.c b/drivers/net/can/m_can/m_can.c
> > index 3752520a7d4b..d783c46cac16 100644
> > --- a/drivers/net/can/m_can/m_can.c
> > +++ b/drivers/net/can/m_can/m_can.c
> > @@ -501,9 +501,6 @@ static int m_can_do_rx_poll(struct net_device *dev, int quota)
> > }
> >
> > while ((rxfs & RXFS_FFL_MASK) && (quota > 0)) {
> > - if (rxfs & RXFS_RFL)
> > - netdev_warn(dev, "Rx FIFO 0 Message Lost\n");
> > -
> > m_can_read_fifo(dev, rxfs);
> >
> > quota--;
> >
>
> Looks good to me. Can you send a proper patch?
Of course, will do shortly.
--
Regards,
Torin Cooper-Bennun
Software Engineer | maxiluxsystems.com
