[Cbe-oss-dev] [patch 1/5] cell: pmi remove support for mutiple devices.

Christian Krafft krafft at de.ibm.com
Sat Jun 23 00:10:39 EST 2007


Hi,

On Thu, 21 Jun 2007 23:07:31 +0900
Akinobu Mita <mita at fixstars.com> wrote:

> >  
> > -	kfree(dev->dev.driver_data);
> > +	kfree(data);
> 
> Shouldn't it set data = NULL here?
> 
> I guess we will get "pmi: driver has already been initialized" message
> and -EBUSY on reloading pmi driver.
> 
> >  
> >  	return 0;
> >  }

That problem did actually not occur.
Anyway, I added the line for completeness and consistency.

But while testing I found another bug that occurred when freeing the irq. The dev_id when requesting the irq was NULL,
but not when freeing the irq. I'm resending the patch as a reply to the first version.

-- 
Mit freundlichen Gruessen,
kind regards,

Christian Krafft
IBM Systems & Technology Group,
Linux Kernel Development
IT Specialist


Vorsitzender des Aufsichtsrats:	Martin Jetter
Geschaeftsfuehrung:		Herbert Kircher
Sitz der Gesellschaft:		Boeblingen
Registriergericht:		Amtsgericht Stuttgart, HRB 243294

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/cbe-oss-dev/attachments/20070622/5b60d2a1/attachment.pgp>


More information about the cbe-oss-dev mailing list