On Wed, 14 Feb 2018, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> The ages old kerneldoc-esque comments still refer to the original sutbs
"Sutbs" sounds like a profanity that should have an urban dictionary
definition. As it doesn't (not yet at least!) I presume it's just a
boring typo for "stubs".
Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> and not the more complete functions. As they were only describing the
> external entry points (or at least thought themselves to be, they had
> drifted!), they don't provide any commentary for the code flow.
>
> drivers/gpu/drm/i915/intel_tv.c:379: warning: cannot understand function prototype: 'const struct tv_mode tv_modes[] = '
> drivers/gpu/drm/i915/intel_tv.c:1133: warning: bad line:
> drivers/gpu/drm/i915/intel_tv.c:1140: warning: Function parameter or member 'intel_tv' not described in 'intel_tv_detect_type'
> drivers/gpu/drm/i915/intel_tv.c:1140: warning: Function parameter or member 'connector' not described in 'intel_tv_detect_type'
> drivers/gpu/drm/i915/intel_tv.c:1272: warning: Function parameter or member 'connector' not described in 'intel_tv_detect'
> drivers/gpu/drm/i915/intel_tv.c:1272: warning: Function parameter or member 'ctx' not described in 'intel_tv_detect'
> drivers/gpu/drm/i915/intel_tv.c:1272: warning: Function parameter or member 'force' not described in 'intel_tv_detect'
> drivers/gpu/drm/i915/intel_tv.c:1351: warning: Function parameter or member 'connector' not described in 'intel_tv_get_modes'
>
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/intel_tv.c | 28 +++-------------------------
> 1 file changed, 3 insertions(+), 25 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c
> index b3dabc219e6a..885fc3809f7f 100644
> --- a/drivers/gpu/drm/i915/intel_tv.c
> +++ b/drivers/gpu/drm/i915/intel_tv.c
> @@ -43,7 +43,6 @@ enum tv_margin {
> TV_MARGIN_RIGHT, TV_MARGIN_BOTTOM
> };
>
> -/** Private structure for the integrated TV support */
> struct intel_tv {
> struct intel_encoder base;
>
> @@ -370,12 +369,11 @@ struct tv_mode {
> * The constants below were all computed using a 107.520MHz clock
> */
>
> -/**
> +/*
> * Register programming values for TV modes.
> *
> * These values account for -1s required.
> */
> -
> static const struct tv_mode tv_modes[] = {
> {
> .name = "NTSC-M",
> @@ -1126,14 +1124,6 @@ static const struct drm_display_mode reported_modes[] = {
> },
> };
>
> -/**
> - * Detects TV presence by checking for load.
> - *
> - * Requires that the current pipe's DPLL is active.
> -
> - * \return true if TV is connected.
> - * \return false if TV is disconnected.
> - */
> static int
> intel_tv_detect_type(struct intel_tv *intel_tv,
> struct drm_connector *connector)
> @@ -1259,12 +1249,6 @@ static void intel_tv_find_better_format(struct drm_connector *connector)
> connector->state->tv.mode = i;
> }
>
> -/**
> - * Detect the TV connection.
> - *
> - * Currently this always returns CONNECTOR_STATUS_UNKNOWN, as we need to be sure
> - * we have a pipe programmed in order to probe the TV.
> - */
> static int
> intel_tv_detect(struct drm_connector *connector,
> struct drm_modeset_acquire_ctx *ctx,
> @@ -1339,13 +1323,6 @@ intel_tv_choose_preferred_modes(const struct tv_mode *tv_mode,
> }
> }
>
> -/**
> - * Stub get_modes function.
> - *
> - * This should probably return a set of fixed modes, unless we can figure out
> - * how to probe modes off of TV connections.
> - */
> -
> static int
> intel_tv_get_modes(struct drm_connector *connector)
> {
> @@ -1512,7 +1489,8 @@ intel_tv_init(struct drm_i915_private *dev_priv)
> connector = &intel_connector->base;
> state = connector->state;
>
> - /* The documentation, for the older chipsets at least, recommend
> + /*
> + * The documentation, for the older chipsets at least, recommend
> * using a polling method rather than hotplug detection for TVs.
> * This is because in order to perform the hotplug detection, the PLLs
> * for the TV must be kept alive increasing power drain and starving
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
