On Fri, Mar 27, 2015 at 09:47:16AM +0100, Greg Kroah-Hartman wrote:
> On Fri, Mar 27, 2015 at 02:00:58PM +0530, Sudip Mukherjee wrote:
> >     }
> >     rc = cdev_add(&file_cdev, MKDEV(MAJOR(major_dev), 0), 1);
> >     if (rc  < 0) {
> >             unregister_chrdev_region(major_dev, 1);
> > -           return -1;
> > +           return rc;
> 
> I should have kept reading in the patch series, nice fix up :)
some effect of learning from you and Dan :)

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

Reply via email to