On Thu, Dec 14, 2017 at 05:06:12PM +0100, Alexander Steffen wrote: > According to the comments, adding/removing the chip from the list should be > the first/last action in (un)register. But currently it is done in a > subfunction in the middle of the process. Moving the code from the > subfunctions to the appropriate places within (un)register ensures that the > code matches the comments. > > Signed-off-by: Alexander Steffen <Alexander.Steffen@xxxxxxxxxxxx> NAK Not compatible with init_module(). http://man7.org/linux/man-pages/man2/init_module.2.html /Jarkko