Hi Trilok, >> +static int __init scorpion_pmu_init(void) >> +{ >> + platform_device_register(&pmu_device); > > platform_device_register can fail. Better do it like > > "return platform_device_register(&pmu_device); Good catch, I will update it. Thanks, Sheetal -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- 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