+ arnd Nathan Chancellor <natechancellor@xxxxxxxxx> writes: >> +int rtw_fw_dump_fifo(struct rtw_dev *rtwdev, u8 fifo_sel, u32 addr, u32 size, >> + u32 *buffer) >> +{ >> + if (!rtwdev->chip->fw_fifo_addr) { > > This causes a clang warning, which points out it is probably not doing > what you think it is: > > drivers/net/wireless/realtek/rtw88/fw.c:1485:21: warning: address of > array 'rtwdev->chip->fw_fifo_addr' will always evaluate to 'true' > [-Wpointer-bool-conversion] > if (!rtwdev->chip->fw_fifo_addr) { > ~~~~~~~~~~~~~~~^~~~~~~~~~~~ > 1 warning generated. > > Was fw_fifo_addr[0] intended or should the check just be deleted? BTW what is the easiest way to install clang for build testing the kernel? For GCC I use crosstool[1] which is awesome as it makes the installation so simple, do we have something similar for clang? Just supporting x86 would be fine, as my use case would be just to reproduce build warnings. [1] https://mirrors.edge.kernel.org/pub/tools/crosstool/ -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches