Baochen Qiang <quic_bqiang@xxxxxxxxxxx> wrote: > Base address of CE ring is defined as u32, currently this works > because coherent DMA mask configured as 32 bit: > > #define ATH11K_PCI_COHERENT_DMA_MASK 32 > > However this mask could be changed once firmware bugs are fixed > to fully support 36 bit DMA addressing. So to protect against any > future changes to the DMA mask, change the type of the fields that > are dependent upon it. > > This is found during code review. Compile tested only. > > Fixes: d5c65159f289 ("ath11k: driver for Qualcomm IEEE 802.11ax devices") > Signed-off-by: Baochen Qiang <quic_bqiang@xxxxxxxxxxx> > Acked-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx> > Signed-off-by: Kalle Valo <quic_kvalo@xxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 5714e25f1d18 wifi: ath11k: fix wrong definition of CE ring's base address -- https://patchwork.kernel.org/project/linux-wireless/patch/20240524021558.34452-1-quic_bqiang@xxxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches