> @@ -67,6 +92,9 @@ module_init(dspbridge_init); > > static void __exit dspbridge_exit(void) > { > + struct dspbridge_platform_data *pdata = &dspbridge_pdata; > + kfree(pdata->mpu_speeds); > + kfree(pdata->dsp_freq_table); > platform_device_unregister(dspbridge_pdev); > } > module_exit(dspbridge_exit); Ok if pdata->mpu_speeds, pdata->dsp_freq_table are NULL to start with, then you need to make them NULL after kfree as well. Otherwise next time you insert the module it may fail if pdata_mpu_speeds is initialized but pdata->dsp_freq_table is not. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html