diff mbox

spi: davinci: fix build warning when printing resource_size_t vars

Message ID 1389707408-24785-1-git-send-email-grygorii.strashko@ti.com (mailing list archive)
State Accepted
Commit a4ee96e4886768a177e4dc4935ac676513913581
Headers show

Commit Message

Grygorii Strashko Jan. 14, 2014, 1:50 p.m. UTC
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(-)

Comments

Mark Brown Jan. 14, 2014, 2:39 p.m. UTC | #1
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.
Santosh Shilimkar Jan. 14, 2014, 2:41 p.m. UTC | #2
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
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Grygorii Strashko Jan. 15, 2014, 10:41 a.m. UTC | #3
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.

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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);
 	}