On 11/12/2024 11:27 PM, Roopni Devanathan wrote: ... > + dword_lock = u32_get_bits(stat_buf->info2, > + ATH12K_HTT_DLPAGER_TOTAL_LOCK_PAGES_INFO2); > + dword_unlock = u32_get_bits(stat_buf->info2, > + ATH12K_HTT_DLPAGER_TOTAL_FREE_PAGES_INFO2); drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2615:43: warning: incorrect type in argument 1 (different base types) drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2615:43: expected unsigned int [usertype] v drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2615:43: got restricted __le32 const [usertype] info2 drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2617:45: warning: incorrect type in argument 1 (different base types) drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2617:45: expected unsigned int [usertype] v drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c:2617:45: got restricted __le32 const [usertype] info2 Note that previously Kalle suggested: There's le32_get_bits() so you can simplify this function quite a lot. So s/u32_get_bits/le32_get_bits/ in your new code
