* Paul Walmsley <[EMAIL PROTECTED]> [080507 12:51]:
> 
> omap_serial_init() in arch/arm/mach-omap2/serial.c clears the wrong
> variable if it can't clk_get() a fclk - fix this.  Also fix some
> checkpatch.pl warnings.

I've pushed this but left out the unnecessary braces around both 
clk_enable() lines.

Tony

> 
> Signed-off-by: Paul Walmsley <[EMAIL PROTECTED]>
> ---
> 
>  arch/arm/mach-omap2/serial.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
> 
> 
> diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
> index 54ab7af..bb79f43 100644
> --- a/arch/arm/mach-omap2/serial.c
> +++ b/arch/arm/mach-omap2/serial.c
> @@ -18,7 +18,7 @@
>  #include <linux/serial_reg.h>
>  #include <linux/clk.h>
>  
> -#include <asm/io.h>
> +#include <linux/io.h>
>  
>  #include <asm/arch/common.h>
>  #include <asm/arch/board.h>
> @@ -127,17 +127,17 @@ void __init omap_serial_init(void)
>               if (IS_ERR(uart_ick[i])) {
>                       printk(KERN_ERR "Could not get uart%d_ick\n", i+1);
>                       uart_ick[i] = NULL;
> -             }
> -             else
> +             } else {
>                       clk_enable(uart_ick[i]);
> +             }
>               sprintf(name, "uart%d_fck", i+1);
>               uart_fck[i] = clk_get(NULL, name);
>               if (IS_ERR(uart_fck[i])) {
>                       printk(KERN_ERR "Could not get uart%d_fck\n", i+1);
> -                     uart_ick[i] = NULL;
> -             }
> -             else
> +                     uart_fck[i] = NULL;
> +             } else {
>                       clk_enable(uart_fck[i]);
> +             }
>  
>               omap_serial_reset(p);
>       }
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to [EMAIL PROTECTED]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to