Francesco Magliocca <franciman12@xxxxxxxxx> writes: > Hi, I picked (void*) to be conformant with the other examples in htt_rx.c > For example at line 1431: >> rxd = HTT_RX_BUF_TO_RX_DESC(hw, >> (void *)msdu->data - hw->rx_desc_ops->rx_desc_size); > > But for me it is ok. Maybe we should fix all the occurrences of this kind. Yeah, it would be good to fix the void pointer arithmetic in a separate patch. I have planning to enable -Wpointer-arith in my ath10k-check and ath11k-check scripts, so patches are very welcome. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
