> +     if (!p) {
>               printk(KERN_ERR DRV_NAME ": cannot remap registers\n");
> -             rc = -ENXIO;
> -             goto probe_err_out;
> +             return -ENOMEM;
>       }

Hi Thomas

Another possible cleanup would be replacing printk(KERN_ERR with
dev_err(), or netdev_err() etc.

           Andrew

Reply via email to