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

Pekka Enberg
2005-06-22T20:07:08Z

On 6/22/05, Bouchard, Sebastien <Sebastien.Bouchard@ca.kontron.com> 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