Thread (17 messages) 17 messages, 5 authors, 2023-10-18

Re: [PATCH v2 01/10] appletalk: make localtalk and ppp support conditional

From: Jiri Pirko <jiri@resnulli.us>
Date: 2023-10-12 13:46:17
Also in: linux-doc, linux-wireless, lkml

Wed, Oct 11, 2023 at 05:57:38PM CEST, arnd@arndb.de wrote:
On Wed, Oct 11, 2023, at 17:04, Jiri Pirko wrote:
quoted
Could you provide a cover letter for the set please?
Subject: [PATCH v2 00/10] remove final .ndo_do_ioctl references

The .ndo_do_ioctl() netdev operation used to be how one communicates
with a network driver from userspace, but since my previous cleanup [1],
it is purely internal to the kernel.

Removing the cops appletalk/localtalk driver made me revisit the
missing pieces from that older series, removing all the unused
implementations in wireless drivers as well as the two kernel-internal
callers in the ieee802154 and appletalk stacks.

One ethernet driver was already merged in the meantime that should
have used .ndo_eth_ioctl instead of .ndo_do_ioctl, so fix that as well.
With the complete removal, any future drivers making this mistake
cause build failures that are easier to spot.
Looks fine.

[1] https://lore.kernel.org/netdev/20201106221743.3271965-1-arnd@kernel.org/ (local)

----
Hope that helps, I had commented on the cops removal about sending
this but of course not everyone here saw that. Let me know if I should
resend the patches together with the cover letter.
Yes please. Thanks!

   Arnd
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help