✓ CI.checkpatch: success for drm/i915/display: Remove kstrdup_const() and kfree_const() usage

Patchwork patchwork at emeril.freedesktop.org
Thu Oct 3 17:49:06 UTC 2024


== Series Details ==

Series: drm/i915/display: Remove kstrdup_const() and kfree_const() usage
URL   : https://patchwork.freedesktop.org/series/139523/
State : success

== 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 75f581193957d44a074de2e27a726d068e3a371a
Author: Christophe JAILLET <christophe.jaillet at wanadoo.fr>
Date:   Thu Oct 3 19:41:08 2024 +0200

    drm/i915/display: Remove kstrdup_const() and kfree_const() usage
    
    kstrdup_const() and kfree_const() can be confusing in code built as a
    module. In such a case, it does not do what one could expect from the name
    of the functions.
    
    The code is not wrong by itself, but in such a case, it is equivalent to
    kstrdup() and kfree().
    
    So, keep thinks simple and straightforward.
    
    This reverts commit 379b63e7e682 ("drm/i915/display: Save a few bytes of
    memory in intel_backlight_device_register()")
    
    Signed-off-by: Christophe JAILLET <christophe.jaillet at wanadoo.fr>
+ /mt/dim checkpatch 17c0158bdb239d8b6d23834db5595ea422b69915 drm-intel
75f581193957 drm/i915/display: Remove kstrdup_const() and kfree_const() usage




More information about the Intel-xe mailing list