> /* Initialized as a defensive measure to handle edge cases > * where try_cnt might be modified > */ > int err = -EIO; We don't use defensive code in the kernel. Defensive code suggests you don't actually know what your driver is doing and you are guessing this might happen. You should convince yourself it is possible/impossible and write the code as needed. Andrew