From: kbuild test robot <lkp@intel.com>
To: "José Roberto de Souza" <jose.souza@intel.com>
Cc: intel-gfx@lists.freedesktop.org, kbuild-all@lists.01.org
Subject: Re: [Intel-gfx] [PATCH 1/6] drm/i915/display: Move out code to return the digital_port of the aux ch
Date: Wed, 1 Apr 2020 13:06:45 +0800 [thread overview]
Message-ID: <202004011348.j9N9ipLN%lkp@intel.com> (raw)
In-Reply-To: <20200401004120.408586-1-jose.souza@intel.com>
[-- Attachment #1: Type: text/plain, Size: 4339 bytes --]
Hi "José,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on drm-tip/drm-tip next-20200331]
[cannot apply to v5.6]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Jos-Roberto-de-Souza/drm-i915-display-Move-out-code-to-return-the-digital_port-of-the-aux-ch/20200401-094021
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-randconfig-s0-20200401 (attached as .config)
compiler: gcc-6 (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/gpu/drm/i915/display/intel_display_power.c: In function 'icl_tc_phy_aux_power_well_enable':
>> drivers/gpu/drm/i915/display/intel_display_power.c:561:40: error: implicit declaration of function 'aux_ch_to_digital_port' [-Werror=implicit-function-declaration]
struct intel_digital_port *dig_port = aux_ch_to_digital_port(dev_priv, aux_ch);
^~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_display_power.c:561:40: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
drivers/gpu/drm/i915/display/intel_display_power.c:564:2: error: too many arguments to function 'icl_tc_port_assert_ref_held'
icl_tc_port_assert_ref_held(dev_priv, power_well, dig_port);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_display_power.c:547:13: note: declared here
static void icl_tc_port_assert_ref_held(struct drm_i915_private *dev_priv,
^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_display_power.c: In function 'icl_tc_phy_aux_power_well_disable':
drivers/gpu/drm/i915/display/intel_display_power.c:593:40: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
struct intel_digital_port *dig_port = aux_ch_to_digital_port(dev_priv, aux_ch);
^~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_display_power.c:595:2: error: too many arguments to function 'icl_tc_port_assert_ref_held'
icl_tc_port_assert_ref_held(dev_priv, power_well, dig_port);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_display_power.c:547:13: note: declared here
static void icl_tc_port_assert_ref_held(struct drm_i915_private *dev_priv,
^~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/aux_ch_to_digital_port +561 drivers/gpu/drm/i915/display/intel_display_power.c
555
556 static void
557 icl_tc_phy_aux_power_well_enable(struct drm_i915_private *dev_priv,
558 struct i915_power_well *power_well)
559 {
560 enum aux_ch aux_ch = icl_tc_phy_aux_ch(dev_priv, power_well);
> 561 struct intel_digital_port *dig_port = aux_ch_to_digital_port(dev_priv, aux_ch);
562 u32 val;
563
564 icl_tc_port_assert_ref_held(dev_priv, power_well, dig_port);
565
566 val = intel_de_read(dev_priv, DP_AUX_CH_CTL(aux_ch));
567 val &= ~DP_AUX_CH_CTL_TBT_IO;
568 if (power_well->desc->hsw.is_tc_tbt)
569 val |= DP_AUX_CH_CTL_TBT_IO;
570 intel_de_write(dev_priv, DP_AUX_CH_CTL(aux_ch), val);
571
572 hsw_power_well_enable(dev_priv, power_well);
573
574 if (INTEL_GEN(dev_priv) >= 12 && !power_well->desc->hsw.is_tc_tbt) {
575 enum tc_port tc_port;
576
577 tc_port = TGL_AUX_PW_TO_TC_PORT(power_well->desc->hsw.idx);
578 intel_de_write(dev_priv, HIP_INDEX_REG(tc_port),
579 HIP_INDEX_VAL(tc_port, 0x2));
580
581 if (intel_de_wait_for_set(dev_priv, DKL_CMN_UC_DW_27(tc_port),
582 DKL_CMN_UC_DW27_UC_HEALTH, 1))
583 drm_warn(&dev_priv->drm,
584 "Timeout waiting TC uC health\n");
585 }
586 }
587
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 31953 bytes --]
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2020-04-01 5:07 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-01 0:41 [Intel-gfx] [PATCH 1/6] drm/i915/display: Move out code to return the digital_port of the aux ch José Roberto de Souza
2020-04-01 0:41 ` [Intel-gfx] [PATCH 2/6] drm/i915/tc: Export tc_port_live_status_mask() José Roberto de Souza
2020-04-01 0:41 ` [Intel-gfx] [PATCH 3/6] drm/i915/display: Add intel_aux_ch_to_power_domain() José Roberto de Souza
2020-04-01 12:09 ` Imre Deak
2020-04-01 0:41 ` [Intel-gfx] [PATCH 4/6] drm/i915/display: Split hsw_power_well_enable() into two José Roberto de Souza
2020-04-01 0:41 ` [Intel-gfx] [PATCH 5/6] drm/i915/tc/icl: Implement TC cold sequences José Roberto de Souza
2020-04-01 12:43 ` Imre Deak
2020-04-01 22:35 ` Souza, Jose
2020-04-02 1:02 ` Imre Deak
2020-04-03 1:18 ` Souza, Jose
2020-04-06 12:57 ` Imre Deak
2020-04-01 0:41 ` [Intel-gfx] [PATCH 6/6] drm/i915/tc/tgl: " José Roberto de Souza
2020-04-01 12:55 ` Imre Deak
2020-04-01 23:36 ` Souza, Jose
2020-04-02 1:08 ` Imre Deak
2020-04-01 1:31 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/6] drm/i915/display: Move out code to return the digital_port of the aux ch Patchwork
2020-04-01 1:43 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
2020-04-01 2:34 ` [Intel-gfx] [PATCH 1/6] " kbuild test robot
2020-04-01 5:06 ` kbuild test robot [this message]
2020-04-01 7:18 ` You-Sheng Yang
2020-04-01 18:18 ` Souza, Jose
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202004011348.j9N9ipLN%lkp@intel.com \
--to=lkp@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jose.souza@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox