]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
drm/i915/wakeref: clean up INTEL_WAKEREF_PUT_* flag macros
authorJani Nikula <jani.nikula@intel.com>
Mon, 15 Dec 2025 12:09:08 +0000 (14:09 +0200)
committerSasha Levin <sashal@kernel.org>
Wed, 4 Mar 2026 12:21:35 +0000 (07:21 -0500)
commitea39bac434c51f6502256baaac9bb9a46c4df144
tree1dac16a3bc4b72ccc4d0cdc3bf2c2fd6e19a4b4e
parentb0bc1aaf938d95c2d6bca423a86cbd953a676140
drm/i915/wakeref: clean up INTEL_WAKEREF_PUT_* flag macros

[ Upstream commit 524696a19e34598c9173fdd5b32fb7e5d16a91d3 ]

Commit 469c1c9eb6c9 ("kernel-doc: Issue warnings that were silently
discarded") started emitting warnings for cases that were previously
silently discarded. One such case is in intel_wakeref.h:

Warning: drivers/gpu/drm/i915/intel_wakeref.h:156 expecting prototype
  for __intel_wakeref_put(). Prototype was for INTEL_WAKEREF_PUT_ASYNC()
  instead

Arguably kernel-doc should be able to handle this, as it's valid C, but
having the flags defined between the function declarator and the body is
just asking for trouble. Move the INTEL_WAKEREF_PUT_* macros away from
there, making kernel-doc's life easier.

While at it, reduce the unnecessary abstraction levels by removing the
enum, and append _MASK to INTEL_WAKEREF_PUT_DELAY for clarity.

Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Jonathan Corbet <corbet@lwn.net>
Acked-by: Randy Dunlap <rdunlap@infradead.org>
Tested-by: Randy Dunlap <rdunlap@infradead.org>
Link: https://patch.msgid.link/20251215120908.3515578-1-jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/i915/intel_wakeref.c
drivers/gpu/drm/i915/intel_wakeref.h