On 27/02/20 12:36, Vitaly Kuznetsov wrote: >> - if (copy_to_user(cpuid_arg, &cpuid, sizeof(cpuid))) >> - goto out; >> - r = 0; >> + r = -EINVAL; >> break; >> } > Braces are not really needed not but all other cases in the switch have > it so let's leave them here too. > We can remove the case altogether. Paolo
