On Mon, Mar 28, 2022 at 12:21:16PM +0200, Alain Volmat wrote:
> Hi Randy,
>
> thanks for the patch.
>
> Acked-by: Alain Volmat <alain.volmat@xxxxxxxxxxx>
Will Philippe apply this one?
Just trying to make sure this wont be lost, just acking isn't enough :-)
-Daniel
>
> Alain
>
> On Sat, Mar 26, 2022 at 04:51:32PM -0700, Randy Dunlap wrote:
> > Don't mark static functions as kernel-doc.
> >
> > Prevents multiple kernel-doc build warnings:
> >
> > drivers/gpu/drm/sti/sti_hdmi.c:187: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * HDMI interrupt handler threaded
> > drivers/gpu/drm/sti/sti_hdmi.c:219: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * HDMI interrupt handler
> > drivers/gpu/drm/sti/sti_hdmi.c:241: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Set hdmi active area depending on the drm display mode selected
> > drivers/gpu/drm/sti/sti_hdmi.c:262: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Overall hdmi configuration
> > drivers/gpu/drm/sti/sti_hdmi.c:340: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Helper to concatenate infoframe in 32 bits word
> > drivers/gpu/drm/sti/sti_hdmi.c:357: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Helper to write info frame
> > drivers/gpu/drm/sti/sti_hdmi.c:427: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Prepare and configure the AVI infoframe
> > drivers/gpu/drm/sti/sti_hdmi.c:470: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Prepare and configure the AUDIO infoframe
> > drivers/gpu/drm/sti/sti_hdmi.c:555: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> > * Software reset of the hdmi subsystem
> >
> > Fixes: 5402626c83a2 ("drm: sti: add HDMI driver")
> > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> > Cc: Aditya Srivastava <yashsri421@xxxxxxxxx>
> > Cc: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx>
> > Cc: Alain Volmat <alain.volmat@xxxxxxxxxxx>
> > Cc: David Airlie <airlied@xxxxxxxx>
> > Cc: Daniel Vetter <daniel@xxxxxxxx>
> > ---
> > drivers/gpu/drm/sti/sti_hdmi.c | 20 ++++++++++----------
> > 1 file changed, 10 insertions(+), 10 deletions(-)
> >
> > --- linux-next-20220325.orig/drivers/gpu/drm/sti/sti_hdmi.c
> > +++ linux-next-20220325/drivers/gpu/drm/sti/sti_hdmi.c
> > @@ -183,7 +183,7 @@ void hdmi_write(struct sti_hdmi *hdmi, u
> > writel(val, hdmi->regs + offset);
> > }
> >
> > -/**
> > +/*
> > * HDMI interrupt handler threaded
> > *
> > * @irq: irq number
> > @@ -215,7 +215,7 @@ static irqreturn_t hdmi_irq_thread(int i
> > return IRQ_HANDLED;
> > }
> >
> > -/**
> > +/*
> > * HDMI interrupt handler
> > *
> > * @irq: irq number
> > @@ -237,7 +237,7 @@ static irqreturn_t hdmi_irq(int irq, voi
> > return IRQ_WAKE_THREAD;
> > }
> >
> > -/**
> > +/*
> > * Set hdmi active area depending on the drm display mode selected
> > *
> > * @hdmi: pointer on the hdmi internal structure
> > @@ -258,7 +258,7 @@ static void hdmi_active_area(struct sti_
> > hdmi_write(hdmi, ymax, HDMI_ACTIVE_VID_YMAX);
> > }
> >
> > -/**
> > +/*
> > * Overall hdmi configuration
> > *
> > * @hdmi: pointer on the hdmi internal structure
> > @@ -336,7 +336,7 @@ static void hdmi_infoframe_reset(struct
> > hdmi_write(hdmi, 0x0, pack_offset + i);
> > }
> >
> > -/**
> > +/*
> > * Helper to concatenate infoframe in 32 bits word
> > *
> > * @ptr: pointer on the hdmi internal structure
> > @@ -353,7 +353,7 @@ static inline unsigned int hdmi_infofram
> > return value;
> > }
> >
> > -/**
> > +/*
> > * Helper to write info frame
> > *
> > * @hdmi: pointer on the hdmi internal structure
> > @@ -423,7 +423,7 @@ static void hdmi_infoframe_write_infopac
> > hdmi_write(hdmi, val, HDMI_SW_DI_CFG);
> > }
> >
> > -/**
> > +/*
> > * Prepare and configure the AVI infoframe
> > *
> > * AVI infoframe are transmitted at least once per two video field and
> > @@ -466,7 +466,7 @@ static int hdmi_avi_infoframe_config(str
> > return 0;
> > }
> >
> > -/**
> > +/*
> > * Prepare and configure the AUDIO infoframe
> > *
> > * AUDIO infoframe are transmitted once per frame and
> > @@ -551,7 +551,7 @@ static int hdmi_vendor_infoframe_config(
> >
> > #define HDMI_TIMEOUT_SWRESET 100 /*milliseconds */
> >
> > -/**
> > +/*
> > * Software reset of the hdmi subsystem
> > *
> > * @hdmi: pointer on the hdmi internal structure
> > @@ -785,7 +785,7 @@ static void sti_hdmi_disable(struct drm_
> > cec_notifier_set_phys_addr(hdmi->notifier, CEC_PHYS_ADDR_INVALID);
> > }
> >
> > -/**
> > +/*
> > * sti_hdmi_audio_get_non_coherent_n() - get N parameter for non-coherent
> > * clocks. None-coherent clocks means that audio and TMDS clocks have not the
> > * same source (drifts between clocks). In this case assumption is that CTS is
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
