Commit cd80ec6f authored May 15, 2008 by Roland Dreier

IB/ipath: Fix printk format for ipath_sdma_status



Commit f018c7e1 ("IB/ipath: Change ipath_devdata.ipath_sdma_status to be
unsigned long") changed ipath_sdma_status to be unsigned long, but left
a few debug messages that printed it out with a %016llx format, which
generates the warnings

    drivers/infiniband/hw/ipath/ipath_sdma.c:348: warning: format '%016llx' expects type 'long long unsigned int', but argument  3 has type 'long unsigned int'
    drivers/infiniband/hw/ipath/ipath_sdma.c:618: warning: format '%016llx' expects type 'long long unsigned int', but argument  3 has type 'long unsigned int'

Fix this by changing the format used to print out the value to %08lx
(8 hex digits are now sufficient, because the highest bit used is 31).

Warnings reported by Randy Dunlap <randy.dunlap@oracle.com>.

Signed-off-by: Roland Dreier <rolandd@cisco.com>

parent 7371fd11

drivers/infiniband/hw/ipath/ipath_sdma.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -345,7 +345,7 @@ static void sdma_abort_task(unsigned long opaque)
		* state change
		*/
		if (jiffies > dd->ipath_sdma_abort_jiffies) {
		ipath_dbg("looping with status 0x%016llx\n",
		ipath_dbg("looping with status 0x%08lx\n",
		dd->ipath_sdma_status);
		dd->ipath_sdma_abort_jiffies = jiffies + 5 * HZ;
		}
		@@ -615,7 +615,7 @@ void ipath_restart_sdma(struct ipath_devdata *dd)
		}
		spin_unlock_irqrestore(&dd->ipath_sdma_lock, flags);
		if (!needed) {
		ipath_dbg("invalid attempt to restart SDMA, status 0x%016llx\n",
		ipath_dbg("invalid attempt to restart SDMA, status 0x%08lx\n",
		dd->ipath_sdma_status);
		goto bail;
		}