* [PATCH] fonts: fixup font.h kernel-doc warnings
@ 2026-07-18 19:13 Randy Dunlap
2026-07-18 19:39 ` Helge Deller
0 siblings, 1 reply; 2+ messages in thread
From: Randy Dunlap @ 2026-07-18 19:13 UTC (permalink / raw)
To: dri-devel; +Cc: Randy Dunlap, Helge Deller, Thomas Zimmermann, linux-fbdev
Use the typedef keyword when describing a typedef.
Add the missing function return value for font_glyph_size().
Warning: include/linux/font.h:84 cannot understand function prototype:
'typedef const unsigned char font_data_t;'
Warning: include/linux/font.h:53 No description found for return value
of 'font_glyph_size'
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Cc: Helge Deller <deller@gmx.de>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: linux-fbdev@vger.kernel.org
include/linux/font.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
--- linux-next-20260717.orig/include/linux/font.h
+++ linux-next-20260717/include/linux/font.h
@@ -49,6 +49,8 @@ static inline unsigned int font_glyph_pi
* scanlines, which is usually the glyph's height in scanlines. Fonts
* coming from user space can sometimes have a different vertical pitch
* with empty scanlines between two adjacent glyphs.
+ *
+ * Returns: the number of bytes per glyph
*/
static inline unsigned int font_glyph_size(unsigned int width, unsigned int vpitch)
{
@@ -60,7 +62,7 @@ static inline unsigned int font_glyph_si
*/
/**
- * font_data_t - Raw font data
+ * typedef font_data_t - Raw font data
*
* Values of type font_data_t store a pointer to raw font data. The format
* is monochrome. Each bit sets a pixel of a stored glyph. Font data does
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] fonts: fixup font.h kernel-doc warnings
2026-07-18 19:13 [PATCH] fonts: fixup font.h kernel-doc warnings Randy Dunlap
@ 2026-07-18 19:39 ` Helge Deller
0 siblings, 0 replies; 2+ messages in thread
From: Helge Deller @ 2026-07-18 19:39 UTC (permalink / raw)
To: Randy Dunlap, dri-devel; +Cc: Thomas Zimmermann, linux-fbdev
On 7/18/26 21:13, Randy Dunlap wrote:
> Use the typedef keyword when describing a typedef.
> Add the missing function return value for font_glyph_size().
>
> Warning: include/linux/font.h:84 cannot understand function prototype:
> 'typedef const unsigned char font_data_t;'
> Warning: include/linux/font.h:53 No description found for return value
> of 'font_glyph_size'
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Cc: Helge Deller <deller@gmx.de>
> Cc: Thomas Zimmermann <tzimmermann@suse.de>
> Cc: linux-fbdev@vger.kernel.org
>
> include/linux/font.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
applied to fb-dev git tree.
Thanks for fixing, Randy!
Helge
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-07-18 19:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-18 19:13 [PATCH] fonts: fixup font.h kernel-doc warnings Randy Dunlap
2026-07-18 19:39 ` Helge Deller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox