Hi Shubhrajyoti, >> + >> +gpio_request_fail: >> + input_unregister_device(context->qcitp_dev); >> + context->qcitp_dev = NULL; >> + >> +register_fail: >> + input_free_device(context->qcitp_dev); > You may like to revisit the use of free after unregister. > That looks OK, because qcitp_dev is set to NULL before this call. ---Trilok Soni -- Sent by a consultant 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-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html