Commit ab2b29e6 authored by Sam Ravnborg's avatar Sam Ravnborg

video: fbdev: s3c-fb: Fix kernel-doc and set but not used warnings

Fix several W=1 warnings
- Updated kernel-doc as needed
- Deleted unused local variable, it was assigned but never used

v2:
  - Updated subject (Lee)
Signed-off-by: default avatarSam Ravnborg <sam@ravnborg.org>
Acked-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Cc: Jingoo Han <jingoohan1@gmail.com>
Cc: linux-fbdev@vger.kernel.org
Cc: Lee Jones <lee.jones@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20201128224114.1033617-25-sam@ravnborg.org
parent c96da175
...@@ -75,6 +75,7 @@ struct s3c_fb; ...@@ -75,6 +75,7 @@ struct s3c_fb;
* @buf_size: Offset of buffer size registers. * @buf_size: Offset of buffer size registers.
* @buf_end: Offset of buffer end registers. * @buf_end: Offset of buffer end registers.
* @osd: The base for the OSD registers. * @osd: The base for the OSD registers.
* @osd_stride: stride of osd
* @palette: Address of palette memory, or 0 if none. * @palette: Address of palette memory, or 0 if none.
* @has_prtcon: Set if has PRTCON register. * @has_prtcon: Set if has PRTCON register.
* @has_shadowcon: Set if has SHADOWCON register. * @has_shadowcon: Set if has SHADOWCON register.
...@@ -155,7 +156,7 @@ struct s3c_fb_palette { ...@@ -155,7 +156,7 @@ struct s3c_fb_palette {
* @windata: The platform data supplied for the window configuration. * @windata: The platform data supplied for the window configuration.
* @parent: The hardware that this window is part of. * @parent: The hardware that this window is part of.
* @fbinfo: Pointer pack to the framebuffer info for this window. * @fbinfo: Pointer pack to the framebuffer info for this window.
* @varint: The variant information for this window. * @variant: The variant information for this window.
* @palette_buffer: Buffer/cache to hold palette entries. * @palette_buffer: Buffer/cache to hold palette entries.
* @pseudo_palette: For use in TRUECOLOUR modes for entries 0..15/ * @pseudo_palette: For use in TRUECOLOUR modes for entries 0..15/
* @index: The window number of this window. * @index: The window number of this window.
...@@ -336,7 +337,7 @@ static int s3c_fb_check_var(struct fb_var_screeninfo *var, ...@@ -336,7 +337,7 @@ static int s3c_fb_check_var(struct fb_var_screeninfo *var,
/** /**
* s3c_fb_calc_pixclk() - calculate the divider to create the pixel clock. * s3c_fb_calc_pixclk() - calculate the divider to create the pixel clock.
* @sfb: The hardware state. * @sfb: The hardware state.
* @pixclock: The pixel clock wanted, in picoseconds. * @pixclk: The pixel clock wanted, in picoseconds.
* *
* Given the specified pixel clock, work out the necessary divider to get * Given the specified pixel clock, work out the necessary divider to get
* close to the output frequency. * close to the output frequency.
...@@ -733,7 +734,7 @@ static inline unsigned int chan_to_field(unsigned int chan, ...@@ -733,7 +734,7 @@ static inline unsigned int chan_to_field(unsigned int chan,
* @red: The red field for the palette data. * @red: The red field for the palette data.
* @green: The green field for the palette data. * @green: The green field for the palette data.
* @blue: The blue field for the palette data. * @blue: The blue field for the palette data.
* @trans: The transparency (alpha) field for the palette data. * @transp: The transparency (alpha) field for the palette data.
* @info: The framebuffer being changed. * @info: The framebuffer being changed.
*/ */
static int s3c_fb_setcolreg(unsigned regno, static int s3c_fb_setcolreg(unsigned regno,
...@@ -1133,6 +1134,7 @@ static void s3c_fb_free_memory(struct s3c_fb *sfb, struct s3c_fb_win *win) ...@@ -1133,6 +1134,7 @@ static void s3c_fb_free_memory(struct s3c_fb *sfb, struct s3c_fb_win *win)
/** /**
* s3c_fb_release_win() - release resources for a framebuffer window. * s3c_fb_release_win() - release resources for a framebuffer window.
* @sfb: The base resources for the hardware.
* @win: The window to cleanup the resources for. * @win: The window to cleanup the resources for.
* *
* Release the resources that where claimed for the hardware window, * Release the resources that where claimed for the hardware window,
...@@ -1160,6 +1162,7 @@ static void s3c_fb_release_win(struct s3c_fb *sfb, struct s3c_fb_win *win) ...@@ -1160,6 +1162,7 @@ static void s3c_fb_release_win(struct s3c_fb *sfb, struct s3c_fb_win *win)
/** /**
* s3c_fb_probe_win() - register an hardware window * s3c_fb_probe_win() - register an hardware window
* @sfb: The base resources for the hardware * @sfb: The base resources for the hardware
* @win_no: The window number
* @variant: The variant information for this window. * @variant: The variant information for this window.
* @res: Pointer to where to place the resultant window. * @res: Pointer to where to place the resultant window.
* *
...@@ -1170,7 +1173,6 @@ static int s3c_fb_probe_win(struct s3c_fb *sfb, unsigned int win_no, ...@@ -1170,7 +1173,6 @@ static int s3c_fb_probe_win(struct s3c_fb *sfb, unsigned int win_no,
struct s3c_fb_win_variant *variant, struct s3c_fb_win_variant *variant,
struct s3c_fb_win **res) struct s3c_fb_win **res)
{ {
struct fb_var_screeninfo *var;
struct fb_videomode initmode; struct fb_videomode initmode;
struct s3c_fb_pd_win *windata; struct s3c_fb_pd_win *windata;
struct s3c_fb_win *win; struct s3c_fb_win *win;
...@@ -1198,7 +1200,6 @@ static int s3c_fb_probe_win(struct s3c_fb *sfb, unsigned int win_no, ...@@ -1198,7 +1200,6 @@ static int s3c_fb_probe_win(struct s3c_fb *sfb, unsigned int win_no,
win = fbinfo->par; win = fbinfo->par;
*res = win; *res = win;
var = &fbinfo->var;
win->variant = *variant; win->variant = *variant;
win->fbinfo = fbinfo; win->fbinfo = fbinfo;
win->parent = sfb; win->parent = sfb;
......
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