Commit 7448d336 authored by Jani Nikula's avatar Jani Nikula

drm/i915/params: use generics for parameter free

Replace the __builtin_strcmp() if ladder with generics.
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarLuca Coelho <luciano.coelho@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230118151800.3669913-3-jani.nikula@intel.com
parent 7ce59bcf
...@@ -296,18 +296,21 @@ void i915_params_copy(struct i915_params *dest, const struct i915_params *src) ...@@ -296,18 +296,21 @@ void i915_params_copy(struct i915_params *dest, const struct i915_params *src)
#undef DUP #undef DUP
} }
static __always_inline void free_param(const char *type, void *x) static void _param_free_charp(char **valp)
{ {
if (!__builtin_strcmp(type, "char *")) { kfree(*valp);
kfree(*(void **)x); *valp = NULL;
*(void **)x = NULL;
}
} }
#define _param_free(valp) \
_Generic(valp, \
char **: _param_free_charp, \
default: _param_nop)(valp)
/* free the allocated members, *not* the passed in params itself */ /* free the allocated members, *not* the passed in params itself */
void i915_params_free(struct i915_params *params) void i915_params_free(struct i915_params *params)
{ {
#define FREE(T, x, ...) free_param(#T, &params->x); #define FREE(T, x, ...) _param_free(&params->x);
I915_PARAMS_FOR_EACH(FREE); I915_PARAMS_FOR_EACH(FREE);
#undef FREE #undef FREE
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment