Hello Greg Sorry for the mess. I did not want to spam the mailing list too much. Repacking and resending. Thanks! On Sun, May 24, 2015 at 9:29 PM, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > On Fri, May 15, 2015 at 01:52:10PM +0200, Ricardo Ribalda Delgado wrote: >> of_platform_device_create_pdata() was using of_device_add() to create >> the devices, but of_platform_device_destroy was using >> platform_device_unregister() to free them. >> >> of_device_add(), do not call insert_resource(), which initializes the >> parent field of the resource structure, needed by release_resource(), >> called by of_platform_device_destroy(). This leads to a NULL pointer >> deference. >> >> Instead of fixing the NULL pointer deference, what could hide other bugs, >> this patch, replaces of_device_add() with platform_device_data(). >> >> Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> >> Acked-by: Rob Herring <robh@xxxxxxxxxx> >> --- >> >> v3.1 Fix comments by Rob Herring, thanks! > > 3.1? > > Please resend the whole series, this is a mess, I can't find where this > goes at all... > > greg k-h -- Ricardo Ribalda -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
