Re: Patch of a new driver for kernel 2.4.x that need review

On 6/22/05, Bouchard, Sebastien <[email protected]> wrote:
> +static int __init
> +tlclk_init(void)
> +{

[snip]

> +       if (check_region(TLCLK_BASE, 8)) {
> +               printk(KERN_ERR
> +                      "telclock: I/O region already used by another
> driver!\n");
> +               return -EBUSY;
> +       } else {
> +               request_region(TLCLK_BASE, 8, "telclock");

request_region can fail too so you'd better handle that.

                               Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: Patch of a new driver for kernel 2.4.x that need review
    • From: Alan Cox <[email protected]>

• References:
  • Patch of a new driver for kernel 2.4.x that need review
    • From: "Bouchard, Sebastien" <[email protected]>

• Prev by Date: Re: PREEMPT_RT vs I-PIPE: the numbers, part 2
• Next by Date: Re: PREEMPT_RT vs I-PIPE: the numbers, part 2
• Previous by thread: Re: Patch of a new driver for kernel 2.4.x that need review
• Next by thread: Re: Patch of a new driver for kernel 2.4.x that need review
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]