[PATCH v5 1/4] wifi: ath12k: Support Downlink Pager Stats

Jeff Johnson quic_jjohnson at quicinc.com
Wed Nov 13 12:45:48 PST 2024


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




More information about the ath12k mailing list