Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: kvalo=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 25611C433CA
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Subject: Re: [PATCH] ath11k: qmi: use %pad to format dma_addr_t
From:   Kalle Valo <kvalo@codeaurora.org>
In-Reply-To: <20210221182754.2071863-1-geert@linux-m68k.org>
References: <20210221182754.2071863-1-geert@linux-m68k.org>
To:     Geert Uytterhoeven <geert@linux-m68k.org>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>, ath11k@lists.infradead.org,
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Geert Uytterhoeven <geert@linux-m68k.org>
User-Agent: pwcli/0.1.0-git (https://github.com/kvalo/pwcli/) Python/3.5.2
Message-Id: <20210224063736.CE9B5C43462@smtp.codeaurora.org>
Date:   Wed, 24 Feb 2021 06:37:36 +0000 (UTC)
Precedence: bulk

Geert Uytterhoeven <geert@linux-m68k.org> wrote:

> If CONFIG_ARCH_DMA_ADDR_T_64BIT=n:
> 
>     drivers/net/wireless/ath/ath11k/qmi.c: In function ‘ath11k_qmi_respond_fw_mem_request’:
>     drivers/net/wireless/ath/ath11k/qmi.c:1690:8: warning: format ‘%llx’ expects argument of type ‘long long unsigned int’, but argument 5 has type ‘dma_addr_t’ {aka ‘unsigned int’} [-Wformat=]
>      1690 |        "qmi req mem_seg[%d] 0x%llx %u %u\n", i,
> 	  |        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>      1691 |         ab->qmi.target_mem[i].paddr,
> 	  |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 	  |                              |
> 	  |                              dma_addr_t {aka unsigned int}
>     drivers/net/wireless/ath/ath11k/debug.h:64:30: note: in definition of macro ‘ath11k_dbg’
>        64 |   __ath11k_dbg(ar, dbg_mask, fmt, ##__VA_ARGS__); \
> 	  |                              ^~~
>     drivers/net/wireless/ath/ath11k/qmi.c:1690:34: note: format string is defined here
>      1690 |        "qmi req mem_seg[%d] 0x%llx %u %u\n", i,
> 	  |                               ~~~^
> 	  |                                  |
> 	  |                                  long long unsigned int
> 	  |                               %x
> 
> Fixes: d5395a5486596308 ("ath11k: qmi: add debug message for allocated memory segment addresses and sizes")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

Patch applied to wireless-drivers.git, thanks.

ebb9d34e073d ath11k: qmi: use %pad to format dma_addr_t

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20210221182754.2071863-1-geert@linux-m68k.org/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches