From: kernel test robot <lkp@intel.com>
To: Jani Nikula <jani.nikula@intel.com>, dri-devel@lists.freedesktop.org
Cc: Pan@freedesktop.org, llvm@lists.linux.dev, jani.nikula@intel.com,
intel-gfx@lists.freedesktop.org, Xinhui <Xinhui.Pan@amd.com>,
amd-gfx@lists.freedesktop.org, oe-kbuild-all@lists.linux.dev,
"Alex Deucher" <alexander.deucher@amd.com>,
"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
Date: Fri, 31 Mar 2023 04:13:28 +0800 [thread overview]
Message-ID: <202303310412.bgDHaLy4-lkp@intel.com> (raw)
In-Reply-To: <e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula@intel.com>
Hi Jani,
I love your patch! Yet something to improve:
[auto build test ERROR on drm-misc/drm-misc-next]
[also build test ERROR on drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-exynos/exynos-drm-next linus/master v6.3-rc4 next-20230330]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link: https://lore.kernel.org/r/e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula%40intel.com
patch subject: [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
config: riscv-randconfig-r042-20230329 (https://download.01.org/0day-ci/archive/20230331/202303310412.bgDHaLy4-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 67409911353323ca5edf2049ef0df54132fa1ca7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/ad73d8b0ebf2124b058e95ef5831caa8f2d34229
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
git checkout ad73d8b0ebf2124b058e95ef5831caa8f2d34229
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/gpu/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202303310412.bgDHaLy4-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/radeon/radeon_audio.c:312:30: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
^
drivers/gpu/drm/radeon/radeon_audio.c:335:45: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_speaker_allocation(radeon_connector->edid, &sadb);
^
2 errors generated.
vim +/radeon_connector +312 drivers/gpu/drm/radeon/radeon_audio.c
301
302 static void radeon_audio_write_sad_regs(struct drm_encoder *encoder)
303 {
304 struct drm_connector *connector = radeon_get_connector_for_encoder(encoder);
305 struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder);
306 struct cea_sad *sads;
307 int sad_count;
308
309 if (!connector)
310 return;
311
> 312 sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
313 if (sad_count < 0)
314 DRM_ERROR("Couldn't read SADs: %d\n", sad_count);
315 if (sad_count <= 0)
316 return;
317 BUG_ON(!sads);
318
319 if (radeon_encoder->audio && radeon_encoder->audio->write_sad_regs)
320 radeon_encoder->audio->write_sad_regs(encoder, sads, sad_count);
321
322 kfree(sads);
323 }
324
--
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: Jani Nikula <jani.nikula@intel.com>, dri-devel@lists.freedesktop.org
Cc: Pan@freedesktop.org, llvm@lists.linux.dev, jani.nikula@intel.com,
intel-gfx@lists.freedesktop.org, Xinhui <Xinhui.Pan@amd.com>,
amd-gfx@lists.freedesktop.org, oe-kbuild-all@lists.linux.dev,
"Alex Deucher" <alexander.deucher@amd.com>,
"Christian König" <christian.koenig@amd.com>
Subject: Re: [Intel-gfx] [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
Date: Fri, 31 Mar 2023 04:13:28 +0800 [thread overview]
Message-ID: <202303310412.bgDHaLy4-lkp@intel.com> (raw)
In-Reply-To: <e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula@intel.com>
Hi Jani,
I love your patch! Yet something to improve:
[auto build test ERROR on drm-misc/drm-misc-next]
[also build test ERROR on drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-exynos/exynos-drm-next linus/master v6.3-rc4 next-20230330]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link: https://lore.kernel.org/r/e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula%40intel.com
patch subject: [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
config: riscv-randconfig-r042-20230329 (https://download.01.org/0day-ci/archive/20230331/202303310412.bgDHaLy4-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 67409911353323ca5edf2049ef0df54132fa1ca7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/ad73d8b0ebf2124b058e95ef5831caa8f2d34229
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
git checkout ad73d8b0ebf2124b058e95ef5831caa8f2d34229
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/gpu/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202303310412.bgDHaLy4-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/radeon/radeon_audio.c:312:30: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
^
drivers/gpu/drm/radeon/radeon_audio.c:335:45: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_speaker_allocation(radeon_connector->edid, &sadb);
^
2 errors generated.
vim +/radeon_connector +312 drivers/gpu/drm/radeon/radeon_audio.c
301
302 static void radeon_audio_write_sad_regs(struct drm_encoder *encoder)
303 {
304 struct drm_connector *connector = radeon_get_connector_for_encoder(encoder);
305 struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder);
306 struct cea_sad *sads;
307 int sad_count;
308
309 if (!connector)
310 return;
311
> 312 sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
313 if (sad_count < 0)
314 DRM_ERROR("Couldn't read SADs: %d\n", sad_count);
315 if (sad_count <= 0)
316 return;
317 BUG_ON(!sads);
318
319 if (radeon_encoder->audio && radeon_encoder->audio->write_sad_regs)
320 radeon_encoder->audio->write_sad_regs(encoder, sads, sad_count);
321
322 kfree(sads);
323 }
324
--
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: Jani Nikula <jani.nikula@intel.com>, dri-devel@lists.freedesktop.org
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
jani.nikula@intel.com, intel-gfx@lists.freedesktop.org,
Xinhui <Xinhui.Pan@amd.com>,
amd-gfx@lists.freedesktop.org,
"Alex Deucher" <alexander.deucher@amd.com>,
"Christian König" <christian.koenig@amd.com>,
Pan@freedesktop.org
Subject: Re: [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
Date: Fri, 31 Mar 2023 04:13:28 +0800 [thread overview]
Message-ID: <202303310412.bgDHaLy4-lkp@intel.com> (raw)
In-Reply-To: <e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula@intel.com>
Hi Jani,
I love your patch! Yet something to improve:
[auto build test ERROR on drm-misc/drm-misc-next]
[also build test ERROR on drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-exynos/exynos-drm-next linus/master v6.3-rc4 next-20230330]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link: https://lore.kernel.org/r/e4cb7b0c7217511429e69c1c78729f0e864c5b24.1680190534.git.jani.nikula%40intel.com
patch subject: [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr
config: riscv-randconfig-r042-20230329 (https://download.01.org/0day-ci/archive/20230331/202303310412.bgDHaLy4-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 67409911353323ca5edf2049ef0df54132fa1ca7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/ad73d8b0ebf2124b058e95ef5831caa8f2d34229
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jani-Nikula/drm-edid-parse-display-info-has_audio-similar-to-is_hdmi/20230330-234201
git checkout ad73d8b0ebf2124b058e95ef5831caa8f2d34229
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/gpu/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202303310412.bgDHaLy4-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/radeon/radeon_audio.c:312:30: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
^
drivers/gpu/drm/radeon/radeon_audio.c:335:45: error: use of undeclared identifier 'radeon_connector'
sad_count = drm_edid_to_speaker_allocation(radeon_connector->edid, &sadb);
^
2 errors generated.
vim +/radeon_connector +312 drivers/gpu/drm/radeon/radeon_audio.c
301
302 static void radeon_audio_write_sad_regs(struct drm_encoder *encoder)
303 {
304 struct drm_connector *connector = radeon_get_connector_for_encoder(encoder);
305 struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder);
306 struct cea_sad *sads;
307 int sad_count;
308
309 if (!connector)
310 return;
311
> 312 sad_count = drm_edid_to_sad(radeon_connector->edid, &sads);
313 if (sad_count < 0)
314 DRM_ERROR("Couldn't read SADs: %d\n", sad_count);
315 if (sad_count <= 0)
316 return;
317 BUG_ON(!sads);
318
319 if (radeon_encoder->audio && radeon_encoder->audio->write_sad_regs)
320 radeon_encoder->audio->write_sad_regs(encoder, sads, sad_count);
321
322 kfree(sads);
323 }
324
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
next prev parent reply other threads:[~2023-03-30 20:14 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-30 15:39 [PATCH 00/12] drm: reduce drm_detect_monitor_audio/drm_detect_hdmi_monitor/edid_blob_ptr usage Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Nouveau] " Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] " Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 01/12] drm/edid: parse display info has_audio similar to is_hdmi Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 02/12] drm/nouveau: convert to using is_hdmi and has_audio from display info Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Nouveau] " Jani Nikula
2023-03-30 15:39 ` [PATCH 03/12] drm/radeon: " Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] " Jani Nikula
2023-03-30 15:39 ` [PATCH 04/12] drm/radeon: remove radeon_connector_edid() and stop using edid_blob_ptr Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] " Jani Nikula
2023-03-30 20:13 ` kernel test robot
2023-03-30 20:13 ` kernel test robot
2023-03-30 20:13 ` [Intel-gfx] " kernel test robot
2023-03-30 20:13 ` kernel test robot [this message]
2023-03-30 20:13 ` kernel test robot
2023-03-30 20:13 ` [Intel-gfx] " kernel test robot
2023-03-31 8:50 ` Jani Nikula
2023-03-31 8:50 ` [Intel-gfx] " Jani Nikula
2023-03-30 15:39 ` [PATCH 05/12] drm/amdgpu: remove amdgpu_connector_edid() " Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] " Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 06/12] drm/i915/debugfs: " Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 07/12] drm/exynos: fix is_hdmi usage Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 08/12] drm/i2c/tda998x: convert to using has_audio from display_info Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 09/12] drm/sti/sti_hdmi: convert to using is_hdmi from display info Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 15:39 ` [Intel-gfx] [PATCH 10/12] drm/rockchip: cdn-dp: call drm_connector_update_edid_property() unconditionally Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 16:38 ` [Intel-gfx] " Heiko Stübner
2023-03-30 16:38 ` Heiko Stübner
2023-03-30 15:39 ` [Intel-gfx] [PATCH 11/12] drm/rockchip: convert to using has_audio from display_info Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 16:39 ` [Intel-gfx] " Heiko Stübner
2023-03-30 16:39 ` Heiko Stübner
2023-03-30 15:39 ` [Intel-gfx] [PATCH 12/12] drm/connector: update edid_blob_ptr documentation Jani Nikula
2023-03-30 15:39 ` Jani Nikula
2023-03-30 20:12 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm: reduce drm_detect_monitor_audio/drm_detect_hdmi_monitor/edid_blob_ptr usage Patchwork
2023-03-30 20:25 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-03-31 18:52 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
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=202303310412.bgDHaLy4-lkp@intel.com \
--to=lkp@intel.com \
--cc=Pan@freedesktop.org \
--cc=Xinhui.Pan@amd.com \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=christian.koenig@amd.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
/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.