From: kernel test robot <lkp@intel.com>
To: Jonathan Cormier <jcormier@criticallink.com>,
Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <robert.foss@linaro.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Tomi Valkeinen <tomi.valkeinen@ti.com>,
Jyri Sarha <jsarha@ti.com>
Cc: oe-kbuild-all@lists.linux.dev, dri-devel@lists.freedesktop.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Michael Williamson <michael.williamson@criticallink.com>,
Bob Duke <bduke@criticallink.com>,
Jonathan Cormier <jcormier@criticallink.com>
Subject: Re: [PATCH 4/4] DRM: BRIDGE: TFP410: If connected, use I2C for polled HPD status.
Date: Sun, 29 Jan 2023 02:57:37 +0800 [thread overview]
Message-ID: <202301290252.zgcWeegX-lkp@intel.com> (raw)
In-Reply-To: <20230125-tfp410_i2c-v1-4-66a4d4e390b7@criticallink.com>
Hi Jonathan,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on 93f875a8526a291005e7f38478079526c843cbec]
url: https://github.com/intel-lab-lkp/linux/commits/Jonathan-Cormier/dt-bindings-display-bridge-tfp410-Add-tfp410-i2c-example/20230128-183627
base: 93f875a8526a291005e7f38478079526c843cbec
patch link: https://lore.kernel.org/r/20230125-tfp410_i2c-v1-4-66a4d4e390b7%40criticallink.com
patch subject: [PATCH 4/4] DRM: BRIDGE: TFP410: If connected, use I2C for polled HPD status.
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20230129/202301290252.zgcWeegX-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/c4659fa4c02b62087c095ca99978e5eac8b490de
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jonathan-Cormier/dt-bindings-display-bridge-tfp410-Add-tfp410-i2c-example/20230128-183627
git checkout c4659fa4c02b62087c095ca99978e5eac8b490de
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 olddefconfig
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/bridge/ti-tfp410.c: In function 'tfp410_connector_detect':
>> drivers/gpu/drm/bridge/ti-tfp410.c:111:13: warning: unused variable 'val' [-Wunused-variable]
111 | u32 val;
| ^~~
vim +/val +111 drivers/gpu/drm/bridge/ti-tfp410.c
106
107 static enum drm_connector_status
108 tfp410_connector_detect(struct drm_connector *connector, bool force)
109 {
110 struct tfp410 *dvi = drm_connector_to_tfp410(connector);
> 111 u32 val;
112 unsigned int ret;
113
114 if (dvi->i2c) {
115 ret = regmap_test_bits(dvi->regmap, TFP410_REG_CTL_2_MODE, TFP410_BIT_HTPLG);
116 if (ret < 0)
117 dev_err(dvi->dev, "%s failed to read HTPLG bit : %d\n", __func__, ret);
118 else
119 return ret ? connector_status_connected : connector_status_disconnected;
120 }
121
122 return drm_bridge_detect(dvi->next_bridge);
123 }
124
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Jonathan Cormier <jcormier@criticallink.com>,
Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <robert.foss@linaro.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Tomi Valkeinen <tomi.valkeinen@ti.com>,
Jyri Sarha <jsarha@ti.com>
Cc: Jonathan Cormier <jcormier@criticallink.com>,
devicetree@vger.kernel.org, Bob Duke <bduke@criticallink.com>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Michael Williamson <michael.williamson@criticallink.com>,
oe-kbuild-all@lists.linux.dev
Subject: Re: [PATCH 4/4] DRM: BRIDGE: TFP410: If connected, use I2C for polled HPD status.
Date: Sun, 29 Jan 2023 02:57:37 +0800 [thread overview]
Message-ID: <202301290252.zgcWeegX-lkp@intel.com> (raw)
In-Reply-To: <20230125-tfp410_i2c-v1-4-66a4d4e390b7@criticallink.com>
Hi Jonathan,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on 93f875a8526a291005e7f38478079526c843cbec]
url: https://github.com/intel-lab-lkp/linux/commits/Jonathan-Cormier/dt-bindings-display-bridge-tfp410-Add-tfp410-i2c-example/20230128-183627
base: 93f875a8526a291005e7f38478079526c843cbec
patch link: https://lore.kernel.org/r/20230125-tfp410_i2c-v1-4-66a4d4e390b7%40criticallink.com
patch subject: [PATCH 4/4] DRM: BRIDGE: TFP410: If connected, use I2C for polled HPD status.
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20230129/202301290252.zgcWeegX-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/c4659fa4c02b62087c095ca99978e5eac8b490de
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jonathan-Cormier/dt-bindings-display-bridge-tfp410-Add-tfp410-i2c-example/20230128-183627
git checkout c4659fa4c02b62087c095ca99978e5eac8b490de
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 olddefconfig
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/bridge/ti-tfp410.c: In function 'tfp410_connector_detect':
>> drivers/gpu/drm/bridge/ti-tfp410.c:111:13: warning: unused variable 'val' [-Wunused-variable]
111 | u32 val;
| ^~~
vim +/val +111 drivers/gpu/drm/bridge/ti-tfp410.c
106
107 static enum drm_connector_status
108 tfp410_connector_detect(struct drm_connector *connector, bool force)
109 {
110 struct tfp410 *dvi = drm_connector_to_tfp410(connector);
> 111 u32 val;
112 unsigned int ret;
113
114 if (dvi->i2c) {
115 ret = regmap_test_bits(dvi->regmap, TFP410_REG_CTL_2_MODE, TFP410_BIT_HTPLG);
116 if (ret < 0)
117 dev_err(dvi->dev, "%s failed to read HTPLG bit : %d\n", __func__, ret);
118 else
119 return ret ? connector_status_connected : connector_status_disconnected;
120 }
121
122 return drm_bridge_detect(dvi->next_bridge);
123 }
124
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
next prev parent reply other threads:[~2023-01-28 18:57 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-25 21:09 [PATCH 0/4] DRM: BRIDGE: TFP410: Add i2c support Jonathan Cormier
2023-01-25 21:09 ` Jonathan Cormier
2023-01-25 21:09 ` [PATCH 1/4] dt-bindings: display: bridge: tfp410: Add tfp410 i2c example Jonathan Cormier
2023-01-25 21:09 ` Jonathan Cormier
2023-01-25 21:24 ` Laurent Pinchart
2023-01-25 21:24 ` Laurent Pinchart
2023-01-25 21:59 ` Jon Cormier
2023-01-25 21:59 ` Jon Cormier
2023-01-26 2:54 ` Rob Herring
2023-01-26 2:54 ` Rob Herring
2023-01-26 15:39 ` Krzysztof Kozlowski
2023-01-26 15:39 ` Krzysztof Kozlowski
2023-01-26 18:36 ` Jon Cormier
2023-01-27 8:30 ` Krzysztof Kozlowski
2023-01-27 8:30 ` Krzysztof Kozlowski
2023-01-25 21:09 ` [PATCH 2/4] DRM: BRIDGE: TFP410: Support basic I2C interface Jonathan Cormier
2023-01-25 21:09 ` Jonathan Cormier
2023-01-26 15:41 ` Krzysztof Kozlowski
2023-01-26 15:41 ` Krzysztof Kozlowski
2023-01-26 18:45 ` Jon Cormier
2023-01-25 21:09 ` [PATCH 3/4] DRM: BRIDGE: TFP410: Fix logic to configured polled HPD Jonathan Cormier
2023-01-25 21:09 ` Jonathan Cormier
2023-01-25 21:09 ` [PATCH 4/4] DRM: BRIDGE: TFP410: If connected, use I2C for polled HPD status Jonathan Cormier
2023-01-25 21:09 ` Jonathan Cormier
2023-01-28 18:57 ` kernel test robot [this message]
2023-01-28 18:57 ` kernel test robot
2023-01-29 0:46 ` kernel test robot
2023-01-29 0:46 ` kernel test robot
2023-01-30 16:24 ` Jon Cormier
2023-01-30 16:24 ` Jon Cormier
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=202301290252.zgcWeegX-lkp@intel.com \
--to=lkp@intel.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=bduke@criticallink.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jcormier@criticallink.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=jsarha@ti.com \
--cc=krzk@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michael.williamson@criticallink.com \
--cc=neil.armstrong@linaro.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=robert.foss@linaro.org \
--cc=robh+dt@kernel.org \
--cc=tomi.valkeinen@ti.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.