diff options
Diffstat (limited to 'drivers/isdn')
-rw-r--r-- | drivers/isdn/gigaset/asyncdata.c | 3 | ||||
-rw-r--r-- | drivers/isdn/gigaset/common.c | 2 | ||||
-rw-r--r-- | drivers/isdn/gigaset/gigaset.h | 3 | ||||
-rw-r--r-- | drivers/isdn/gigaset/i4l.c | 56 | ||||
-rw-r--r-- | drivers/isdn/gigaset/interface.c | 25 | ||||
-rw-r--r-- | drivers/isdn/gigaset/usb-gigaset.c | 7 |
6 files changed, 53 insertions, 43 deletions
diff --git a/drivers/isdn/gigaset/asyncdata.c b/drivers/isdn/gigaset/asyncdata.c index 091deb9d1c47..c2bd97d29273 100644 --- a/drivers/isdn/gigaset/asyncdata.c +++ b/drivers/isdn/gigaset/asyncdata.c | |||
@@ -575,7 +575,8 @@ int gigaset_m10x_send_skb(struct bc_state *bcs, struct sk_buff *skb) | |||
575 | else | 575 | else |
576 | skb = iraw_encode(skb, HW_HDR_LEN, 0); | 576 | skb = iraw_encode(skb, HW_HDR_LEN, 0); |
577 | if (!skb) { | 577 | if (!skb) { |
578 | err("unable to allocate memory for encoding!\n"); | 578 | dev_err(bcs->cs->dev, |
579 | "unable to allocate memory for encoding!\n"); | ||
579 | return -ENOMEM; | 580 | return -ENOMEM; |
580 | } | 581 | } |
581 | 582 | ||
diff --git a/drivers/isdn/gigaset/common.c b/drivers/isdn/gigaset/common.c index 827c32c16795..9d3ce7718e58 100644 --- a/drivers/isdn/gigaset/common.c +++ b/drivers/isdn/gigaset/common.c | |||
@@ -287,7 +287,7 @@ struct event_t *gigaset_add_event(struct cardstate *cs, | |||
287 | tail = cs->ev_tail; | 287 | tail = cs->ev_tail; |
288 | next = (tail + 1) % MAX_EVENTS; | 288 | next = (tail + 1) % MAX_EVENTS; |
289 | if (unlikely(next == cs->ev_head)) | 289 | if (unlikely(next == cs->ev_head)) |
290 | err("event queue full"); | 290 | dev_err(cs->dev, "event queue full\n"); |
291 | else { | 291 | else { |
292 | event = cs->events + tail; | 292 | event = cs->events + tail; |
293 | event->type = type; | 293 | event->type = type; |
diff --git a/drivers/isdn/gigaset/gigaset.h b/drivers/isdn/gigaset/gigaset.h index f365993161fc..003752954993 100644 --- a/drivers/isdn/gigaset/gigaset.h +++ b/drivers/isdn/gigaset/gigaset.h | |||
@@ -106,7 +106,6 @@ enum debuglevel { | |||
106 | #undef err | 106 | #undef err |
107 | #undef info | 107 | #undef info |
108 | #undef warn | 108 | #undef warn |
109 | #undef notice | ||
110 | 109 | ||
111 | #define err(format, arg...) printk(KERN_ERR KBUILD_MODNAME ": " \ | 110 | #define err(format, arg...) printk(KERN_ERR KBUILD_MODNAME ": " \ |
112 | format "\n" , ## arg) | 111 | format "\n" , ## arg) |
@@ -114,8 +113,6 @@ enum debuglevel { | |||
114 | format "\n" , ## arg) | 113 | format "\n" , ## arg) |
115 | #define warn(format, arg...) printk(KERN_WARNING KBUILD_MODNAME ": " \ | 114 | #define warn(format, arg...) printk(KERN_WARNING KBUILD_MODNAME ": " \ |
116 | format "\n" , ## arg) | 115 | format "\n" , ## arg) |
117 | #define notice(format, arg...) printk(KERN_NOTICE KBUILD_MODNAME ": " \ | ||
118 | format "\n" , ## arg) | ||
119 | 116 | ||
120 | #ifdef CONFIG_GIGASET_DEBUG | 117 | #ifdef CONFIG_GIGASET_DEBUG |
121 | 118 | ||
diff --git a/drivers/isdn/gigaset/i4l.c b/drivers/isdn/gigaset/i4l.c index 9e089f06a942..3c127a8cbaf2 100644 --- a/drivers/isdn/gigaset/i4l.c +++ b/drivers/isdn/gigaset/i4l.c | |||
@@ -46,7 +46,8 @@ static int writebuf_from_LL(int driverID, int channel, int ack, | |||
46 | return -ENODEV; | 46 | return -ENODEV; |
47 | } | 47 | } |
48 | if (channel < 0 || channel >= cs->channels) { | 48 | if (channel < 0 || channel >= cs->channels) { |
49 | err("%s: invalid channel ID (%d)", __func__, channel); | 49 | dev_err(cs->dev, "%s: invalid channel ID (%d)\n", |
50 | __func__, channel); | ||
50 | return -ENODEV; | 51 | return -ENODEV; |
51 | } | 52 | } |
52 | bcs = &cs->bcs[channel]; | 53 | bcs = &cs->bcs[channel]; |
@@ -58,11 +59,13 @@ static int writebuf_from_LL(int driverID, int channel, int ack, | |||
58 | 59 | ||
59 | if (!len) { | 60 | if (!len) { |
60 | if (ack) | 61 | if (ack) |
61 | notice("%s: not ACKing empty packet", __func__); | 62 | dev_notice(cs->dev, "%s: not ACKing empty packet\n", |
63 | __func__); | ||
62 | return 0; | 64 | return 0; |
63 | } | 65 | } |
64 | if (len > MAX_BUF_SIZE) { | 66 | if (len > MAX_BUF_SIZE) { |
65 | err("%s: packet too large (%d bytes)", __func__, len); | 67 | dev_err(cs->dev, "%s: packet too large (%d bytes)\n", |
68 | __func__, len); | ||
66 | return -EINVAL; | 69 | return -EINVAL; |
67 | } | 70 | } |
68 | 71 | ||
@@ -116,8 +119,7 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
116 | gigaset_debugdrivers(); | 119 | gigaset_debugdrivers(); |
117 | 120 | ||
118 | if (!cs) { | 121 | if (!cs) { |
119 | warn("LL tried to access unknown device with nr. %d", | 122 | err("%s: invalid driver ID (%d)", __func__, cntrl->driver); |
120 | cntrl->driver); | ||
121 | return -ENODEV; | 123 | return -ENODEV; |
122 | } | 124 | } |
123 | 125 | ||
@@ -126,7 +128,7 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
126 | gig_dbg(DEBUG_ANY, "ISDN_CMD_IOCTL (driver: %d, arg: %ld)", | 128 | gig_dbg(DEBUG_ANY, "ISDN_CMD_IOCTL (driver: %d, arg: %ld)", |
127 | cntrl->driver, cntrl->arg); | 129 | cntrl->driver, cntrl->arg); |
128 | 130 | ||
129 | warn("ISDN_CMD_IOCTL is not supported."); | 131 | dev_warn(cs->dev, "ISDN_CMD_IOCTL not supported\n"); |
130 | return -EINVAL; | 132 | return -EINVAL; |
131 | 133 | ||
132 | case ISDN_CMD_DIAL: | 134 | case ISDN_CMD_DIAL: |
@@ -138,22 +140,23 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
138 | cntrl->parm.setup.si1, cntrl->parm.setup.si2); | 140 | cntrl->parm.setup.si1, cntrl->parm.setup.si2); |
139 | 141 | ||
140 | if (cntrl->arg >= cs->channels) { | 142 | if (cntrl->arg >= cs->channels) { |
141 | err("ISDN_CMD_DIAL: invalid channel (%d)", | 143 | dev_err(cs->dev, |
142 | (int) cntrl->arg); | 144 | "ISDN_CMD_DIAL: invalid channel (%d)\n", |
145 | (int) cntrl->arg); | ||
143 | return -EINVAL; | 146 | return -EINVAL; |
144 | } | 147 | } |
145 | 148 | ||
146 | bcs = cs->bcs + cntrl->arg; | 149 | bcs = cs->bcs + cntrl->arg; |
147 | 150 | ||
148 | if (!gigaset_get_channel(bcs)) { | 151 | if (!gigaset_get_channel(bcs)) { |
149 | err("ISDN_CMD_DIAL: channel not free"); | 152 | dev_err(cs->dev, "ISDN_CMD_DIAL: channel not free\n"); |
150 | return -EBUSY; | 153 | return -EBUSY; |
151 | } | 154 | } |
152 | 155 | ||
153 | sp = kmalloc(sizeof *sp, GFP_ATOMIC); | 156 | sp = kmalloc(sizeof *sp, GFP_ATOMIC); |
154 | if (!sp) { | 157 | if (!sp) { |
155 | gigaset_free_channel(bcs); | 158 | gigaset_free_channel(bcs); |
156 | err("ISDN_CMD_DIAL: out of memory"); | 159 | dev_err(cs->dev, "ISDN_CMD_DIAL: out of memory\n"); |
157 | return -ENOMEM; | 160 | return -ENOMEM; |
158 | } | 161 | } |
159 | *sp = cntrl->parm.setup; | 162 | *sp = cntrl->parm.setup; |
@@ -173,8 +176,9 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
173 | gig_dbg(DEBUG_ANY, "ISDN_CMD_ACCEPTD"); | 176 | gig_dbg(DEBUG_ANY, "ISDN_CMD_ACCEPTD"); |
174 | 177 | ||
175 | if (cntrl->arg >= cs->channels) { | 178 | if (cntrl->arg >= cs->channels) { |
176 | err("ISDN_CMD_ACCEPTD: invalid channel (%d)", | 179 | dev_err(cs->dev, |
177 | (int) cntrl->arg); | 180 | "ISDN_CMD_ACCEPTD: invalid channel (%d)\n", |
181 | (int) cntrl->arg); | ||
178 | return -EINVAL; | 182 | return -EINVAL; |
179 | } | 183 | } |
180 | 184 | ||
@@ -196,8 +200,9 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
196 | (int) cntrl->arg); | 200 | (int) cntrl->arg); |
197 | 201 | ||
198 | if (cntrl->arg >= cs->channels) { | 202 | if (cntrl->arg >= cs->channels) { |
199 | err("ISDN_CMD_HANGUP: invalid channel (%u)", | 203 | dev_err(cs->dev, |
200 | (unsigned) cntrl->arg); | 204 | "ISDN_CMD_HANGUP: invalid channel (%d)\n", |
205 | (int) cntrl->arg); | ||
201 | return -EINVAL; | 206 | return -EINVAL; |
202 | } | 207 | } |
203 | 208 | ||
@@ -224,8 +229,9 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
224 | cntrl->arg & 0xff, (cntrl->arg >> 8)); | 229 | cntrl->arg & 0xff, (cntrl->arg >> 8)); |
225 | 230 | ||
226 | if ((cntrl->arg & 0xff) >= cs->channels) { | 231 | if ((cntrl->arg & 0xff) >= cs->channels) { |
227 | err("ISDN_CMD_SETL2: invalid channel (%u)", | 232 | dev_err(cs->dev, |
228 | (unsigned) cntrl->arg & 0xff); | 233 | "ISDN_CMD_SETL2: invalid channel (%d)\n", |
234 | (int) cntrl->arg & 0xff); | ||
229 | return -EINVAL; | 235 | return -EINVAL; |
230 | } | 236 | } |
231 | 237 | ||
@@ -244,14 +250,16 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
244 | cntrl->arg & 0xff, (cntrl->arg >> 8)); | 250 | cntrl->arg & 0xff, (cntrl->arg >> 8)); |
245 | 251 | ||
246 | if ((cntrl->arg & 0xff) >= cs->channels) { | 252 | if ((cntrl->arg & 0xff) >= cs->channels) { |
247 | err("ISDN_CMD_SETL3: invalid channel (%u)", | 253 | dev_err(cs->dev, |
248 | (unsigned) cntrl->arg & 0xff); | 254 | "ISDN_CMD_SETL3: invalid channel (%d)\n", |
255 | (int) cntrl->arg & 0xff); | ||
249 | return -EINVAL; | 256 | return -EINVAL; |
250 | } | 257 | } |
251 | 258 | ||
252 | if (cntrl->arg >> 8 != ISDN_PROTO_L3_TRANS) { | 259 | if (cntrl->arg >> 8 != ISDN_PROTO_L3_TRANS) { |
253 | err("ISDN_CMD_SETL3: invalid protocol %lu", | 260 | dev_err(cs->dev, |
254 | cntrl->arg >> 8); | 261 | "ISDN_CMD_SETL3: invalid protocol %lu\n", |
262 | cntrl->arg >> 8); | ||
255 | return -EINVAL; | 263 | return -EINVAL; |
256 | } | 264 | } |
257 | 265 | ||
@@ -262,8 +270,9 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
262 | case ISDN_CMD_ALERT: | 270 | case ISDN_CMD_ALERT: |
263 | gig_dbg(DEBUG_ANY, "ISDN_CMD_ALERT"); //FIXME | 271 | gig_dbg(DEBUG_ANY, "ISDN_CMD_ALERT"); //FIXME |
264 | if (cntrl->arg >= cs->channels) { | 272 | if (cntrl->arg >= cs->channels) { |
265 | err("ISDN_CMD_ALERT: invalid channel (%d)", | 273 | dev_err(cs->dev, |
266 | (int) cntrl->arg); | 274 | "ISDN_CMD_ALERT: invalid channel (%d)\n", |
275 | (int) cntrl->arg); | ||
267 | return -EINVAL; | 276 | return -EINVAL; |
268 | } | 277 | } |
269 | //bcs = cs->bcs + cntrl->arg; | 278 | //bcs = cs->bcs + cntrl->arg; |
@@ -295,7 +304,8 @@ static int command_from_LL(isdn_ctrl *cntrl) | |||
295 | gig_dbg(DEBUG_ANY, "ISDN_CMD_GETSIL"); | 304 | gig_dbg(DEBUG_ANY, "ISDN_CMD_GETSIL"); |
296 | break; | 305 | break; |
297 | default: | 306 | default: |
298 | err("unknown command %d from LL", cntrl->command); | 307 | dev_err(cs->dev, "unknown command %d from LL\n", |
308 | cntrl->command); | ||
299 | return -EINVAL; | 309 | return -EINVAL; |
300 | } | 310 | } |
301 | 311 | ||
diff --git a/drivers/isdn/gigaset/interface.c b/drivers/isdn/gigaset/interface.c index af195b07c191..521951a898ec 100644 --- a/drivers/isdn/gigaset/interface.c +++ b/drivers/isdn/gigaset/interface.c | |||
@@ -197,7 +197,7 @@ static void if_close(struct tty_struct *tty, struct file *filp) | |||
197 | mutex_lock(&cs->mutex); | 197 | mutex_lock(&cs->mutex); |
198 | 198 | ||
199 | if (!cs->open_count) | 199 | if (!cs->open_count) |
200 | warn("%s: device not opened", __func__); | 200 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
201 | else { | 201 | else { |
202 | if (!--cs->open_count) { | 202 | if (!--cs->open_count) { |
203 | spin_lock_irqsave(&cs->lock, flags); | 203 | spin_lock_irqsave(&cs->lock, flags); |
@@ -232,7 +232,7 @@ static int if_ioctl(struct tty_struct *tty, struct file *file, | |||
232 | return -ERESTARTSYS; // FIXME -EINTR? | 232 | return -ERESTARTSYS; // FIXME -EINTR? |
233 | 233 | ||
234 | if (!cs->open_count) | 234 | if (!cs->open_count) |
235 | warn("%s: device not opened", __func__); | 235 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
236 | else { | 236 | else { |
237 | retval = 0; | 237 | retval = 0; |
238 | switch (cmd) { | 238 | switch (cmd) { |
@@ -364,9 +364,9 @@ static int if_write(struct tty_struct *tty, const unsigned char *buf, int count) | |||
364 | return -ERESTARTSYS; // FIXME -EINTR? | 364 | return -ERESTARTSYS; // FIXME -EINTR? |
365 | 365 | ||
366 | if (!cs->open_count) | 366 | if (!cs->open_count) |
367 | warn("%s: device not opened", __func__); | 367 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
368 | else if (cs->mstate != MS_LOCKED) { | 368 | else if (cs->mstate != MS_LOCKED) { |
369 | warn("can't write to unlocked device"); | 369 | dev_warn(cs->dev, "can't write to unlocked device\n"); |
370 | retval = -EBUSY; | 370 | retval = -EBUSY; |
371 | } else if (!cs->connected) { | 371 | } else if (!cs->connected) { |
372 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); | 372 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); |
@@ -398,9 +398,9 @@ static int if_write_room(struct tty_struct *tty) | |||
398 | return -ERESTARTSYS; // FIXME -EINTR? | 398 | return -ERESTARTSYS; // FIXME -EINTR? |
399 | 399 | ||
400 | if (!cs->open_count) | 400 | if (!cs->open_count) |
401 | warn("%s: device not opened", __func__); | 401 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
402 | else if (cs->mstate != MS_LOCKED) { | 402 | else if (cs->mstate != MS_LOCKED) { |
403 | warn("can't write to unlocked device"); | 403 | dev_warn(cs->dev, "can't write to unlocked device\n"); |
404 | retval = -EBUSY; | 404 | retval = -EBUSY; |
405 | } else if (!cs->connected) { | 405 | } else if (!cs->connected) { |
406 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); | 406 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); |
@@ -430,9 +430,9 @@ static int if_chars_in_buffer(struct tty_struct *tty) | |||
430 | return -ERESTARTSYS; // FIXME -EINTR? | 430 | return -ERESTARTSYS; // FIXME -EINTR? |
431 | 431 | ||
432 | if (!cs->open_count) | 432 | if (!cs->open_count) |
433 | warn("%s: device not opened", __func__); | 433 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
434 | else if (cs->mstate != MS_LOCKED) { | 434 | else if (cs->mstate != MS_LOCKED) { |
435 | warn("can't write to unlocked device"); | 435 | dev_warn(cs->dev, "can't write to unlocked device\n"); |
436 | retval = -EBUSY; | 436 | retval = -EBUSY; |
437 | } else if (!cs->connected) { | 437 | } else if (!cs->connected) { |
438 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); | 438 | gig_dbg(DEBUG_ANY, "can't write to unplugged device"); |
@@ -460,7 +460,7 @@ static void if_throttle(struct tty_struct *tty) | |||
460 | mutex_lock(&cs->mutex); | 460 | mutex_lock(&cs->mutex); |
461 | 461 | ||
462 | if (!cs->open_count) | 462 | if (!cs->open_count) |
463 | warn("%s: device not opened", __func__); | 463 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
464 | else { | 464 | else { |
465 | //FIXME | 465 | //FIXME |
466 | } | 466 | } |
@@ -483,7 +483,7 @@ static void if_unthrottle(struct tty_struct *tty) | |||
483 | mutex_lock(&cs->mutex); | 483 | mutex_lock(&cs->mutex); |
484 | 484 | ||
485 | if (!cs->open_count) | 485 | if (!cs->open_count) |
486 | warn("%s: device not opened", __func__); | 486 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
487 | else { | 487 | else { |
488 | //FIXME | 488 | //FIXME |
489 | } | 489 | } |
@@ -510,7 +510,7 @@ static void if_set_termios(struct tty_struct *tty, struct ktermios *old) | |||
510 | mutex_lock(&cs->mutex); | 510 | mutex_lock(&cs->mutex); |
511 | 511 | ||
512 | if (!cs->open_count) { | 512 | if (!cs->open_count) { |
513 | warn("%s: device not opened", __func__); | 513 | dev_warn(cs->dev, "%s: device not opened\n", __func__); |
514 | goto out; | 514 | goto out; |
515 | } | 515 | } |
516 | 516 | ||
@@ -623,7 +623,8 @@ void gigaset_if_init(struct cardstate *cs) | |||
623 | if (!IS_ERR(cs->tty_dev)) | 623 | if (!IS_ERR(cs->tty_dev)) |
624 | dev_set_drvdata(cs->tty_dev, cs); | 624 | dev_set_drvdata(cs->tty_dev, cs); |
625 | else { | 625 | else { |
626 | warn("could not register device to the tty subsystem"); | 626 | dev_warn(cs->dev, |
627 | "could not register device to the tty subsystem\n"); | ||
627 | cs->tty_dev = NULL; | 628 | cs->tty_dev = NULL; |
628 | } | 629 | } |
629 | mutex_unlock(&cs->mutex); | 630 | mutex_unlock(&cs->mutex); |
diff --git a/drivers/isdn/gigaset/usb-gigaset.c b/drivers/isdn/gigaset/usb-gigaset.c index 77d20ab0cd4d..4661830a49db 100644 --- a/drivers/isdn/gigaset/usb-gigaset.c +++ b/drivers/isdn/gigaset/usb-gigaset.c | |||
@@ -498,8 +498,9 @@ static int send_cb(struct cardstate *cs, struct cmdbuf_t *cb) | |||
498 | 498 | ||
499 | if (status) { | 499 | if (status) { |
500 | ucs->busy = 0; | 500 | ucs->busy = 0; |
501 | err("could not submit urb (error %d)\n", | 501 | dev_err(cs->dev, |
502 | -status); | 502 | "could not submit urb (error %d)\n", |
503 | -status); | ||
503 | cb->len = 0; /* skip urb => remove cb+wakeup | 504 | cb->len = 0; /* skip urb => remove cb+wakeup |
504 | in next loop cycle */ | 505 | in next loop cycle */ |
505 | } | 506 | } |
@@ -670,7 +671,7 @@ static int write_modem(struct cardstate *cs) | |||
670 | spin_unlock_irqrestore(&cs->lock, flags); | 671 | spin_unlock_irqrestore(&cs->lock, flags); |
671 | 672 | ||
672 | if (ret) { | 673 | if (ret) { |
673 | err("could not submit urb (error %d)\n", -ret); | 674 | dev_err(cs->dev, "could not submit urb (error %d)\n", -ret); |
674 | ucs->busy = 0; | 675 | ucs->busy = 0; |
675 | } | 676 | } |
676 | 677 | ||