mirrors/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2025-11-09 05:00:09 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

drm/i915/display: Save a few bytes of memory in intel_backlight_device_register()

Browse source 
'name' may still be "intel_backlight" when backlight_device_register()
is called. In such a case, using kstrdup_const() saves a memory
duplication when dev_set_name() is called in
backlight_device_register().

Use kfree_const() accordingly.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/ecfdb3af5005e05131e2fb93fd870830f39a8c29.1708708142.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
This commit is contained in:
Christophe JAILLET 2024-02-23 18:09:28 +01:00 committed by Jani Nikula
parent a1789b3f5d
commit 379b63e7e6
1 changed files with 3 additions and 3 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

6
drivers/gpu/drm/i915/display/intel_backlight.c
View file

@ -949,7 +949,7 @@ int intel_backlight_device_register(struct intel_connector *connector)
else else
props.power = FB_BLANK_POWERDOWN; props.power = FB_BLANK_POWERDOWN;
name = kstrdup("intel_backlight", GFP_KERNEL); name = kstrdup_const("intel_backlight", GFP_KERNEL);
if (!name) if (!name)
return -ENOMEM; return -ENOMEM;
@ -963,7 +963,7 @@ int intel_backlight_device_register(struct intel_connector *connector)
* compatibility. Use unique names for subsequent backlight devices as a * compatibility. Use unique names for subsequent backlight devices as a
* fallback when the default name already exists. * fallback when the default name already exists.
*/ */
kfree(name); kfree_const(name);
name = kasprintf(GFP_KERNEL, "card%d-%s-backlight", name = kasprintf(GFP_KERNEL, "card%d-%s-backlight",
i915->drm.primary->index, connector->base.name); i915->drm.primary->index, connector->base.name);
if (!name) if (!name)
@ -987,7 +987,7 @@ int intel_backlight_device_register(struct intel_connector *connector)
connector->base.base.id, connector->base.name, name); connector->base.base.id, connector->base.name, name);
out: out:
kfree(name); kfree_const(name);
return ret; return ret;
} }