Message ID | 1476877404-1327-1-git-send-email-jani.nikula@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, Oct 19, 2016 at 02:43:24PM +0300, Jani Nikula wrote: > Fix warnings on building htmldocs. > > Fixes: 056996b95686 ("drm: Helper for lspcon in drm_dp_dual_mode") > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: Shashank Sharma <shashank.sharma@intel.com> > Cc: <dri-devel@lists.freedesktop.org> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > > --- > > n.b. 056996b95686 ("drm: Helper for lspcon in drm_dp_dual_mode") has > been merged via drm-intel tree > --- > drivers/gpu/drm/drm_dp_dual_mode_helper.c | 14 +++++++------- > include/drm/drm_dp_dual_mode_helper.h | 15 ++++++++------- > 2 files changed, 15 insertions(+), 14 deletions(-) > > diff --git a/drivers/gpu/drm/drm_dp_dual_mode_helper.c b/drivers/gpu/drm/drm_dp_dual_mode_helper.c > index 2624e266abbd..488355bdafb9 100644 > --- a/drivers/gpu/drm/drm_dp_dual_mode_helper.c > +++ b/drivers/gpu/drm/drm_dp_dual_mode_helper.c > @@ -377,9 +377,9 @@ EXPORT_SYMBOL(drm_dp_get_dual_mode_type_name); > > /** > * drm_lspcon_get_mode: Get LSPCON's current mode of operation by > - * by reading offset (0x80, 0x41) > - * @i2c_adapter: I2C-over-aux adapter > - * @current_mode: out vaiable, current lspcon mode of operation > + * reading offset (0x80, 0x41) > + * @adapter: I2C-over-aux adapter > + * @mode: current lspcon mode of operation output variable > * > * Returns: > * 0 on success, sets the current_mode value to appropriate mode > @@ -413,10 +413,10 @@ int drm_lspcon_get_mode(struct i2c_adapter *adapter, > EXPORT_SYMBOL(drm_lspcon_get_mode); > > /** > - * drm_lspcon_change_mode: Change LSPCON's mode of operation by > - * by writing offset (0x80, 0x40) > - * @i2c_adapter: I2C-over-aux adapter > - * @reqd_mode: required mode of operation > + * drm_lspcon_set_mode: Change LSPCON's mode of operation by > + * writing offset (0x80, 0x40) > + * @adapter: I2C-over-aux adapter > + * @mode: required mode of operation > * > * Returns: > * 0 on success, -error on failure/timeout > diff --git a/include/drm/drm_dp_dual_mode_helper.h b/include/drm/drm_dp_dual_mode_helper.h > index 55677704add8..1f04c5cfd402 100644 > --- a/include/drm/drm_dp_dual_mode_helper.h > +++ b/include/drm/drm_dp_dual_mode_helper.h > @@ -70,12 +70,13 @@ ssize_t drm_dp_dual_mode_write(struct i2c_adapter *adapter, > u8 offset, const void *buffer, size_t size); > > /** > -* enum drm_lspcon_mode > -* @lspcon_mode_ls: Level shifter mode of LSPCON > -* which drives DP++ to HDMI 1.4 conversion. > -* @lspcon_mode_pcon: Protocol converter mode of LSPCON > -* which drives DP++ to HDMI 2.0 active conversion. > -*/ > + * enum drm_lspcon_mode > + * @DRM_LSPCON_MODE_INVALID: No LSPCON. > + * @DRM_LSPCON_MODE_LS: Level shifter mode of LSPCON > + * which drives DP++ to HDMI 1.4 conversion. > + * @DRM_LSPCON_MODE_PCON: Protocol converter mode of LSPCON > + * which drives DP++ to HDMI 2.0 active conversion. > + */ > enum drm_lspcon_mode { > DRM_LSPCON_MODE_INVALID, > DRM_LSPCON_MODE_LS, > @@ -90,7 +91,7 @@ enum drm_lspcon_mode { > * @DRM_DP_DUAL_MODE_TYPE1_HDMI: Type 1 HDMI adaptor > * @DRM_DP_DUAL_MODE_TYPE2_DVI: Type 2 DVI adaptor > * @DRM_DP_DUAL_MODE_TYPE2_HDMI: Type 2 HDMI adaptor > - * @DRM_DP_DUAL_MODE_TYPE2_LSPCON: Level shifter /protocol converter > + * @DRM_DP_DUAL_MODE_LSPCON: Level shifter /protocol converter May want to fix up the whitespace around '/' a bit too. Looks weird now. > */ > enum drm_dp_dual_mode_type { > DRM_DP_DUAL_MODE_NONE, > -- > 2.1.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Wed, 19 Oct 2016, Patchwork <patchwork@emeril.freedesktop.org> wrote: > == Series Details == > > Series: drm: Fix LSPCON kernel-doc (rev2) > URL : https://patchwork.freedesktop.org/series/14017/ > State : failure > > == Summary == > > Series 14017v2 drm: Fix LSPCON kernel-doc > https://patchwork.freedesktop.org/api/1.0/series/14017/revisions/2/mbox/ > > Test gem_exec_suspend: > Subgroup basic-s3: > incomplete -> PASS (fi-snb-2520m) > fail -> DMESG-WARN (fi-ilk-650) > Test kms_busy: > Subgroup basic-flip-default-a: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-default-b: > pass -> DMESG-WARN (fi-ilk-650) > Test kms_cursor_legacy: > Subgroup basic-busy-flip-before-cursor-legacy: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-busy-flip-before-cursor-varying-size: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-after-cursor-legacy: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-after-cursor-varying-size: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-before-cursor-legacy: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-before-cursor-varying-size: > pass -> DMESG-WARN (fi-ilk-650) > Test kms_flip: > Subgroup basic-flip-vs-dpms: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-vs-modeset: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-flip-vs-wf_vblank: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup basic-plain-flip: > pass -> DMESG-WARN (fi-ilk-650) > Test kms_force_connector_basic: > Subgroup force-load-detect: > incomplete -> PASS (fi-ivb-3770) > incomplete -> PASS (fi-byt-j1900) > Test kms_pipe_crc_basic: > Subgroup hang-read-crc-pipe-a: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup hang-read-crc-pipe-b: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup nonblocking-crc-pipe-a: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup nonblocking-crc-pipe-a-frame-sequence: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup nonblocking-crc-pipe-b: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup nonblocking-crc-pipe-b-frame-sequence: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup read-crc-pipe-a: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup read-crc-pipe-a-frame-sequence: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup read-crc-pipe-b: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup read-crc-pipe-b-frame-sequence: > pass -> DMESG-WARN (fi-ilk-650) > Subgroup suspend-read-crc-pipe-a: > fail -> DMESG-WARN (fi-ilk-650) > Subgroup suspend-read-crc-pipe-b: > pass -> INCOMPLETE (fi-snb-2600) > fail -> PASS (fi-ilk-650) ilk on an underrun spree. Funny how a comments only change causes that. :/ J. > > fi-bdw-5557u total:246 pass:231 dwarn:0 dfail:0 fail:0 skip:15 > fi-bsw-n3050 total:246 pass:203 dwarn:1 dfail:0 fail:0 skip:42 > fi-bxt-t5700 total:246 pass:216 dwarn:0 dfail:0 fail:0 skip:30 > fi-byt-j1900 total:246 pass:213 dwarn:1 dfail:0 fail:1 skip:31 > fi-byt-n2820 total:246 pass:209 dwarn:1 dfail:0 fail:1 skip:35 > fi-hsw-4770 total:246 pass:224 dwarn:0 dfail:0 fail:0 skip:22 > fi-hsw-4770r total:246 pass:223 dwarn:1 dfail:0 fail:0 skip:22 > fi-ilk-650 total:246 pass:160 dwarn:24 dfail:0 fail:2 skip:60 > fi-ivb-3520m total:246 pass:221 dwarn:0 dfail:0 fail:0 skip:25 > fi-ivb-3770 total:246 pass:221 dwarn:0 dfail:0 fail:0 skip:25 > fi-kbl-7200u total:246 pass:222 dwarn:0 dfail:0 fail:0 skip:24 > fi-skl-6260u total:246 pass:232 dwarn:0 dfail:0 fail:0 skip:14 > fi-skl-6700hq total:246 pass:222 dwarn:1 dfail:0 fail:0 skip:23 > fi-skl-6700k total:246 pass:221 dwarn:1 dfail:0 fail:0 skip:24 > fi-skl-6770hq total:246 pass:232 dwarn:0 dfail:0 fail:0 skip:14 > fi-snb-2520m total:246 pass:210 dwarn:0 dfail:0 fail:0 skip:36 > fi-snb-2600 total:209 pass:176 dwarn:0 dfail:0 fail:0 skip:32 > > Results at /archive/results/CI_IGT_test/Patchwork_2760/ > > d5cbeba648bec880aa0e1f7a531e684499a079a7 drm-intel-nightly: 2016y-10m-19d-11h-13m-08s UTC integration manifest > 827d5ad drm: Fix LSPCON kernel-doc >
diff --git a/drivers/gpu/drm/drm_dp_dual_mode_helper.c b/drivers/gpu/drm/drm_dp_dual_mode_helper.c index 2624e266abbd..488355bdafb9 100644 --- a/drivers/gpu/drm/drm_dp_dual_mode_helper.c +++ b/drivers/gpu/drm/drm_dp_dual_mode_helper.c @@ -377,9 +377,9 @@ EXPORT_SYMBOL(drm_dp_get_dual_mode_type_name); /** * drm_lspcon_get_mode: Get LSPCON's current mode of operation by - * by reading offset (0x80, 0x41) - * @i2c_adapter: I2C-over-aux adapter - * @current_mode: out vaiable, current lspcon mode of operation + * reading offset (0x80, 0x41) + * @adapter: I2C-over-aux adapter + * @mode: current lspcon mode of operation output variable * * Returns: * 0 on success, sets the current_mode value to appropriate mode @@ -413,10 +413,10 @@ int drm_lspcon_get_mode(struct i2c_adapter *adapter, EXPORT_SYMBOL(drm_lspcon_get_mode); /** - * drm_lspcon_change_mode: Change LSPCON's mode of operation by - * by writing offset (0x80, 0x40) - * @i2c_adapter: I2C-over-aux adapter - * @reqd_mode: required mode of operation + * drm_lspcon_set_mode: Change LSPCON's mode of operation by + * writing offset (0x80, 0x40) + * @adapter: I2C-over-aux adapter + * @mode: required mode of operation * * Returns: * 0 on success, -error on failure/timeout diff --git a/include/drm/drm_dp_dual_mode_helper.h b/include/drm/drm_dp_dual_mode_helper.h index 55677704add8..1f04c5cfd402 100644 --- a/include/drm/drm_dp_dual_mode_helper.h +++ b/include/drm/drm_dp_dual_mode_helper.h @@ -70,12 +70,13 @@ ssize_t drm_dp_dual_mode_write(struct i2c_adapter *adapter, u8 offset, const void *buffer, size_t size); /** -* enum drm_lspcon_mode -* @lspcon_mode_ls: Level shifter mode of LSPCON -* which drives DP++ to HDMI 1.4 conversion. -* @lspcon_mode_pcon: Protocol converter mode of LSPCON -* which drives DP++ to HDMI 2.0 active conversion. -*/ + * enum drm_lspcon_mode + * @DRM_LSPCON_MODE_INVALID: No LSPCON. + * @DRM_LSPCON_MODE_LS: Level shifter mode of LSPCON + * which drives DP++ to HDMI 1.4 conversion. + * @DRM_LSPCON_MODE_PCON: Protocol converter mode of LSPCON + * which drives DP++ to HDMI 2.0 active conversion. + */ enum drm_lspcon_mode { DRM_LSPCON_MODE_INVALID, DRM_LSPCON_MODE_LS, @@ -90,7 +91,7 @@ enum drm_lspcon_mode { * @DRM_DP_DUAL_MODE_TYPE1_HDMI: Type 1 HDMI adaptor * @DRM_DP_DUAL_MODE_TYPE2_DVI: Type 2 DVI adaptor * @DRM_DP_DUAL_MODE_TYPE2_HDMI: Type 2 HDMI adaptor - * @DRM_DP_DUAL_MODE_TYPE2_LSPCON: Level shifter /protocol converter + * @DRM_DP_DUAL_MODE_LSPCON: Level shifter /protocol converter */ enum drm_dp_dual_mode_type { DRM_DP_DUAL_MODE_NONE,
Fix warnings on building htmldocs. Fixes: 056996b95686 ("drm: Helper for lspcon in drm_dp_dual_mode") Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Shashank Sharma <shashank.sharma@intel.com> Cc: <dri-devel@lists.freedesktop.org> Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- n.b. 056996b95686 ("drm: Helper for lspcon in drm_dp_dual_mode") has been merged via drm-intel tree --- drivers/gpu/drm/drm_dp_dual_mode_helper.c | 14 +++++++------- include/drm/drm_dp_dual_mode_helper.h | 15 ++++++++------- 2 files changed, 15 insertions(+), 14 deletions(-)