✗ CI.checkpatch: warning for drm/i915/backlight: Return immediately when scale() finds invalid parameters (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/backlight: Return immediately when scale() finds invalid parameters (rev2)
URL   : https://patchwork.freedesktop.org/series/143773/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit ecedcbf12214233391fd579258462412b032c6a6
Author: Guenter Roeck <linux at roeck-us.net>
Date:   Tue Jan 21 06:52:03 2025 -0800

    drm/i915/backlight: Return immediately when scale() finds invalid parameters
    
    The scale() functions detects invalid parameters, but continues
    its calculations anyway. This causes bad results if negative values
    are used for unsigned operations. Worst case, a division by 0 error
    will be seen if source_min == source_max.
    
    On top of that, after v6.13, the sequence of WARN_ON() followed by clamp()
    may result in a build error with gcc 13.x.
    
    drivers/gpu/drm/i915/display/intel_backlight.c: In function 'scale':
    include/linux/compiler_types.h:542:45: error:
            call to '__compiletime_assert_415' declared with attribute error:
            clamp() low limit source_min greater than high limit source_max
    
    This happens if the compiler decides to rearrange the code as follows.
    
            if (source_min > source_max) {
                    WARN(..);
                    /* Do the clamp() knowing that source_min > source_max */
                    source_val = clamp(source_val, source_min, source_max);
            } else {
                    /* Do the clamp knowing that source_min <= source_max */
                    source_val = clamp(source_val, source_min, source_max);
            }
    
    Fix the problem by evaluating the return values from WARN_ON and returning
    immediately after a warning. While at it, fix divide by zero error seen
    if source_min == source_max.
    
    Analyzed-by: Linus Torvalds <torvalds at linux-foundation.org>
    Suggested-by: Linus Torvalds <torvalds at linux-foundation.org>
    Suggested-by: David Laight <david.laight.linux at gmail.com>
    Cc: David Laight <david.laight.linux at gmail.com>
    Cc: Jani Nikula <jani.nikula at linux.intel.com>
    Cc: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
    Signed-off-by: Guenter Roeck <linux at roeck-us.net>
+ /mt/dim checkpatch 44010d7f04695da899c2860de32625a71011fbab drm-intel
ecedcbf12214 drm/i915/backlight: Return immediately when scale() finds invalid parameters
-:35: WARNING:BAD_SIGN_OFF: Non-standard signature: Analyzed-by:
#35: 
Analyzed-by: Linus Torvalds <torvalds at linux-foundation.org>

total: 0 errors, 1 warnings, 0 checks, 11 lines checked