On Thursday 21 October 2021 19:54:45 W_Armin@xxxxxx wrote: > From: Armin Wolf <W_Armin@xxxxxx> > > Returning -ENOIOCTLCMD gives the callers a better > hint of what went wrong and is the recommended > behavior. > > Signed-off-by: Armin Wolf <W_Armin@xxxxxx> Acked-by: Pali Rohár <pali@xxxxxxxxxx> > --- > drivers/hwmon/dell-smm-hwmon.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c > index b0c591bb761a..5f0338b4a717 100644 > --- a/drivers/hwmon/dell-smm-hwmon.c > +++ b/drivers/hwmon/dell-smm-hwmon.c > @@ -18,6 +18,7 @@ > #include <linux/delay.h> > #include <linux/dmi.h> > #include <linux/err.h> > +#include <linux/errno.h> > #include <linux/hwmon.h> > #include <linux/init.h> > #include <linux/module.h> > @@ -516,7 +517,7 @@ i8k_ioctl_unlocked(struct file *fp, struct dell_smm_data *data, unsigned int cmd > break; > > default: > - return -EINVAL; > + return -ENOIOCTLCMD; > } > > if (val < 0) > -- > 2.20.1 >
