From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.coquelin@st.com (Maxime Coquelin) Date: Tue, 13 Jan 2015 15:43:11 +0100 Subject: [PATCH] PCI: xilinx: fix harmless format string warning In-Reply-To: <1759235.dcJCuiggea@wuerfel> References: <1759235.dcJCuiggea@wuerfel> Message-ID: <54B52EFF.3070505@st.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Arnd, On 01/13/2015 03:20 PM, Arnd Bergmann wrote: > The xilinx pcie driver prints a register value whose type is > propagated to the type returned by the GENMASK() macro. > Unfortunately, that type has recently changed as the result > of a bug fix, so now we get a warning about the type: > > drivers/pci/host/pcie-xilinx.c: In function 'xilinx_pcie_clear_err_interrupts': > drivers/pci/host/pcie-xilinx.c:154:3: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] > > This changes the code so we always print the number as an > 'unsigned long' type to avoid the warning. The original code > was fine on 32-bit architectures but not on 64-bit. Now it > works as expected on both. > > Signed-off-by: Arnd Bergmann > Fixes: 00b4d9a1412 ("bitops: Fix shift overflow in GENMASK macros") > > You can add my: Acked-by: Maxime Coquelin Thanks! Maxime