Re: [PATCH 2/3] Restructure output format computation for better expandability

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Werner,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on next-20210505]
[cannot apply to v5.12]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Werner-Sembach/New-function-to-avoid-duplicate-code-in-upcomming-commits/20210505-231103
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-defconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/66df58af8a9f5c112690fc80f77271b9f6567765
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Werner-Sembach/New-function-to-avoid-duplicate-code-in-upcomming-commits/20210505-231103
        git checkout 66df58af8a9f5c112690fc80f77271b9f6567765
        # save the attached .config to linux build tree
        make W=1 W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

   drivers/gpu/drm/i915/display/intel_hdmi.c: In function 'intel_hdmi_compute_config':
>> drivers/gpu/drm/i915/display/intel_hdmi.c:2166:6: error: 'crtc_state' undeclared (first use in this function); did you mean 'ctx_state'?
    2166 |  if (crtc_state->output_format == INTEL_OUTPUT_FORMAT_YCBCR420) {
         |      ^~~~~~~~~~
         |      ctx_state
   drivers/gpu/drm/i915/display/intel_hdmi.c:2166:6: note: each undeclared identifier is reported only once for each function it appears in


vim +2166 drivers/gpu/drm/i915/display/intel_hdmi.c

  2131	
  2132	int intel_hdmi_compute_config(struct intel_encoder *encoder,
  2133				      struct intel_crtc_state *pipe_config,
  2134				      struct drm_connector_state *conn_state)
  2135	{
  2136		struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
  2137		struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
  2138		struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
  2139		struct drm_connector *connector = conn_state->connector;
  2140		struct drm_scdc *scdc = &connector->display_info.hdmi.scdc;
  2141		int ret;
  2142	
  2143		if (adjusted_mode->flags & DRM_MODE_FLAG_DBLSCAN)
  2144			return -EINVAL;
  2145	
  2146		pipe_config->output_format = INTEL_OUTPUT_FORMAT_RGB;
  2147		pipe_config->has_hdmi_sink = intel_has_hdmi_sink(intel_hdmi,
  2148								 conn_state);
  2149	
  2150		if (pipe_config->has_hdmi_sink)
  2151			pipe_config->has_infoframe = true;
  2152	
  2153		if (adjusted_mode->flags & DRM_MODE_FLAG_DBLCLK)
  2154			pipe_config->pixel_multiplier = 2;
  2155	
  2156		if (HAS_PCH_SPLIT(dev_priv) && !HAS_DDI(dev_priv))
  2157			pipe_config->has_pch_encoder = true;
  2158	
  2159		pipe_config->has_audio =
  2160			intel_hdmi_has_audio(encoder, pipe_config, conn_state);
  2161	
  2162		ret = intel_hdmi_compute_output_format(encoder, pipe_config, conn_state);
  2163		if (ret)
  2164			return ret;
  2165	
> 2166		if (crtc_state->output_format == INTEL_OUTPUT_FORMAT_YCBCR420) {
  2167			ret = intel_pch_panel_fitting(pipe_config, conn_state);
  2168			if (ret)
  2169				return ret;
  2170		}
  2171	
  2172		pipe_config->limited_color_range =
  2173			intel_hdmi_limited_color_range(pipe_config, conn_state);
  2174	
  2175		if (conn_state->picture_aspect_ratio)
  2176			adjusted_mode->picture_aspect_ratio =
  2177				conn_state->picture_aspect_ratio;
  2178	
  2179		pipe_config->lane_count = 4;
  2180	
  2181		if (scdc->scrambling.supported && DISPLAY_VER(dev_priv) >= 10) {
  2182			if (scdc->scrambling.low_rates)
  2183				pipe_config->hdmi_scrambling = true;
  2184	
  2185			if (pipe_config->port_clock > 340000) {
  2186				pipe_config->hdmi_scrambling = true;
  2187				pipe_config->hdmi_high_tmds_clock_ratio = true;
  2188			}
  2189		}
  2190	
  2191		intel_hdmi_compute_gcp_infoframe(encoder, pipe_config,
  2192						 conn_state);
  2193	
  2194		if (!intel_hdmi_compute_avi_infoframe(encoder, pipe_config, conn_state)) {
  2195			drm_dbg_kms(&dev_priv->drm, "bad AVI infoframe\n");
  2196			return -EINVAL;
  2197		}
  2198	
  2199		if (!intel_hdmi_compute_spd_infoframe(encoder, pipe_config, conn_state)) {
  2200			drm_dbg_kms(&dev_priv->drm, "bad SPD infoframe\n");
  2201			return -EINVAL;
  2202		}
  2203	
  2204		if (!intel_hdmi_compute_hdmi_infoframe(encoder, pipe_config, conn_state)) {
  2205			drm_dbg_kms(&dev_priv->drm, "bad HDMI infoframe\n");
  2206			return -EINVAL;
  2207		}
  2208	
  2209		if (!intel_hdmi_compute_drm_infoframe(encoder, pipe_config, conn_state)) {
  2210			drm_dbg_kms(&dev_priv->drm, "bad DRM infoframe\n");
  2211			return -EINVAL;
  2212		}
  2213	
  2214		return 0;
  2215	}
  2216	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux