Hi Hans, > >+ if (id != 0) > >+ return ret; > > > I believe the check above should be: > if (id != 0 || ret != -EBUSY) > return ret; > > IOW we should not retry the registration if it failed for another reason > then the id already being taken by a legacy driver. Than for readibility we should say: if (!(id == 0 && ret == -EBUSY)) return ret Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
