Commit 8eed9260 authored by Cyrill Gorcunov's avatar Cyrill Gorcunov Committed by Ingo Molnar

x86: coding style cleanup for kernel/bootflag.c

This patch eliminates checkpatch.pl complaints on bootflag.c

No code changed:

   text    data     bss     dec     hex filename
    321       8       0     329     149 bootflag.o.before
    321       8       0     329     149 bootflag.o.after

   md5:
      9c1b474bcf25ddc1724a29c19880043f  bootflag.o.before.asm
      9c1b474bcf25ddc1724a29c19880043f  bootflag.o.after.asm
Signed-off-by: default avatarCyrill Gorcunov <gorcunov@gmail.com>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent ff3cf856
/* /*
* Implement 'Simple Boot Flag Specification 2.0' * Implement 'Simple Boot Flag Specification 2.0'
*/ */
#include <linux/types.h> #include <linux/types.h>
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/init.h> #include <linux/init.h>
...@@ -14,40 +12,38 @@ ...@@ -14,40 +12,38 @@
#include <linux/mc146818rtc.h> #include <linux/mc146818rtc.h>
#define SBF_RESERVED (0x78) #define SBF_RESERVED (0x78)
#define SBF_PNPOS (1<<0) #define SBF_PNPOS (1<<0)
#define SBF_BOOTING (1<<1) #define SBF_BOOTING (1<<1)
#define SBF_DIAG (1<<2) #define SBF_DIAG (1<<2)
#define SBF_PARITY (1<<7) #define SBF_PARITY (1<<7)
int sbf_port __initdata = -1; /* set via acpi_boot_init() */ int sbf_port __initdata = -1; /* set via acpi_boot_init() */
static int __init parity(u8 v) static int __init parity(u8 v)
{ {
int x = 0; int x = 0;
int i; int i;
for(i=0;i<8;i++) for (i = 0; i < 8; i++) {
{ x ^= (v & 1);
x^=(v&1); v >>= 1;
v>>=1;
} }
return x; return x;
} }
static void __init sbf_write(u8 v) static void __init sbf_write(u8 v)
{ {
unsigned long flags; unsigned long flags;
if(sbf_port != -1)
{ if (sbf_port != -1) {
v &= ~SBF_PARITY; v &= ~SBF_PARITY;
if(!parity(v)) if (!parity(v))
v|=SBF_PARITY; v |= SBF_PARITY;
printk(KERN_INFO "Simple Boot Flag at 0x%x set to 0x%x\n", sbf_port, v); printk(KERN_INFO "Simple Boot Flag at 0x%x set to 0x%x\n",
sbf_port, v);
spin_lock_irqsave(&rtc_lock, flags); spin_lock_irqsave(&rtc_lock, flags);
CMOS_WRITE(v, sbf_port); CMOS_WRITE(v, sbf_port);
...@@ -57,33 +53,41 @@ static void __init sbf_write(u8 v) ...@@ -57,33 +53,41 @@ static void __init sbf_write(u8 v)
static u8 __init sbf_read(void) static u8 __init sbf_read(void)
{ {
u8 v;
unsigned long flags; unsigned long flags;
if(sbf_port == -1) u8 v;
if (sbf_port == -1)
return 0; return 0;
spin_lock_irqsave(&rtc_lock, flags); spin_lock_irqsave(&rtc_lock, flags);
v = CMOS_READ(sbf_port); v = CMOS_READ(sbf_port);
spin_unlock_irqrestore(&rtc_lock, flags); spin_unlock_irqrestore(&rtc_lock, flags);
return v; return v;
} }
static int __init sbf_value_valid(u8 v) static int __init sbf_value_valid(u8 v)
{ {
if(v&SBF_RESERVED) /* Reserved bits */ if (v & SBF_RESERVED) /* Reserved bits */
return 0; return 0;
if(!parity(v)) if (!parity(v))
return 0; return 0;
return 1; return 1;
} }
static int __init sbf_init(void) static int __init sbf_init(void)
{ {
u8 v; u8 v;
if(sbf_port == -1)
if (sbf_port == -1)
return 0; return 0;
v = sbf_read(); v = sbf_read();
if(!sbf_value_valid(v)) if (!sbf_value_valid(v)) {
printk(KERN_WARNING "Simple Boot Flag value 0x%x read from CMOS RAM was invalid\n",v); printk(KERN_WARNING "Simple Boot Flag value 0x%x read from "
"CMOS RAM was invalid\n", v);
}
v &= ~SBF_RESERVED; v &= ~SBF_RESERVED;
v &= ~SBF_BOOTING; v &= ~SBF_BOOTING;
...@@ -92,7 +96,7 @@ static int __init sbf_init(void) ...@@ -92,7 +96,7 @@ static int __init sbf_init(void)
v |= SBF_PNPOS; v |= SBF_PNPOS;
#endif #endif
sbf_write(v); sbf_write(v);
return 0; return 0;
} }
module_init(sbf_init); module_init(sbf_init);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment