✓ 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