Commit 971ddcf8 authored by Wim Van Sebroeck's avatar Wim Van Sebroeck

[PATCH] nvram - CodingStyle

Bring drivers/char/nvram.c in line with the Coding Style.
Signed-off-by: default avatarWim Van Sebroeck <wim@iguana.be>
parent 3ad4f597
...@@ -46,7 +46,7 @@ ...@@ -46,7 +46,7 @@
/* select machine configuration */ /* select machine configuration */
#if defined(CONFIG_ATARI) #if defined(CONFIG_ATARI)
# define MACH ATARI # define MACH ATARI
#elif defined(__i386__) || defined(__x86_64__) || defined(__arm__) /* and others?? */ #elif defined(__i386__) || defined(__x86_64__) || defined(__arm__) /* and ?? */
# define MACH PC # define MACH PC
#else #else
# error Cannot build nvram driver for this machine configuration. # error Cannot build nvram driver for this machine configuration.
...@@ -107,9 +107,9 @@ ...@@ -107,9 +107,9 @@
#include <linux/init.h> #include <linux/init.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/io.h>
#include <linux/uaccess.h>
#include <asm/io.h>
#include <asm/uaccess.h>
#include <asm/system.h> #include <asm/system.h>
static DEFINE_SPINLOCK(nvram_state_lock); static DEFINE_SPINLOCK(nvram_state_lock);
...@@ -137,14 +137,13 @@ static int mach_proc_infos(unsigned char *contents, char *buffer, int *len, ...@@ -137,14 +137,13 @@ static int mach_proc_infos(unsigned char *contents, char *buffer, int *len,
* know about the RTC cruft. * know about the RTC cruft.
*/ */
unsigned char unsigned char __nvram_read_byte(int i)
__nvram_read_byte(int i)
{ {
return CMOS_READ(NVRAM_FIRST_BYTE + i); return CMOS_READ(NVRAM_FIRST_BYTE + i);
} }
EXPORT_SYMBOL(__nvram_read_byte);
unsigned char unsigned char nvram_read_byte(int i)
nvram_read_byte(int i)
{ {
unsigned long flags; unsigned long flags;
unsigned char c; unsigned char c;
...@@ -154,16 +153,16 @@ nvram_read_byte(int i) ...@@ -154,16 +153,16 @@ nvram_read_byte(int i)
spin_unlock_irqrestore(&rtc_lock, flags); spin_unlock_irqrestore(&rtc_lock, flags);
return c; return c;
} }
EXPORT_SYMBOL(nvram_read_byte);
/* This races nicely with trying to read with checksum checking (nvram_read) */ /* This races nicely with trying to read with checksum checking (nvram_read) */
void void __nvram_write_byte(unsigned char c, int i)
__nvram_write_byte(unsigned char c, int i)
{ {
CMOS_WRITE(c, NVRAM_FIRST_BYTE + i); CMOS_WRITE(c, NVRAM_FIRST_BYTE + i);
} }
EXPORT_SYMBOL(__nvram_write_byte);
void void nvram_write_byte(unsigned char c, int i)
nvram_write_byte(unsigned char c, int i)
{ {
unsigned long flags; unsigned long flags;
...@@ -171,15 +170,15 @@ nvram_write_byte(unsigned char c, int i) ...@@ -171,15 +170,15 @@ nvram_write_byte(unsigned char c, int i)
__nvram_write_byte(c, i); __nvram_write_byte(c, i);
spin_unlock_irqrestore(&rtc_lock, flags); spin_unlock_irqrestore(&rtc_lock, flags);
} }
EXPORT_SYMBOL(nvram_write_byte);
int int __nvram_check_checksum(void)
__nvram_check_checksum(void)
{ {
return mach_check_checksum(); return mach_check_checksum();
} }
EXPORT_SYMBOL(__nvram_check_checksum);
int int nvram_check_checksum(void)
nvram_check_checksum(void)
{ {
unsigned long flags; unsigned long flags;
int rv; int rv;
...@@ -189,16 +188,15 @@ nvram_check_checksum(void) ...@@ -189,16 +188,15 @@ nvram_check_checksum(void)
spin_unlock_irqrestore(&rtc_lock, flags); spin_unlock_irqrestore(&rtc_lock, flags);
return rv; return rv;
} }
EXPORT_SYMBOL(nvram_check_checksum);
static void static void __nvram_set_checksum(void)
__nvram_set_checksum(void)
{ {
mach_set_checksum(); mach_set_checksum();
} }
#if 0 #if 0
void void nvram_set_checksum(void)
nvram_set_checksum(void)
{ {
unsigned long flags; unsigned long flags;
...@@ -212,7 +210,7 @@ nvram_set_checksum(void) ...@@ -212,7 +210,7 @@ nvram_set_checksum(void)
* The are the file operation function for user access to /dev/nvram * The are the file operation function for user access to /dev/nvram
*/ */
static loff_t nvram_llseek(struct file *file,loff_t offset, int origin ) static loff_t nvram_llseek(struct file *file, loff_t offset, int origin)
{ {
lock_kernel(); lock_kernel();
switch (origin) { switch (origin) {
...@@ -230,8 +228,8 @@ static loff_t nvram_llseek(struct file *file,loff_t offset, int origin ) ...@@ -230,8 +228,8 @@ static loff_t nvram_llseek(struct file *file,loff_t offset, int origin )
return (offset >= 0) ? (file->f_pos = offset) : -EINVAL; return (offset >= 0) ? (file->f_pos = offset) : -EINVAL;
} }
static ssize_t static ssize_t nvram_read(struct file *file, char __user *buf,
nvram_read(struct file *file, char __user *buf, size_t count, loff_t *ppos) size_t count, loff_t *ppos)
{ {
unsigned char contents[NVRAM_BYTES]; unsigned char contents[NVRAM_BYTES];
unsigned i = *ppos; unsigned i = *ppos;
...@@ -254,13 +252,13 @@ nvram_read(struct file *file, char __user *buf, size_t count, loff_t *ppos) ...@@ -254,13 +252,13 @@ nvram_read(struct file *file, char __user *buf, size_t count, loff_t *ppos)
return tmp - contents; return tmp - contents;
checksum_err: checksum_err:
spin_unlock_irq(&rtc_lock); spin_unlock_irq(&rtc_lock);
return -EIO; return -EIO;
} }
static ssize_t static ssize_t nvram_write(struct file *file, const char __user *buf,
nvram_write(struct file *file, const char __user *buf, size_t count, loff_t *ppos) size_t count, loff_t *ppos)
{ {
unsigned char contents[NVRAM_BYTES]; unsigned char contents[NVRAM_BYTES];
unsigned i = *ppos; unsigned i = *ppos;
...@@ -287,13 +285,12 @@ nvram_write(struct file *file, const char __user *buf, size_t count, loff_t *ppo ...@@ -287,13 +285,12 @@ nvram_write(struct file *file, const char __user *buf, size_t count, loff_t *ppo
return tmp - contents; return tmp - contents;
checksum_err: checksum_err:
spin_unlock_irq(&rtc_lock); spin_unlock_irq(&rtc_lock);
return -EIO; return -EIO;
} }
static int static int nvram_ioctl(struct inode *inode, struct file *file,
nvram_ioctl(struct inode *inode, struct file *file,
unsigned int cmd, unsigned long arg) unsigned int cmd, unsigned long arg)
{ {
int i; int i;
...@@ -330,8 +327,7 @@ nvram_ioctl(struct inode *inode, struct file *file, ...@@ -330,8 +327,7 @@ nvram_ioctl(struct inode *inode, struct file *file,
} }
} }
static int static int nvram_open(struct inode *inode, struct file *file)
nvram_open(struct inode *inode, struct file *file)
{ {
lock_kernel(); lock_kernel();
spin_lock(&nvram_state_lock); spin_lock(&nvram_state_lock);
...@@ -356,8 +352,7 @@ nvram_open(struct inode *inode, struct file *file) ...@@ -356,8 +352,7 @@ nvram_open(struct inode *inode, struct file *file)
return 0; return 0;
} }
static int static int nvram_release(struct inode *inode, struct file *file)
nvram_release(struct inode *inode, struct file *file)
{ {
spin_lock(&nvram_state_lock); spin_lock(&nvram_state_lock);
...@@ -375,16 +370,14 @@ nvram_release(struct inode *inode, struct file *file) ...@@ -375,16 +370,14 @@ nvram_release(struct inode *inode, struct file *file)
} }
#ifndef CONFIG_PROC_FS #ifndef CONFIG_PROC_FS
static int static int nvram_read_proc(char *buffer, char **start, off_t offset,
nvram_read_proc(char *buffer, char **start, off_t offset,
int size, int *eof, void *data) int size, int *eof, void *data)
{ {
return 0; return 0;
} }
#else #else
static int static int nvram_read_proc(char *buffer, char **start, off_t offset,
nvram_read_proc(char *buffer, char **start, off_t offset,
int size, int *eof, void *data) int size, int *eof, void *data)
{ {
unsigned char contents[NVRAM_BYTES]; unsigned char contents[NVRAM_BYTES];
...@@ -409,14 +402,14 @@ nvram_read_proc(char *buffer, char **start, off_t offset, ...@@ -409,14 +402,14 @@ nvram_read_proc(char *buffer, char **start, off_t offset,
* this like that... */ * this like that... */
#define PRINT_PROC(fmt,args...) \ #define PRINT_PROC(fmt,args...) \
do { \ do { \
*len += sprintf(buffer+*len, fmt, ##args); \ *len += sprintf(buffer + *len, fmt, ##args); \
if (*begin + *len > offset + size) \ if (*begin + *len > offset + size) \
return 0; \ return 0; \
if (*begin + *len < offset) { \ if (*begin + *len < offset) { \
*begin += *len; \ *begin += *len; \
*len = 0; \ *len = 0; \
} \ } \
} while(0) } while (0)
#endif /* CONFIG_PROC_FS */ #endif /* CONFIG_PROC_FS */
...@@ -436,8 +429,7 @@ static struct miscdevice nvram_dev = { ...@@ -436,8 +429,7 @@ static struct miscdevice nvram_dev = {
&nvram_fops &nvram_fops
}; };
static int __init static int __init nvram_init(void)
nvram_init(void)
{ {
int ret; int ret;
...@@ -459,15 +451,14 @@ nvram_init(void) ...@@ -459,15 +451,14 @@ nvram_init(void)
} }
ret = 0; ret = 0;
printk(KERN_INFO "Non-volatile memory driver v" NVRAM_VERSION "\n"); printk(KERN_INFO "Non-volatile memory driver v" NVRAM_VERSION "\n");
out: out:
return ret; return ret;
outmisc: outmisc:
misc_deregister(&nvram_dev); misc_deregister(&nvram_dev);
goto out; goto out;
} }
static void __exit static void __exit nvram_cleanup_module(void)
nvram_cleanup_module(void)
{ {
remove_proc_entry("driver/nvram", NULL); remove_proc_entry("driver/nvram", NULL);
misc_deregister(&nvram_dev); misc_deregister(&nvram_dev);
...@@ -482,8 +473,7 @@ module_exit(nvram_cleanup_module); ...@@ -482,8 +473,7 @@ module_exit(nvram_cleanup_module);
#if MACH == PC #if MACH == PC
static int static int pc_check_checksum(void)
pc_check_checksum(void)
{ {
int i; int i;
unsigned short sum = 0; unsigned short sum = 0;
...@@ -493,11 +483,10 @@ pc_check_checksum(void) ...@@ -493,11 +483,10 @@ pc_check_checksum(void)
sum += __nvram_read_byte(i); sum += __nvram_read_byte(i);
expect = __nvram_read_byte(PC_CKS_LOC)<<8 | expect = __nvram_read_byte(PC_CKS_LOC)<<8 |
__nvram_read_byte(PC_CKS_LOC+1); __nvram_read_byte(PC_CKS_LOC+1);
return ((sum & 0xffff) == expect); return (sum & 0xffff) == expect;
} }
static void static void pc_set_checksum(void)
pc_set_checksum(void)
{ {
int i; int i;
unsigned short sum = 0; unsigned short sum = 0;
...@@ -522,8 +511,7 @@ static char *gfx_types[] = { ...@@ -522,8 +511,7 @@ static char *gfx_types[] = {
"monochrome", "monochrome",
}; };
static int static int pc_proc_infos(unsigned char *nvram, char *buffer, int *len,
pc_proc_infos(unsigned char *nvram, char *buffer, int *len,
off_t *begin, off_t offset, int size) off_t *begin, off_t offset, int size)
{ {
int checksum; int checksum;
...@@ -590,20 +578,18 @@ pc_proc_infos(unsigned char *nvram, char *buffer, int *len, ...@@ -590,20 +578,18 @@ pc_proc_infos(unsigned char *nvram, char *buffer, int *len,
#if MACH == ATARI #if MACH == ATARI
static int static int atari_check_checksum(void)
atari_check_checksum(void)
{ {
int i; int i;
unsigned char sum = 0; unsigned char sum = 0;
for (i = ATARI_CKS_RANGE_START; i <= ATARI_CKS_RANGE_END; ++i) for (i = ATARI_CKS_RANGE_START; i <= ATARI_CKS_RANGE_END; ++i)
sum += __nvram_read_byte(i); sum += __nvram_read_byte(i);
return (__nvram_read_byte(ATARI_CKS_LOC) == (~sum & 0xff) && return (__nvram_read_byte(ATARI_CKS_LOC) == (~sum & 0xff)) &&
__nvram_read_byte(ATARI_CKS_LOC + 1) == (sum & 0xff)); (__nvram_read_byte(ATARI_CKS_LOC + 1) == (sum & 0xff));
} }
static void static void atari_set_checksum(void)
atari_set_checksum(void)
{ {
int i; int i;
unsigned char sum = 0; unsigned char sum = 0;
...@@ -654,8 +640,7 @@ static char *colors[] = { ...@@ -654,8 +640,7 @@ static char *colors[] = {
"2", "4", "16", "256", "65536", "??", "??", "??" "2", "4", "16", "256", "65536", "??", "??", "??"
}; };
static int static int atari_proc_infos(unsigned char *nvram, char *buffer, int *len,
atari_proc_infos(unsigned char *nvram, char *buffer, int *len,
off_t *begin, off_t offset, int size) off_t *begin, off_t offset, int size)
{ {
int checksum = nvram_check_checksum(); int checksum = nvram_check_checksum();
...@@ -725,11 +710,4 @@ atari_proc_infos(unsigned char *nvram, char *buffer, int *len, ...@@ -725,11 +710,4 @@ atari_proc_infos(unsigned char *nvram, char *buffer, int *len,
#endif /* MACH == ATARI */ #endif /* MACH == ATARI */
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
EXPORT_SYMBOL(__nvram_read_byte);
EXPORT_SYMBOL(nvram_read_byte);
EXPORT_SYMBOL(__nvram_write_byte);
EXPORT_SYMBOL(nvram_write_byte);
EXPORT_SYMBOL(__nvram_check_checksum);
EXPORT_SYMBOL(nvram_check_checksum);
MODULE_ALIAS_MISCDEV(NVRAM_MINOR); MODULE_ALIAS_MISCDEV(NVRAM_MINOR);
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