Re: [PATCH] drm/i915: Apply correct ddi translation table for AML device

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

 



Hi Shawn,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on v4.19-rc5 next-20180926]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Lee-Shawn-C/drm-i915-Apply-correct-ddi-translation-table-for-AML-device/20180925-190446
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
config: i386-randconfig-c0-09270251 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   In file included from include/linux/string.h:6:0,
                    from include/linux/uuid.h:20,
                    from include/linux/mod_devicetable.h:13,
                    from include/linux/i2c.h:29,
                    from include/drm/drm_scdc_helper.h:27,
                    from drivers/gpu//drm/i915/intel_ddi.c:28:
   drivers/gpu//drm/i915/intel_ddi.c: In function 'kbl_get_buf_trans_dp':
   drivers/gpu//drm/i915/intel_ddi.c:645:30: error: implicit declaration of function 'IS_AML_ULX'; did you mean 'IS_KBL_ULX'? [-Werror=implicit-function-declaration]
     if (IS_KBL_ULX(dev_priv) || IS_AML_ULX(dev_priv)) {
                                 ^
   include/linux/compiler.h:58:30: note: in definition of macro '__trace_if'
     if (__builtin_constant_p(!!(cond)) ? !!(cond) :   \
                                 ^~~~
>> drivers/gpu//drm/i915/intel_ddi.c:645:2: note: in expansion of macro 'if'
     if (IS_KBL_ULX(dev_priv) || IS_AML_ULX(dev_priv)) {
     ^~
   Cyclomatic Complexity 1 arch/x86/include/asm/bitops.h:fls
   Cyclomatic Complexity 1 include/linux/log2.h:__ilog2_u32
   Cyclomatic Complexity 1 include/linux/err.h:PTR_ERR
   Cyclomatic Complexity 1 include/linux/err.h:IS_ERR
   Cyclomatic Complexity 1 include/asm-generic/getorder.h:__get_order
   Cyclomatic Complexity 1 arch/x86/include/asm/refcount.h:refcount_dec_and_test
   Cyclomatic Complexity 2 include/linux/kref.h:kref_put
   Cyclomatic Complexity 1 include/drm/drm_scdc_helper.h:drm_scdc_readb
   Cyclomatic Complexity 56 include/linux/slab.h:kmalloc_index
   Cyclomatic Complexity 67 include/linux/slab.h:kmalloc_large
   Cyclomatic Complexity 9 include/linux/slab.h:kmalloc
   Cyclomatic Complexity 1 include/linux/slab.h:kzalloc
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_display.h:transcoder_is_dsi
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/i915_drv.h:to_i915
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/i915_drv.h:intel_info
   Cyclomatic Complexity 3 include/drm/drm_dp_helper.h:drm_dp_enhanced_frame_cap
   Cyclomatic Complexity 1 include/drm/drm_atomic.h:drm_atomic_state_put
   Cyclomatic Complexity 2 drivers/gpu//drm/i915/intel_drv.h:intel_encoder_is_dig_port
   Cyclomatic Complexity 6 drivers/gpu//drm/i915/intel_drv.h:enc_to_dig_port
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_drv.h:enc_to_intel_dp
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_drv.h:dp_to_dig_port
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_drv.h:intel_crtc_has_type
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_drv.h:intel_crtc_has_dp_encoder
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:bxt_get_buf_trans_dp
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:bxt_get_buf_trans_hdmi
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_init_dp_buf_reg
   Cyclomatic Complexity 2 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_main_link_aux_domain
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_buf_trans_fdi
   Cyclomatic Complexity 23 drivers/gpu//drm/i915/intel_ddi.c:skl_get_buf_trans_dp
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:skl_buf_trans_num_entries
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:bdw_get_buf_trans_edp
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:bxt_get_buf_trans_edp
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:_skl_ddi_set_iboost
   Cyclomatic Complexity 13 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_clk_disable
   Cyclomatic Complexity 11 drivers/gpu//drm/i915/intel_ddi.c:skl_calc_wrpll_link
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:hsw_pll_to_ddi_pll_sel
   Cyclomatic Complexity 17 drivers/gpu//drm/i915/intel_ddi.c:icl_get_combo_buf_trans
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:cnl_get_buf_trans_dp
   Cyclomatic Complexity 10 drivers/gpu//drm/i915/intel_ddi.c:cnl_get_buf_trans_edp
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:translate_signal_level
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_dp_level
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:cnl_get_buf_trans_hdmi
   Cyclomatic Complexity 20 drivers/gpu//drm/i915/intel_ddi.c:cnl_ddi_vswing_program
   Cyclomatic Complexity 20 drivers/gpu//drm/i915/intel_ddi.c:cnl_ddi_vswing_sequence
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:hsw_ddi_calc_wrpll_link
   Cyclomatic Complexity 16 drivers/gpu//drm/i915/intel_ddi.c:cnl_calc_wrpll_link
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_power_domains
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_compute_output_type
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:icl_ddi_combo_vswing_program
   Cyclomatic Complexity 20 drivers/gpu//drm/i915/intel_ddi.c:icl_combo_phy_ddi_vswing_sequence
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:icl_ddi_vswing_sequence
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:intel_wait_ddi_buf_idle
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:intel_disable_ddi_buf
   Cyclomatic Complexity 25 drivers/gpu//drm/i915/intel_ddi.c:kbl_get_buf_trans_dp
   Cyclomatic Complexity 43 drivers/gpu//drm/i915/intel_ddi.c:skl_get_buf_trans_edp
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_buf_trans_edp
   Cyclomatic Complexity 9 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_buf_trans_dp
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:intel_prepare_dp_ddi_buffers
   Cyclomatic Complexity 7 drivers/gpu//drm/i915/intel_ddi.c:skl_get_buf_trans_hdmi
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_buf_trans_hdmi
   Cyclomatic Complexity 39 drivers/gpu//drm/i915/intel_ddi.c:skl_ddi_set_iboost
   Cyclomatic Complexity 40 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_hdmi_level
   Cyclomatic Complexity 19 drivers/gpu//drm/i915/intel_ddi.c:intel_prepare_hdmi_ddi_buffers
   Cyclomatic Complexity 11 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_crtc_encoder
   Cyclomatic Complexity 20 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_get_hw_state
   Cyclomatic Complexity 19 drivers/gpu//drm/i915/intel_ddi.c:bxt_ddi_vswing_sequence
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_is_audio_enabled
   Cyclomatic Complexity 16 drivers/gpu//drm/i915/intel_ddi.c:ddi_dotclock_get
   Cyclomatic Complexity 15 drivers/gpu//drm/i915/intel_ddi.c:hsw_ddi_clock_get
   Cyclomatic Complexity 10 drivers/gpu//drm/i915/intel_ddi.c:skl_ddi_clock_get
   Cyclomatic Complexity 12 drivers/gpu//drm/i915/intel_ddi.c:cnl_ddi_clock_get
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:bxt_calc_pll_link
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:bxt_ddi_clock_get
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:icl_ddi_clock_get
   Cyclomatic Complexity 13 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_clock_get
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:intel_disable_ddi_hdmi
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:intel_disable_ddi_dp
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:intel_disable_ddi
   Cyclomatic Complexity 12 drivers/gpu//drm/i915/intel_ddi.c:icl_pll_to_ddi_pll_sel
   Cyclomatic Complexity 17 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_clk_select
   Cyclomatic Complexity 1 drivers/gpu//drm/i915/intel_ddi.c:bxt_ddi_pre_pll_enable
   Cyclomatic Complexity 11 drivers/gpu//drm/i915/intel_ddi.c:intel_enable_ddi_hdmi
   Cyclomatic Complexity 11 drivers/gpu//drm/i915/intel_ddi.c:intel_enable_ddi_dp
   Cyclomatic Complexity 6 drivers/gpu//drm/i915/intel_ddi.c:intel_enable_ddi
   Cyclomatic Complexity 10 drivers/gpu//drm/i915/intel_ddi.c:modeset_pipe
   Cyclomatic Complexity 43 drivers/gpu//drm/i915/intel_ddi.c:intel_hdmi_reset_link
   Cyclomatic Complexity 6 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_hotplug
   Cyclomatic Complexity 11 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_a_force_4_lanes
   Cyclomatic Complexity 8 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_max_lanes
   Cyclomatic Complexity 4 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_init_dp_connector
   Cyclomatic Complexity 3 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_init_hdmi_connector
   Cyclomatic Complexity 15 drivers/gpu//drm/i915/intel_ddi.c:hsw_fdi_link_train
   Cyclomatic Complexity 12 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_set_pipe_settings
   Cyclomatic Complexity 5 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_set_vc_payload_alloc
   Cyclomatic Complexity 31 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_enable_transcoder_func
   Cyclomatic Complexity 6 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_disable_transcoder_func
   Cyclomatic Complexity 7 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_toggle_hdcp_signalling
   Cyclomatic Complexity 9 drivers/gpu//drm/i915/intel_ddi.c:intel_ddi_connector_get_hw_state

vim +/if +645 drivers/gpu//drm/i915/intel_ddi.c

   641	
   642	static const struct ddi_buf_trans *
   643	kbl_get_buf_trans_dp(struct drm_i915_private *dev_priv, int *n_entries)
   644	{
 > 645		if (IS_KBL_ULX(dev_priv) || IS_AML_ULX(dev_priv)) {
   646			*n_entries = ARRAY_SIZE(kbl_y_ddi_translations_dp);
   647			return kbl_y_ddi_translations_dp;
   648		} else if (IS_KBL_ULT(dev_priv) || IS_CFL_ULT(dev_priv)) {
   649			*n_entries = ARRAY_SIZE(kbl_u_ddi_translations_dp);
   650			return kbl_u_ddi_translations_dp;
   651		} else {
   652			*n_entries = ARRAY_SIZE(kbl_ddi_translations_dp);
   653			return kbl_ddi_translations_dp;
   654		}
   655	}
   656	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux