Hi Sachin, >> + if (ret < 0) { >> + pr_err("Pipeline already opened.\n"); >> + return -EBUSY; > > why not propogate 'ret'? Same for other instances below. > Yes it can be done. Will change it. Regards Arun -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html