From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg0-x241.google.com ([2607:f8b0:400e:c05::241]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cgou0-0007VZ-IR for linux-mtd@lists.infradead.org; Thu, 23 Feb 2017 08:37:30 +0000 Received: by mail-pg0-x241.google.com with SMTP id 5so3806643pgj.0 for ; Thu, 23 Feb 2017 00:37:07 -0800 (PST) Date: Thu, 23 Feb 2017 00:37:03 -0800 From: Dmitry Torokhov To: Logan Gunthorpe Cc: Greg Kroah-Hartman , Dan Williams , Alexander Viro , Johannes Thumshirn , Jan Kara , Arnd Bergmann , Sajjan Vikas C , Linus Walleij , Alexandre Courbot , Peter Huewe , Marcel Selhorst , Jarkko Sakkinen , Jason Gunthorpe , Olof Johansson , Doug Ledford , Sean Hefty , Hal Rosenstock , Dmitry Vyukov , Haggai Eran , Parav Pandit , Leon Romanovsky , Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Hans Verkuil , Mauro Carvalho Chehab , Artem Bityutskiy , Richard Weinberger , David Woodhouse , Brian Norris , Boris Brezillon , Marek Vasut , Cyrille Pitchen , Matt Porter , Alexandre Bounine , Andrew Morton , Joe Perches , Lorenzo Stoakes , Vladimir Zapolskiy , Alessandro Zummo , Alexandre Belloni , Boaz Harrosh , Benny Halevy , "James E.J. Bottomley" , "Martin K. Petersen" , Stephen Bates , Bjorn Helgaas , linux-pci@vger.kernel.org, osd-dev@open-osd.org, linux-scsi@vger.kernel.org, rtc-linux@googlegroups.com, linux-mtd@lists.infradead.org, linux-media@vger.kernel.org, linux-iio@vger.kernel.org, linux-rdma@vger.kernel.org, tpmdd-devel@lists.sourceforge.net, linux-gpio@vger.kernel.org, linux-input@vger.kernel.org, linux-nvdimm@lists.01.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 03/14] input: utilize new device_add_cdev helper function Message-ID: <20170223083703.GF18639@dtor-ws> References: <1487653253-11497-1-git-send-email-logang@deltatee.com> <1487653253-11497-4-git-send-email-logang@deltatee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1487653253-11497-4-git-send-email-logang@deltatee.com> List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Feb 20, 2017 at 10:00:42PM -0700, Logan Gunthorpe wrote: > Signed-off-by: Logan Gunthorpe > --- Please add some more verbage, otherwise Acked-by: Dmitry Torokhov > drivers/input/evdev.c | 3 +-- > drivers/input/joydev.c | 3 +-- > drivers/input/mousedev.c | 3 +-- > 3 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c > index e9ae3d5..a3407ea 100644 > --- a/drivers/input/evdev.c > +++ b/drivers/input/evdev.c > @@ -1416,8 +1416,7 @@ static int evdev_connect(struct input_handler *handler, struct input_dev *dev, > goto err_free_evdev; > > cdev_init(&evdev->cdev, &evdev_fops); > - evdev->cdev.kobj.parent = &evdev->dev.kobj; > - error = cdev_add(&evdev->cdev, evdev->dev.devt, 1); > + error = device_add_cdev(&evdev->dev, &evdev->cdev); > if (error) > goto err_unregister_handle; > > diff --git a/drivers/input/joydev.c b/drivers/input/joydev.c > index abd18f3..012e06f 100644 > --- a/drivers/input/joydev.c > +++ b/drivers/input/joydev.c > @@ -905,8 +905,7 @@ static int joydev_connect(struct input_handler *handler, struct input_dev *dev, > goto err_free_joydev; > > cdev_init(&joydev->cdev, &joydev_fops); > - joydev->cdev.kobj.parent = &joydev->dev.kobj; > - error = cdev_add(&joydev->cdev, joydev->dev.devt, 1); > + error = device_add_cdev(&joydev->dev, &joydev->cdev); > if (error) > goto err_unregister_handle; > > diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c > index b604564..efd8666 100644 > --- a/drivers/input/mousedev.c > +++ b/drivers/input/mousedev.c > @@ -901,8 +901,7 @@ static struct mousedev *mousedev_create(struct input_dev *dev, > } > > cdev_init(&mousedev->cdev, &mousedev_fops); > - mousedev->cdev.kobj.parent = &mousedev->dev.kobj; > - error = cdev_add(&mousedev->cdev, mousedev->dev.devt, 1); > + error = device_add_cdev(&mousedev->dev, &mousedev->cdev); > if (error) > goto err_unregister_handle; > > -- > 2.1.4 > -- Dmitry