It was <2021-07-20 wto 12:36>, when Jakub Kicinski wrote: > On Mon, 19 Jul 2021 21:28:52 +0200, Łukasz Stelmach wrote: >> + ax_local->stats = >> + devm_netdev_alloc_pcpu_stats(&spi->dev, >> + struct ax88796c_pcpu_stats); >> + if (!ax_local->stats) >> + return -ENOMEM; >> + u64_stats_init(&ax_local->stats->syncp); > > ../drivers/net/ethernet/asix/ax88796c_main.c:971:33: warning: incorrect type in argument 1 (different address spaces) > ../drivers/net/ethernet/asix/ax88796c_main.c:971:33: expected struct u64_stats_sync *syncp > ../drivers/net/ethernet/asix/ax88796c_main.c:971:33: got struct u64_stats_sync [noderef] __percpu * > Fixed, thanks. Apparently separate u64_stats_init() is not necessary as netdev_alloc_pcpu_stats() invokes it for_each_possible_cpu. -- Łukasz Stelmach Samsung R&D Institute Poland Samsung Electronics
Attachment:
signature.asc
Description: PGP signature
