Re: [PATCH net 4/5] can: dev: fix skb drop check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 04.11.22 23:36, Jakub Kicinski wrote:
On Fri, 4 Nov 2022 21:33:16 +0100 Oliver Hartkopp wrote:
On 04.11.22 19:50, Jakub Kicinski wrote:
On Fri,  4 Nov 2022 14:05:34 +0100 Marc Kleine-Budde wrote:
-	if (can_dropped_invalid_skb(ndev, skb))
+	if (can_dev_dropped_skb(dev, skb))

Compiler says "Did you mean ndev"?

Your compiler is a smart buddy! Sorry!

Marc added that single change to my patch for the pch_can.c driver
(which is removed in net-next but not in 6.1-rc).

And in pch_can.c the netdev is named ndev.

Would you like to fix this up on your own or should we send an updated
PR for the series?

Updated PR would be better, if possible.
We don't edit patches locally much (at all?) when applying to netdev.

Ok, thanks!

I have posted another patch which could be added to the updated PR too then.

https://lore.kernel.org/linux-can/20221104142551.16924-1-socketcan@xxxxxxxxxxxx/T/#u

Best regards,
Oliver



[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux