| Message ID | 1389707408-24785-1-git-send-email-grygorii.strashko@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, Jan 14, 2014 at 03:50:08PM +0200, Grygorii Strashko wrote: > Use %pa format specifier when printing variables of resource_size_t type > to fix build warnings: > drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 3 has type 'resource_size_t' > drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 4 has type 'resource_size_t' This doesn't apply against current code, it seems the same patch has already been applied.
On Tuesday 14 January 2014 09:39 AM, Mark Brown wrote: > On Tue, Jan 14, 2014 at 03:50:08PM +0200, Grygorii Strashko wrote: >> Use %pa format specifier when printing variables of resource_size_t type >> to fix build warnings: >> drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 3 has type 'resource_size_t' >> drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 4 has type 'resource_size_t' > > This doesn't apply against current code, it seems the same patch has > already been applied. > Yeah. I did send same patch last merge window time-frame which has been applied by you. Regards, Santosh
On 01/14/2014 04:41 PM, Santosh Shilimkar wrote: > On Tuesday 14 January 2014 09:39 AM, Mark Brown wrote: >> On Tue, Jan 14, 2014 at 03:50:08PM +0200, Grygorii Strashko wrote: >>> Use %pa format specifier when printing variables of resource_size_t type >>> to fix build warnings: >>> drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 3 has type 'resource_size_t' >>> drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 4 has type 'resource_size_t' >> >> This doesn't apply against current code, it seems the same patch has >> already been applied. >> > Yeah. I did send same patch last merge window time-frame which has been > applied by you. Ops. Sorry for the noise.
diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c index 50b2d88..d3f6386 100644 --- a/drivers/spi/spi-davinci.c +++ b/drivers/spi/spi-davinci.c @@ -963,8 +963,8 @@ static int davinci_spi_probe(struct platform_device *pdev) goto free_clk; dev_info(&pdev->dev, "DMA: supported\n"); - dev_info(&pdev->dev, "DMA: RX channel: %d, TX channel: %d, " - "event queue: %d\n", dma_rx_chan, dma_tx_chan, + dev_info(&pdev->dev, "DMA: RX channel: %pa, TX channel: %pa, " + "event queue: %d\n", &dma_rx_chan, &dma_tx_chan, pdata->dma_event_q); }
Use %pa format specifier when printing variables of resource_size_t type to fix build warnings: drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 3 has type 'resource_size_t' drivers/spi/spi-davinci.c:966:3: warning: format '%d' expects argument of type 'int', but argument 4 has type 'resource_size_t' Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- drivers/spi/spi-davinci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)