Input: ct82710c - return proper error code for ct82c710_open
If request_irq() fails we should return the proper error instead of -1. Signed-off-by: Davidlohr Bueso <dave@gnu.org> Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
This commit is contained in:
parent
81e78deafb
commit
b0f05aadf1
1 changed files with 5 additions and 3 deletions
|
@ -111,9 +111,11 @@ static void ct82c710_close(struct serio *serio)
|
||||||
static int ct82c710_open(struct serio *serio)
|
static int ct82c710_open(struct serio *serio)
|
||||||
{
|
{
|
||||||
unsigned char status;
|
unsigned char status;
|
||||||
|
int err;
|
||||||
|
|
||||||
if (request_irq(CT82C710_IRQ, ct82c710_interrupt, 0, "ct82c710", NULL))
|
err = request_irq(CT82C710_IRQ, ct82c710_interrupt, 0, "ct82c710", NULL);
|
||||||
return -1;
|
if (err)
|
||||||
|
return err;
|
||||||
|
|
||||||
status = inb_p(CT82C710_STATUS);
|
status = inb_p(CT82C710_STATUS);
|
||||||
|
|
||||||
|
@ -131,7 +133,7 @@ static int ct82c710_open(struct serio *serio)
|
||||||
status &= ~(CT82C710_ENABLE | CT82C710_INTS_ON);
|
status &= ~(CT82C710_ENABLE | CT82C710_INTS_ON);
|
||||||
outb_p(status, CT82C710_STATUS);
|
outb_p(status, CT82C710_STATUS);
|
||||||
free_irq(CT82C710_IRQ, NULL);
|
free_irq(CT82C710_IRQ, NULL);
|
||||||
return -1;
|
return -EBUSY;
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Loading…
Reference in a new issue