serial: max310x: Fix sparse warnings
authorAlexander Shiyan <shc_work@mail.ru>
Sat, 15 Feb 2014 10:59:01 +0000 (14:59 +0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 15 Feb 2014 20:16:21 +0000 (12:16 -0800)
sparse warnings: (new ones prefixed by >>)

>> drivers/tty/serial/max310x.c:906:45: sparse: incorrect type in argument 2 (different address spaces)
 drivers/tty/serial/max310x.c:906:45: expected void const [noderef] <asn:1>*from
 drivers/tty/serial/max310x.c:906:45: got struct serial_rs485 *<noident>
>> drivers/tty/serial/max310x.c:938:35: sparse: incorrect type in argument 1 (different address spaces)
 drivers/tty/serial/max310x.c:938:35: expected void [noderef] <asn:1>*to
 drivers/tty/serial/max310x.c:938:35: got struct serial_rs485 *<noident>

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/max310x.c

index 5836168..471dbc1 100644 (file)
@@ -882,8 +882,7 @@ static int max310x_ioctl(struct uart_port *port, unsigned int cmd,
 
        switch (cmd) {
        case TIOCSRS485:
-               if (copy_from_user(&rs485, (struct serial_rs485 *)arg,
-                                  sizeof(rs485)))
+               if (copy_from_user(&rs485, (void __user *)arg, sizeof(rs485)))
                        return -EFAULT;
                if (rs485.delay_rts_before_send > 0x0f ||
                    rs485.delay_rts_after_send > 0x0f)
@@ -914,8 +913,7 @@ static int max310x_ioctl(struct uart_port *port, unsigned int cmd,
                val = max310x_port_read(port, MAX310X_HDPIXDELAY_REG);
                rs485.delay_rts_before_send = val >> 4;
                rs485.delay_rts_after_send = val & 0x0f;
-               if (copy_to_user((struct serial_rs485 *)arg, &rs485,
-                                sizeof(rs485)))
+               if (copy_to_user((void __user *)arg, &rs485, sizeof(rs485)))
                        return -EFAULT;
                return 0;
        default: