Bug in i2c-core?

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Fri Feb 27 10:39:49 PST 2015


Hello,

On Fri, Feb 27, 2015 at 09:46:11AM -0800, Dmitry Torokhov wrote:
> On Fri, Feb 27, 2015 at 06:05:45PM +0100, Uwe Kleine-König wrote:
> > On Fri, Feb 27, 2015 at 07:29:30AM -0800, Dmitry Torokhov wrote:
> > > On February 27, 2015 6:37:25 AM PST, Thomas Petazzoni <thomas.petazzoni at free-electrons.com> wrote:
> > > >Dear Sébastien SZYMANSKI,
> > > >
> > > >On Fri, 27 Feb 2015 12:09:51 +0100, Sébastien SZYMANSKI wrote:
> > > >
> > > >>         error = input_register_device(sx8654->input);
> > > >>         if (error)
> > > >>                 return error;
> > > >
> > > >Where is your ->remove() function that unregisters the input device?
> > > 
> > > Everything seems to be allocated with devm so remove is not needed.
> > Everything is devm_* but input_register_device, right?
> 
> Input is allocated with devm_* and input_register_device knows how to
> deal with that:
ah, something new each day. Thanks for teaching me.

Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list