Document the new field smem_start in struct drm_fb_helper and avoid
a compile-time warning. An error message is shown below and the bug
report is at [1].

  include/drm/drm_fb_helper.h:204: warning: Function parameter or member 
'hint_leak_smem_start' not described in 'drm_fb_helper'

Reported-by: Stephen Rothwell <s...@canb.auug.org.au>
Signed-off-by: Thomas Zimmermann <tzimmerm...@suse.de>
Fixes: e7c5c29a9eb1 ("drm/fb-helper: Set flag in struct drm_fb_helper for 
leaking physical addresses")
Cc: Thomas Zimmermann <tzimmerm...@suse.de>
Cc: Javier Martinez Canillas <javi...@redhat.com>
Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
Cc: Maxime Ripard <mrip...@kernel.org>
Cc: David Airlie <airl...@gmail.com>
Cc: Daniel Vetter <dan...@ffwll.ch>
Cc: dri-devel@lists.freedesktop.org
Link: 
https://lore.kernel.org/dri-devel/20221107143858.0253a...@canb.auug.org.au/T/#u 
# [1]
---
 include/drm/drm_fb_helper.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/include/drm/drm_fb_helper.h b/include/drm/drm_fb_helper.h
index ecfcd2c56d95a..b111dc7ada78d 100644
--- a/include/drm/drm_fb_helper.h
+++ b/include/drm/drm_fb_helper.h
@@ -200,6 +200,13 @@ struct drm_fb_helper {
         */
        int preferred_bpp;
 
+       /**
+        * @hint_leak_smem_start:
+        *
+        * Hint to the fbdev emulation to store the framebuffer's physical
+        * address in struct &fb_info.fix.smem_start. If the hint is unset,
+        * the smem_start field should always be cleared to zero.
+        */
        bool hint_leak_smem_start;
 };
 
-- 
2.38.0

Reply via email to