On Mon, 17 Nov 2025 09:06:02 +0000
Nam Cao <[email protected]> wrote:

> @@ -644,13 +640,11 @@ static ssize_t enabled_monitors_write(struct file 
> *filp, const char __user *user
>               else
>                       retval = rv_disable_monitor(mon);
>  
> -             if (!retval)
> -                     retval = count;
> -
> -             break;
> +             if (retval)
> +                     return retval;
> +             return count;

No biggy, but I wonder if this would look better as:

                return retval ? : count;

-- Steve

>       }
>  
> -     mutex_unlock(&rv_interface_lock);
>       return retval;
>  }
>  

Reply via email to