On Sat, Mar 19, 2005 at 04:17:25PM +0100, Sylvain Munaut wrote: > Hi all, > > Found the problem : dma_mask & coherent_dma_mask were not initialized > properly in my struct device. The "old" OCP driver did force them to > default values but adapting it to the new glue model I dropped that.
I was bitten by that before. It would be good to add a check in for NULL dma_mask, to save some time for the next guy, but I haven't looked for the best place to add such a test. > Anyway now it works, sorry for the inconvenience. > > I included a few fixes for that bus glue. The first is a supplementary > safety if the platform data is not set. The second is a typo fix for an > error added in the last change set. > > Thanks for the help, > > Sylvain Thank you Sylvain. Would you add a Signed-of-by line and resubmit? -Dale This patch avoids a null-pointer dereference and fixes a typo. Acked-by: Dale Farnsworth <[EMAIL PROTECTED]> ===== ohci-ppc-soc.c 1.2 vs edited ===== --- 1.2/drivers/usb/host/ohci-ppc-soc.c 2005-03-08 05:43:18 +01:00 +++ edited/ohci-ppc-soc.c 2005-03-19 16:06:23 +01:00 @@ -73,7 +73,7 @@ goto err2; } - if (pd->start && (retval = pd->start(pdev))) + if (pd && pd->start && (retval = pd->start(pdev))) goto err3; ohci = hcd_to_ohci(hcd); @@ -123,7 +123,7 @@ iounmap(hcd->regs); release_mem_region(hcd->rsrc_start, hcd->rsrc_len); - usb_hcd_put(hcd); + usb_put_hcd(hcd); } static int __devinit ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel