On 02/24, Georgi Djakov wrote: > On 02/24/2015 12:46 AM, Stephen Boyd wrote: > > On 02/06/15 10:58, Georgi Djakov wrote: > > > >> + > >> + return qcom_cc_really_probe(pdev, &gcc_msm8916_desc, regmap); > >> +} > >> + > >> +static int gcc_msm8916_remove(struct platform_device *pdev) > >> +{ > >> + qcom_cc_remove(pdev); > >> + return 0; > >> +} > >> + > >> +static struct platform_driver gcc_msm8916_driver = { > >> + .probe = gcc_msm8916_probe, > >> + .remove = gcc_msm8916_remove, > >> + .driver = { > >> + .name = "gcc-msm8916", > > > > We need owner = THIS_MODULE here because the platform_driver_module > > macro isn't being used. > > > > But below i use the platform_driver_register macro and therefore it seems > not needed. > Ah ok, sorry for the confusion. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html