I suggest to use the summary phrase “Use common error handling code in pi433_init()” instead. > pi433_init had "unregister_chrdev" called twice. > Remove it using goto statements. How do you think about to use the following change description? unregister_chrdev() was called in two if branches. Thus add jump targets so that a bit of exception handling can be better reused at the end of this function implementation. … v2->v3: … a seperate patch. … Would you like to avoid a typo here? Regards, Markus