On Wed, Jul 24, 2013 at 12:28:02PM -0400, Rhyland Klein wrote: > + /* Store the pdata pointer after everything else passes */ > + pdev->dev.platform_data = pdata;
No, this is bad - the driver should not be modifying the platform_data pointer, it's owned by the thing that created the device. If the platform data needs to be modified it should be copied somewhere in the driver data.
signature.asc
Description: Digital signature
