[06/10,V2] spi: Add SPI driver for mx233/mx28
Marek Vasut
marex at denx.de
Wed Aug 1 00:48:05 EDT 2012
Dear Shawn Guo,
> On Wed, Aug 01, 2012 at 04:31:04AM +0200, Marek Vasut wrote:
> > > Couple of reasons for asking: No other SPI master driver calls it in
> > > the remove function (unless I missed it), most drivers don't call it
> > > in the probe function error path, and if I call it in the remove
> > > function in a SPI master driver I am working on, and load/unload the
> > > module several times in a row, I get a nasty kernel crash.
> >
> > It seems the spi_master class takes care of that kfree() in
> > spi.c:spi_master_release() . Good catch, thanks!
>
> I do not hardware setup to confirm that right away. When
> spi_master_release will be called exactly? The time that
> spi_master_put gets called? I'm trying to understand if the kfree
> is not needed only in remove function, or both probe and remove.
I checked, it's called in both cases ... (if .probe() crashes, release() is
called, so kfree() is unneeded)
> Regards,
> Shawn
Best regards,
Marek Vasut
More information about the linux-arm-kernel
mailing list