Commit c2d7f743 authored by William Hubbs's avatar William Hubbs Committed by Greg Kroah-Hartman

staging: speakup: kobjects: remove custom locking macros

Signed-off-by: default avatarWilliam Hubbs <w.d.hubbs@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e7b7691f
...@@ -35,7 +35,7 @@ static ssize_t chars_chartab_show(struct kobject *kobj, ...@@ -35,7 +35,7 @@ static ssize_t chars_chartab_show(struct kobject *kobj,
size_t bufsize = PAGE_SIZE; size_t bufsize = PAGE_SIZE;
unsigned long flags; unsigned long flags;
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
*buf_pointer = '\0'; *buf_pointer = '\0';
for (i = 0; i < 256; i++) { for (i = 0; i < 256; i++) {
if (bufsize <= 1) if (bufsize <= 1)
...@@ -70,7 +70,7 @@ static ssize_t chars_chartab_show(struct kobject *kobj, ...@@ -70,7 +70,7 @@ static ssize_t chars_chartab_show(struct kobject *kobj,
bufsize -= len; bufsize -= len;
buf_pointer += len; buf_pointer += len;
} }
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return buf_pointer - buf; return buf_pointer - buf;
} }
...@@ -127,7 +127,7 @@ static ssize_t chars_chartab_store(struct kobject *kobj, ...@@ -127,7 +127,7 @@ static ssize_t chars_chartab_store(struct kobject *kobj,
size_t desc_length = 0; size_t desc_length = 0;
int i; int i;
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
while (cp < end) { while (cp < end) {
while ((cp < end) && (*cp == ' ' || *cp == '\t')) while ((cp < end) && (*cp == ' ' || *cp == '\t'))
...@@ -212,7 +212,7 @@ static ssize_t chars_chartab_store(struct kobject *kobj, ...@@ -212,7 +212,7 @@ static ssize_t chars_chartab_store(struct kobject *kobj,
spk_reset_default_chartab(); spk_reset_default_chartab();
} }
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
report_char_chartab_status(reset, received, used, rejected, report_char_chartab_status(reset, received, used, rejected,
do_characters); do_characters);
return retval; return retval;
...@@ -232,7 +232,7 @@ static ssize_t keymap_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -232,7 +232,7 @@ static ssize_t keymap_show(struct kobject *kobj, struct kobj_attribute *attr,
u_char *cp1; u_char *cp1;
u_char ch; u_char ch;
unsigned long flags; unsigned long flags;
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
cp1 = spk_key_buf + SHIFT_TBL_SIZE; cp1 = spk_key_buf + SHIFT_TBL_SIZE;
num_keys = (int)(*cp1); num_keys = (int)(*cp1);
nstates = (int)cp1[1]; nstates = (int)cp1[1];
...@@ -248,7 +248,7 @@ static ssize_t keymap_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -248,7 +248,7 @@ static ssize_t keymap_show(struct kobject *kobj, struct kobj_attribute *attr,
} }
} }
cp += sprintf(cp, "0, %d\n", KEY_MAP_VER); cp += sprintf(cp, "0, %d\n", KEY_MAP_VER);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return (int)(cp-buf); return (int)(cp-buf);
} }
...@@ -265,17 +265,17 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -265,17 +265,17 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr,
u_char *cp1; u_char *cp1;
unsigned long flags; unsigned long flags;
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
in_buff = kmemdup(buf, count + 1, GFP_ATOMIC); in_buff = kmemdup(buf, count + 1, GFP_ATOMIC);
if (!in_buff) { if (!in_buff) {
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return -ENOMEM; return -ENOMEM;
} }
if (strchr("dDrR", *in_buff)) { if (strchr("dDrR", *in_buff)) {
spk_set_key_info(spk_key_defaults, spk_key_buf); spk_set_key_info(spk_key_defaults, spk_key_buf);
pr_info("keymap set to default values\n"); pr_info("keymap set to default values\n");
kfree(in_buff); kfree(in_buff);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return count; return count;
} }
if (in_buff[count - 1] == '\n') if (in_buff[count - 1] == '\n')
...@@ -294,7 +294,7 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -294,7 +294,7 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr,
pr_warn("i %d %d %d %d\n", i, pr_warn("i %d %d %d %d\n", i,
(int)cp1[-3], (int)cp1[-2], (int)cp1[-1]); (int)cp1[-3], (int)cp1[-2], (int)cp1[-1]);
kfree(in_buff); kfree(in_buff);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return -EINVAL; return -EINVAL;
} }
while (--i >= 0) { while (--i >= 0) {
...@@ -315,7 +315,7 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -315,7 +315,7 @@ static ssize_t keymap_store(struct kobject *kobj, struct kobj_attribute *attr,
} }
} }
kfree(in_buff); kfree(in_buff);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return ret; return ret;
} }
...@@ -341,7 +341,7 @@ static ssize_t silent_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -341,7 +341,7 @@ static ssize_t silent_store(struct kobject *kobj, struct kobj_attribute *attr,
pr_warn("silent value '%c' not in range (0,7)\n", ch); pr_warn("silent value '%c' not in range (0,7)\n", ch);
return -EINVAL; return -EINVAL;
} }
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
if (ch&2) { if (ch&2) {
shut = 1; shut = 1;
spk_do_flush(); spk_do_flush();
...@@ -354,7 +354,7 @@ static ssize_t silent_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -354,7 +354,7 @@ static ssize_t silent_store(struct kobject *kobj, struct kobj_attribute *attr,
spk_shut_up |= shut; spk_shut_up |= shut;
else else
spk_shut_up &= ~shut; spk_shut_up &= ~shut;
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return count; return count;
} }
...@@ -470,7 +470,7 @@ static ssize_t punc_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -470,7 +470,7 @@ static ssize_t punc_show(struct kobject *kobj, struct kobj_attribute *attr,
return -EINVAL; return -EINVAL;
} }
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
pb = (struct st_bits_data *) &spk_punc_info[var->value]; pb = (struct st_bits_data *) &spk_punc_info[var->value];
mask = pb->mask; mask = pb->mask;
for (i = 33; i < 128; i++) { for (i = 33; i < 128; i++) {
...@@ -478,7 +478,7 @@ static ssize_t punc_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -478,7 +478,7 @@ static ssize_t punc_show(struct kobject *kobj, struct kobj_attribute *attr,
continue; continue;
*cp++ = (char)i; *cp++ = (char)i;
} }
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return cp-buf; return cp-buf;
} }
...@@ -518,14 +518,14 @@ static ssize_t punc_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -518,14 +518,14 @@ static ssize_t punc_store(struct kobject *kobj, struct kobj_attribute *attr,
x--; x--;
punc_buf[x] = '\0'; punc_buf[x] = '\0';
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
if (*punc_buf == 'd' || *punc_buf == 'r') if (*punc_buf == 'd' || *punc_buf == 'r')
x = spk_set_mask_bits(0, var->value, 3); x = spk_set_mask_bits(0, var->value, 3);
else else
x = spk_set_mask_bits(punc_buf, var->value, 3); x = spk_set_mask_bits(punc_buf, var->value, 3);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return count; return count;
} }
...@@ -547,7 +547,7 @@ ssize_t spk_var_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -547,7 +547,7 @@ ssize_t spk_var_show(struct kobject *kobj, struct kobj_attribute *attr,
if (param == NULL) if (param == NULL)
return -EINVAL; return -EINVAL;
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
var = (struct var_t *) param->data; var = (struct var_t *) param->data;
switch (param->var_type) { switch (param->var_type) {
case VAR_NUM: case VAR_NUM:
...@@ -580,7 +580,7 @@ ssize_t spk_var_show(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -580,7 +580,7 @@ ssize_t spk_var_show(struct kobject *kobj, struct kobj_attribute *attr,
param->name, param->var_type); param->name, param->var_type);
break; break;
} }
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return rv; return rv;
} }
EXPORT_SYMBOL_GPL(spk_var_show); EXPORT_SYMBOL_GPL(spk_var_show);
...@@ -609,7 +609,7 @@ ssize_t spk_var_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -609,7 +609,7 @@ ssize_t spk_var_store(struct kobject *kobj, struct kobj_attribute *attr,
cp = (char *)buf; cp = (char *)buf;
string_unescape_any_inplace(cp); string_unescape_any_inplace(cp);
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
switch (param->var_type) { switch (param->var_type) {
case VAR_NUM: case VAR_NUM:
case VAR_TIME: case VAR_TIME:
...@@ -670,7 +670,7 @@ ssize_t spk_var_store(struct kobject *kobj, struct kobj_attribute *attr, ...@@ -670,7 +670,7 @@ ssize_t spk_var_store(struct kobject *kobj, struct kobj_attribute *attr,
} }
} }
} }
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
if (ret == -ERESTART) if (ret == -ERESTART)
pr_info("%s reset to default value\n", attr->attr.name); pr_info("%s reset to default value\n", attr->attr.name);
...@@ -818,9 +818,9 @@ static ssize_t message_show(struct kobject *kobj, ...@@ -818,9 +818,9 @@ static ssize_t message_show(struct kobject *kobj,
unsigned long flags; unsigned long flags;
BUG_ON(!group); BUG_ON(!group);
spk_lock(flags); spin_lock_irqsave(&speakup_info.spinlock, flags);
retval = message_show_helper(buf, group->start, group->end); retval = message_show_helper(buf, group->start, group->end);
spk_unlock(flags); spin_unlock_irqrestore(&speakup_info.spinlock, flags);
return retval; return retval;
} }
......
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