On Thu, Nov 30, 2023 at 04:33:00PM +0200, Ilpo Järvinen wrote: > On Thu, 30 Nov 2023, srinivas pandruvada wrote: > > On Thu, 2023-11-30 at 14:26 +0200, Ilpo Järvinen wrote: > > > On Tue, 28 Nov 2023, Srinivas Pandruvada wrote: ... > > > > + if (!feature_state.enabled) > > > > + return -EOPNOTSUPP; > > > > > > -ENODEV sounds more appropriate. > > > > The -EOPNOTSUPP is returned matching the next return statement, which > > causes to continue to create devices which are supported and not > > disabled. Any other error is real device creation will causes driver > > modprobe to fail. > > Oh, I see... I didn't look that deep into the code during my review > (perhaps note that down into the commit message?). Maybe we should even use -ENOTSUPP (Linux internal error code), so it will be clear that it's _not_ going to user space? -- With Best Regards, Andy Shevchenko
