[ARM] serial: s3c2410: platform_get_irq() may return signed unnoticed
authorRoel Kluin <12o3l@tiscali.nl>
Wed, 23 Apr 2008 21:59:36 +0000 (23:59 +0200)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Mon, 28 Apr 2008 20:24:45 +0000 (21:24 +0100)
port->irq is unsigned, platform_get_irq() may return signed unnoticed

Signed-off-by: Roel Kluin <12o3l@tiscali.nl>
Acked-by: Alan Cox <alan@redhat.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
drivers/serial/s3c2410.c

index 4ffa2585429a4965d979b67cea2ba870357baef5..da5a02cb4f636605e5a9a2d30f9555a4f66093ed 100644 (file)
@@ -1022,6 +1022,7 @@ static int s3c24xx_serial_init_port(struct s3c24xx_uart_port *ourport,
        struct uart_port *port = &ourport->port;
        struct s3c2410_uartcfg *cfg;
        struct resource *res;
+       int ret;
 
        dbg("s3c24xx_serial_init_port: port=%p, platdev=%p\n", port, platdev);
 
@@ -1064,9 +1065,11 @@ static int s3c24xx_serial_init_port(struct s3c24xx_uart_port *ourport,
 
        port->mapbase   = res->start;
        port->membase   = S3C24XX_VA_UART + (res->start - S3C24XX_PA_UART);
-       port->irq       = platform_get_irq(platdev, 0);
-       if (port->irq < 0)
+       ret = platform_get_irq(platdev, 0);
+       if (ret < 0)
                port->irq = 0;
+       else
+               port->irq = ret;
 
        ourport->clk    = clk_get(&platdev->dev, "uart");