driver: usb: Fix pointer conversion warnings for hikey
authorPrabhakar Kushwaha <prabhakar@freescale.com>
Sun, 25 Oct 2015 07:48:41 +0000 (13:18 +0530)
committerTom Rini <trini@konsulko.com>
Thu, 12 Nov 2015 20:59:00 +0000 (15:59 -0500)
Fix below compilation warings happening for hikey_defconfig

drivers/usb/eth/smsc95xx.c:698:56: warning: cast from pointer to integer
of different size [-Wpointer-to-int-cast]
  debug("** %s(), len %d, buf %#x\n", __func__, length, (int)msg);
                                                        ^
include/common.h:109:26: note: in definition of macro ‘debug_cond’
    printf(pr_fmt(fmt), ##args); \
                          ^
drivers/usb/eth/smsc95xx.c:698:2: note: in expansion of macro ‘debug’
  debug("** %s(), len %d, buf %#x\n", __func__, length, (int)msg);
  ^
drivers/usb/eth/smsc95xx.c:718:2: warning: format ‘%u’ expects argument of
type ‘unsigned int’, but argument 2 has type ‘long unsigned int’ [-Wformat=]
  debug("Tx: len = %u, actual = %u, err = %d\n",
  ^
drivers/usb/eth/smsc95xx.c: In function ‘smsc95xx_recv’:
drivers/usb/eth/smsc95xx.c:802:19: warning: cast from pointer to integer
of different size [-Wpointer-to-int-cast]
   cur_buf_align = (int)buf_ptr - (int)recv_buf;
                   ^
drivers/usb/eth/smsc95xx.c:802:34: warning: cast from pointer to integer
of different size [-Wpointer-to-int-cast]
   cur_buf_align = (int)buf_ptr - (int)recv_buf;

Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
drivers/usb/eth/smsc95xx.c

index 202e48285f9c590b422d85ae7850b1c50528fa1f..3099bf4fd4e5d5f8151fd5d65e4b7931c4760f71 100644 (file)
@@ -680,7 +680,8 @@ static int smsc95xx_send_common(struct ueth_data *dev, void *packet, int length)
        ALLOC_CACHE_ALIGN_BUFFER(unsigned char, msg,
                                 PKTSIZE + sizeof(tx_cmd_a) + sizeof(tx_cmd_b));
 
-       debug("** %s(), len %d, buf %#x\n", __func__, length, (int)msg);
+       debug("** %s(), len %d, buf %#x\n", __func__, length,
+             (unsigned int)(ulong)msg);
        if (length > PKTSIZE)
                return -ENOSPC;
 
@@ -701,8 +702,8 @@ static int smsc95xx_send_common(struct ueth_data *dev, void *packet, int length)
                                &actual_len,
                                USB_BULK_SEND_TIMEOUT);
        debug("Tx: len = %u, actual = %u, err = %d\n",
-             length + sizeof(tx_cmd_a) + sizeof(tx_cmd_b),
-             actual_len, err);
+             (unsigned int)(length + sizeof(tx_cmd_a) + sizeof(tx_cmd_b)),
+             (unsigned int)actual_len, err);
 
        return err;
 }
@@ -784,7 +785,7 @@ static int smsc95xx_recv(struct eth_device *eth)
                /* Adjust for next iteration */
                actual_len -= sizeof(packet_len) + packet_len;
                buf_ptr += sizeof(packet_len) + packet_len;
-               cur_buf_align = (int)buf_ptr - (int)recv_buf;
+               cur_buf_align = (ulong)buf_ptr - (ulong)recv_buf;
 
                if (cur_buf_align & 0x03) {
                        int align = 4 - (cur_buf_align & 0x03);