Re: [PATCH] net: dsa: sja1105: Remove unneeded cast in sja1105_crc32()

2021-02-25 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Tue, 23 Feb 2021 12:20:03 +0100 you wrote: > sja1105_unpack() takes a "const void *buf" as its first parameter, so > there is no need to cast away the "const" of the "buf" variable before > calling it. > > Drop the cast, as

Re: [PATCH] net: dsa: sja1105: Remove unneeded cast in sja1105_crc32()

2021-02-24 Thread Geert Uytterhoeven
Hi Vladimir, On Wed, Feb 24, 2021 at 11:44 PM Vladimir Oltean wrote: > On Tue, Feb 23, 2021 at 12:20:03PM +0100, Geert Uytterhoeven wrote: > > sja1105_unpack() takes a "const void *buf" as its first parameter, so > > there is no need to cast away the "const" of the "buf" variable before > > calli

Re: [PATCH] net: dsa: sja1105: Remove unneeded cast in sja1105_crc32()

2021-02-24 Thread Vladimir Oltean
On Tue, Feb 23, 2021 at 12:20:03PM +0100, Geert Uytterhoeven wrote: > sja1105_unpack() takes a "const void *buf" as its first parameter, so > there is no need to cast away the "const" of the "buf" variable before > calling it. > > Drop the cast, as it prevents the compiler performing some checks.

[PATCH] net: dsa: sja1105: Remove unneeded cast in sja1105_crc32()

2021-02-23 Thread Geert Uytterhoeven
sja1105_unpack() takes a "const void *buf" as its first parameter, so there is no need to cast away the "const" of the "buf" variable before calling it. Drop the cast, as it prevents the compiler performing some checks. Signed-off-by: Geert Uytterhoeven --- Compile-tested only. BTW, sja1105_pac