Re: [PATCH] drm: sti: fix W=1 warnings

From: Daniel Vetter
Date: Tue Sep 17 2019 - 08:46:43 EST


On Mon, Sep 16, 2019 at 03:16:49PM +0200, Benjamin Gaignard wrote:
> Le lun. 9 sept. 2019 à 12:29, Benjamin Gaignard
> <benjamin.gaignard@xxxxxx> a écrit :
> >
> > Fix warnings when W=1.
> > No code changes, only clean up in sti internal structures and functions
> > descriptions.
> >
> > Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx>
>
> For my own reference, applied on drm-misc-next

Dude seriously no:

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx>
Reviewed-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx>
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx>
Link: https://patchwork.freedesktop.org/patch/msgid/20190909101254.24191-1-benjamin.gaignard@xxxxxx

Self-review ain't ok in drm-misc, you need someone to
ack/double-check/have a look. And given that you had to fabricate your
self-review yourself somehow (the tools really don't do that for you) this
doesn't look like an accident.

Adding other maintainers.
-Daniel

>
> > ---
> > drivers/gpu/drm/sti/sti_cursor.c | 2 +-
> > drivers/gpu/drm/sti/sti_dvo.c | 2 +-
> > drivers/gpu/drm/sti/sti_gdp.c | 2 +-
> > drivers/gpu/drm/sti/sti_hda.c | 2 +-
> > drivers/gpu/drm/sti/sti_hdmi.c | 4 ++--
> > drivers/gpu/drm/sti/sti_tvout.c | 10 +++++-----
> > drivers/gpu/drm/sti/sti_vtg.c | 2 +-
> > 7 files changed, 12 insertions(+), 12 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/sti/sti_cursor.c b/drivers/gpu/drm/sti/sti_cursor.c
> > index 0bf7c332cf0b..ea64c1dcaf63 100644
> > --- a/drivers/gpu/drm/sti/sti_cursor.c
> > +++ b/drivers/gpu/drm/sti/sti_cursor.c
> > @@ -47,7 +47,7 @@ struct dma_pixmap {
> > void *base;
> > };
> >
> > -/**
> > +/*
> > * STI Cursor structure
> > *
> > * @sti_plane: sti_plane structure
> > diff --git a/drivers/gpu/drm/sti/sti_dvo.c b/drivers/gpu/drm/sti/sti_dvo.c
> > index 9e6d5d8b7030..c33d0aaee82b 100644
> > --- a/drivers/gpu/drm/sti/sti_dvo.c
> > +++ b/drivers/gpu/drm/sti/sti_dvo.c
> > @@ -65,7 +65,7 @@ static struct dvo_config rgb_24bit_de_cfg = {
> > .awg_fwgen_fct = sti_awg_generate_code_data_enable_mode,
> > };
> >
> > -/**
> > +/*
> > * STI digital video output structure
> > *
> > * @dev: driver device
> > diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c
> > index 8e926cd6a1c8..11595c748844 100644
> > --- a/drivers/gpu/drm/sti/sti_gdp.c
> > +++ b/drivers/gpu/drm/sti/sti_gdp.c
> > @@ -103,7 +103,7 @@ struct sti_gdp_node_list {
> > dma_addr_t btm_field_paddr;
> > };
> >
> > -/**
> > +/*
> > * STI GDP structure
> > *
> > * @sti_plane: sti_plane structure
> > diff --git a/drivers/gpu/drm/sti/sti_hda.c b/drivers/gpu/drm/sti/sti_hda.c
> > index 94e404f13234..3512a94a0fca 100644
> > --- a/drivers/gpu/drm/sti/sti_hda.c
> > +++ b/drivers/gpu/drm/sti/sti_hda.c
> > @@ -230,7 +230,7 @@ static const struct sti_hda_video_config hda_supported_modes[] = {
> > AWGi_720x480p_60, NN_720x480p_60, VID_ED}
> > };
> >
> > -/**
> > +/*
> > * STI hd analog structure
> > *
> > * @dev: driver device
> > diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
> > index f03d617edc4c..87e34f7a6cfe 100644
> > --- a/drivers/gpu/drm/sti/sti_hdmi.c
> > +++ b/drivers/gpu/drm/sti/sti_hdmi.c
> > @@ -333,7 +333,6 @@ static void hdmi_infoframe_reset(struct sti_hdmi *hdmi,
> > * Helper to concatenate infoframe in 32 bits word
> > *
> > * @ptr: pointer on the hdmi internal structure
> > - * @data: infoframe to write
> > * @size: size to write
> > */
> > static inline unsigned int hdmi_infoframe_subpack(const u8 *ptr, size_t size)
> > @@ -543,13 +542,14 @@ static int hdmi_vendor_infoframe_config(struct sti_hdmi *hdmi)
> > return 0;
> > }
> >
> > +#define HDMI_TIMEOUT_SWRESET 100 /*milliseconds */
> > +
> > /**
> > * Software reset of the hdmi subsystem
> > *
> > * @hdmi: pointer on the hdmi internal structure
> > *
> > */
> > -#define HDMI_TIMEOUT_SWRESET 100 /*milliseconds */
> > static void hdmi_swreset(struct sti_hdmi *hdmi)
> > {
> > u32 val;
> > diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
> > index e1b3c8cb7287..b1fc77b150da 100644
> > --- a/drivers/gpu/drm/sti/sti_tvout.c
> > +++ b/drivers/gpu/drm/sti/sti_tvout.c
> > @@ -157,9 +157,9 @@ static void tvout_write(struct sti_tvout *tvout, u32 val, int offset)
> > *
> > * @tvout: tvout structure
> > * @reg: register to set
> > - * @cr_r:
> > - * @y_g:
> > - * @cb_b:
> > + * @cr_r: red chroma or red order
> > + * @y_g: y or green order
> > + * @cb_b: blue chroma or blue order
> > */
> > static void tvout_vip_set_color_order(struct sti_tvout *tvout, int reg,
> > u32 cr_r, u32 y_g, u32 cb_b)
> > @@ -214,7 +214,7 @@ static void tvout_vip_set_rnd(struct sti_tvout *tvout, int reg, u32 rnd)
> > * @tvout: tvout structure
> > * @reg: register to set
> > * @main_path: main or auxiliary path
> > - * @sel_input: selected_input (main/aux + conv)
> > + * @video_out: selected_input (main/aux + conv)
> > */
> > static void tvout_vip_set_sel_input(struct sti_tvout *tvout,
> > int reg,
> > @@ -251,7 +251,7 @@ static void tvout_vip_set_sel_input(struct sti_tvout *tvout,
> > *
> > * @tvout: tvout structure
> > * @reg: register to set
> > - * @in_vid_signed: used video input format
> > + * @in_vid_fmt: used video input format
> > */
> > static void tvout_vip_set_in_vid_fmt(struct sti_tvout *tvout,
> > int reg, u32 in_vid_fmt)
> > diff --git a/drivers/gpu/drm/sti/sti_vtg.c b/drivers/gpu/drm/sti/sti_vtg.c
> > index ef4009f11396..0b17ac8a3faa 100644
> > --- a/drivers/gpu/drm/sti/sti_vtg.c
> > +++ b/drivers/gpu/drm/sti/sti_vtg.c
> > @@ -121,7 +121,7 @@ struct sti_vtg_sync_params {
> > u32 vsync_off_bot;
> > };
> >
> > -/**
> > +/*
> > * STI VTG structure
> > *
> > * @regs: register mapping
> > --
> > 2.15.0
> >

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch