All of lore.kernel.org
 help / color / mirror / Atom feed
From: <Thangaraj.S@microchip.com>
To: <andrew@lunn.ch>
Cc: <andrew+netdev@lunn.ch>, <linux-usb@vger.kernel.org>,
	<davem@davemloft.net>, <Rengarajan.S@microchip.com>,
	<kernel-list@raspberrypi.com>, <fiona.klute@gmx.de>,
	<linux-kernel@vger.kernel.org>, <netdev@vger.kernel.org>,
	<kuba@kernel.org>, <edumazet@google.com>, <pabeni@redhat.com>,
	<UNGLinuxDriver@microchip.com>
Subject: Re: [PATCH net v2] net: phy: microchip: force IRQ polling mode for lan88xx
Date: Mon, 21 Apr 2025 04:17:23 +0000	[thread overview]
Message-ID: <151585637e9ada09bbc2bd2e3ab85c4d491bcc5e.camel@microchip.com> (raw)
In-Reply-To: <3c6abdaf-b4e4-440f-9d75-745741287007@lunn.ch>

On Thu, 2025-04-17 at 20:11 +0200, Andrew Lunn wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you
> know the content is safe
> 
> On Thu, Apr 17, 2025 at 02:17:20PM +0000, Thangaraj.S@microchip.com
> wrote:
> > Hi Flona,
> > We haven't observed this issue with our LAN7801 + LAN88xx setup
> > during
> > testing in our setups. Issue reported  specifically in RPI and
> > NVidea
> > with which we have not tested recently. Additionally, community
> > discussions suggest that this issue may have been introduced due to
> > driver updates in the LAN78xx. There's no hardware errata
> > indicating
> > any problems with the interrupts in LAN88xx.
> > 
> > If the issue lies in the interrupt handling within the LAN78xx,
> > it's
> > likely that similar problems could arise with other PHYs that
> > support
> > interrupt handling. This need to be debugged and addressed from
> > LAN78xx
> > driver rather than removing interrupt support in LAN88xx.
> 
> Hi Thangaraj
> 
> It is an easily available platform. Maybe you can get one and try to
> debug it?
> 
> This is a self contained simple fix. From a pragmatic standpoint, it
> solves an issue which is bothering people. If you do find and fix an
> issue in the LAN78xx this change can be easily reverted. So i'm
> learning towards merging it.
> 
>         Andrew

Hi Andrew,
Sure, I will debug this issue from my end. Agree we can merge this fix
for now, and revert once this is fixed in LAN78xx.

Thanks,
Thangaraj Samynathan

  reply	other threads:[~2025-04-21  4:17 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-16 10:24 [PATCH net v2] net: phy: microchip: force IRQ polling mode for lan88xx Fiona Klute
2025-04-16 12:27 ` Andrew Lunn
2025-04-17  9:05 ` Fiona Klute
2025-04-17 14:17   ` Thangaraj.S
2025-04-17 18:11     ` Andrew Lunn
2025-04-21  4:17       ` Thangaraj.S [this message]
2025-04-22 10:51   ` Paolo Abeni
2025-04-22 11:10 ` patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=151585637e9ada09bbc2bd2e3ab85c4d491bcc5e.camel@microchip.com \
    --to=thangaraj.s@microchip.com \
    --cc=Rengarajan.S@microchip.com \
    --cc=UNGLinuxDriver@microchip.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=fiona.klute@gmx.de \
    --cc=kernel-list@raspberrypi.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.