From: Jakub Kicinski <kuba@kernel.org>
To: "Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Paolo Abeni <pabeni@redhat.com>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH net 0/5] ethernet: 3c509: Bring driver back and make some fixes
Date: Thu, 21 May 2026 08:28:27 -0700 [thread overview]
Message-ID: <20260521082827.279e640b@kernel.org> (raw)
In-Reply-To: <alpine.DEB.2.21.2605201115010.1450@angie.orcam.me.uk>
On Wed, 20 May 2026 12:18:44 +0100 (BST) Maciej W. Rozycki wrote:
> As per the previous discussions[1][2] this patch series brings the 3c509
> driver back. Picking up net rather than net-next as I consider it a fix
> to accidental removal and so that any downstream users do not suffer from
> disruption when using released kernels.
>
> In the course of making the coding style changes requested I have come
> across an actual bug in transceiver type selection code, where the old
> setting is not masked out before ORing in the new one, causing no change
> to be actually made in a requested transition from BNC to AUI. I guess
> this code must have been executed exceedingly rarely, as it's always been
> wrong ever since it was added in 2.5.42 back in 2002.
>
> Therefore I find it not worth backporting to stable branches, however for
> the sake of appropriateness, in case someone downstream does want to have
> the fix, I chose to apply it second in the series, right after the actual
> revert and before code clean-ups.
>
> The remaining patches of the series should be obvious; see the respective
> commit descriptions for details.
FWIW sashiko points out a number of potential issues:
https://sashiko.dev/#/patchset/alpine.DEB.2.21.2605201115010.1450%40angie.orcam.me.uk
next prev parent reply other threads:[~2026-05-21 15:28 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-20 11:18 [PATCH net 0/5] ethernet: 3c509: Bring driver back and make some fixes Maciej W. Rozycki
2026-05-20 11:18 ` [PATCH net 1/5] Revert "drivers: net: 3com: 3c509: Remove this driver" Maciej W. Rozycki
2026-05-20 11:18 ` [PATCH net 2/5] ethernet: 3c509: Fix AUI transceiver type selection Maciej W. Rozycki
2026-05-20 11:18 ` [PATCH net 3/5] ethernet: 3c509: Add GPL 2.0 SPDX license identifier Maciej W. Rozycki
2026-05-20 11:19 ` [PATCH net 4/5] ethernet: 3c509: Update documentation to match MAINTAINERS Maciej W. Rozycki
2026-05-20 11:19 ` [PATCH net 5/5] ethernet: 3c509: Fix most coding style issues Maciej W. Rozycki
2026-05-21 12:17 ` [PATCH net 0/5] ethernet: 3c509: Bring driver back and make some fixes Andrew Lunn
2026-05-21 12:45 ` Maciej W. Rozycki
2026-05-21 16:55 ` David Laight
2026-05-21 17:49 ` Maciej W. Rozycki
2026-05-21 21:55 ` David Laight
2026-05-21 15:27 ` Jakub Kicinski
2026-05-21 17:55 ` Maciej W. Rozycki
2026-05-21 15:28 ` Jakub Kicinski [this message]
2026-05-21 18:01 ` Maciej W. Rozycki
2026-05-21 15:50 ` 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=20260521082827.279e640b@kernel.org \
--to=kuba@kernel.org \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=macro@orcam.me.uk \
--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.