* Re: [PATCH v18 07/10] drm/mediatek: add mediatek-drm of vdosys0 support for mt8195
[not found] <20220412103114.19922-8-jason-jh.lin@mediatek.com>
@ 2022-04-14 15:31 ` kernel test robot
2022-04-15 6:27 ` Jason-JH Lin
0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2022-04-14 15:31 UTC (permalink / raw)
To: jason-jh.lin, Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
Chun-Kuang Hu, AngeloGioacchino Del Regno
Cc: llvm, kbuild-all, fshao, David Airlie, singo.chang, dri-devel,
Fabien Parent, linux-stm32, roy-cw.yeh,
Project_Global_Chrome_Upstream_Group, Yongqiang Niu, Rex-BC Chen,
devicetree, nancy.lin, linux-mediatek, hsinyi, linux-arm-kernel,
jason-jh . lin, linux-kernel, moudy.ho, Maxime Coquelin
Hi "jason-jh.lin",
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next krzk/for-next linus/master v5.18-rc2 next-20220414]
[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/intel-lab-lkp/linux/commits/jason-jh-lin/Add-Mediatek-Soc-DRM-vdosys0-support-for-mt8195/20220412-183359
base: git://anongit.freedesktop.org/drm/drm drm-next
config: arm64-buildonly-randconfig-r001-20220413 (https://download.01.org/0day-ci/archive/20220414/202204142333.qXgcGMI1-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 6b7e6ea489f6dd45a9b0da9ac20871560917b9b0)
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 arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/7c175317aa80bbc885609a730214448147a46b47
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review jason-jh-lin/Add-Mediatek-Soc-DRM-vdosys0-support-for-mt8195/20220412-183359
git checkout 7c175317aa80bbc885609a730214448147a46b47
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/gpu/drm/mediatek/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/mediatek/mtk_drm_drv.c:707:15: warning: cast to smaller integer type 'enum mtk_ddp_comp_type' from 'const void *' [-Wvoid-pointer-to-enum-cast]
comp_type = (enum mtk_ddp_comp_type)of_id->data;
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:203:42: warning: unused variable 'mt2701_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt2701_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:218:42: warning: unused variable 'mt7623_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt7623_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:234:42: warning: unused variable 'mt2712_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt2712_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:246:42: warning: unused variable 'mt8167_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt8167_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:260:42: warning: unused variable 'mt8173_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt8173_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:274:42: warning: unused variable 'mt8183_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt8183_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:288:42: warning: unused variable 'mt8192_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt8192_mmsys_match_data = {
^
>> drivers/gpu/drm/mediatek/mtk_drm_drv.c:305:42: warning: unused variable 'mt8195_mmsys_match_data' [-Wunused-const-variable]
static const struct mtk_mmsys_match_data mt8195_mmsys_match_data = {
^
9 warnings generated.
vim +/mt2701_mmsys_match_data +203 drivers/gpu/drm/mediatek/mtk_drm_drv.c
202
> 203 static const struct mtk_mmsys_match_data mt2701_mmsys_match_data = {
204 .num_drv_data = 1,
205 .drv_data = {
206 &mt2701_mmsys_driver_data,
207 },
208 };
209
210 static const struct mtk_mmsys_driver_data mt7623_mmsys_driver_data = {
211 .main_path = mt7623_mtk_ddp_main,
212 .main_len = ARRAY_SIZE(mt7623_mtk_ddp_main),
213 .ext_path = mt7623_mtk_ddp_ext,
214 .ext_len = ARRAY_SIZE(mt7623_mtk_ddp_ext),
215 .shadow_register = true,
216 };
217
> 218 static const struct mtk_mmsys_match_data mt7623_mmsys_match_data = {
219 .num_drv_data = 1,
220 .drv_data = {
221 &mt7623_mmsys_driver_data,
222 },
223 };
224
225 static const struct mtk_mmsys_driver_data mt2712_mmsys_driver_data = {
226 .main_path = mt2712_mtk_ddp_main,
227 .main_len = ARRAY_SIZE(mt2712_mtk_ddp_main),
228 .ext_path = mt2712_mtk_ddp_ext,
229 .ext_len = ARRAY_SIZE(mt2712_mtk_ddp_ext),
230 .third_path = mt2712_mtk_ddp_third,
231 .third_len = ARRAY_SIZE(mt2712_mtk_ddp_third),
232 };
233
> 234 static const struct mtk_mmsys_match_data mt2712_mmsys_match_data = {
235 .num_drv_data = 1,
236 .drv_data = {
237 &mt2712_mmsys_driver_data,
238 },
239 };
240
241 static const struct mtk_mmsys_driver_data mt8167_mmsys_driver_data = {
242 .main_path = mt8167_mtk_ddp_main,
243 .main_len = ARRAY_SIZE(mt8167_mtk_ddp_main),
244 };
245
> 246 static const struct mtk_mmsys_match_data mt8167_mmsys_match_data = {
247 .num_drv_data = 1,
248 .drv_data = {
249 &mt8167_mmsys_driver_data,
250 },
251 };
252
253 static const struct mtk_mmsys_driver_data mt8173_mmsys_driver_data = {
254 .main_path = mt8173_mtk_ddp_main,
255 .main_len = ARRAY_SIZE(mt8173_mtk_ddp_main),
256 .ext_path = mt8173_mtk_ddp_ext,
257 .ext_len = ARRAY_SIZE(mt8173_mtk_ddp_ext),
258 };
259
> 260 static const struct mtk_mmsys_match_data mt8173_mmsys_match_data = {
261 .num_drv_data = 1,
262 .drv_data = {
263 &mt8173_mmsys_driver_data,
264 },
265 };
266
267 static const struct mtk_mmsys_driver_data mt8183_mmsys_driver_data = {
268 .main_path = mt8183_mtk_ddp_main,
269 .main_len = ARRAY_SIZE(mt8183_mtk_ddp_main),
270 .ext_path = mt8183_mtk_ddp_ext,
271 .ext_len = ARRAY_SIZE(mt8183_mtk_ddp_ext),
272 };
273
> 274 static const struct mtk_mmsys_match_data mt8183_mmsys_match_data = {
275 .num_drv_data = 1,
276 .drv_data = {
277 &mt8183_mmsys_driver_data,
278 },
279 };
280
281 static const struct mtk_mmsys_driver_data mt8192_mmsys_driver_data = {
282 .main_path = mt8192_mtk_ddp_main,
283 .main_len = ARRAY_SIZE(mt8192_mtk_ddp_main),
284 .ext_path = mt8192_mtk_ddp_ext,
285 .ext_len = ARRAY_SIZE(mt8192_mtk_ddp_ext),
286 };
287
> 288 static const struct mtk_mmsys_match_data mt8192_mmsys_match_data = {
289 .num_drv_data = 1,
290 .drv_data = {
291 &mt8192_mmsys_driver_data,
292 },
293 };
294
295 static const struct mtk_mmsys_driver_data mt8195_vdosys0_driver_data = {
296 .io_start = 0x1c01a000,
297 .main_path = mt8195_mtk_ddp_main,
298 .main_len = ARRAY_SIZE(mt8195_mtk_ddp_main),
299 };
300
301 static const struct mtk_mmsys_driver_data mt8195_vdosys1_driver_data = {
302 .io_start = 0x1c100000,
303 };
304
> 305 static const struct mtk_mmsys_match_data mt8195_mmsys_match_data = {
306 .num_drv_data = 1,
307 .drv_data = {
308 &mt8195_vdosys0_driver_data,
309 &mt8195_vdosys1_driver_data,
310 },
311 };
312
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v18 07/10] drm/mediatek: add mediatek-drm of vdosys0 support for mt8195
2022-04-14 15:31 ` [PATCH v18 07/10] drm/mediatek: add mediatek-drm of vdosys0 support for mt8195 kernel test robot
@ 2022-04-15 6:27 ` Jason-JH Lin
0 siblings, 0 replies; 2+ messages in thread
From: Jason-JH Lin @ 2022-04-15 6:27 UTC (permalink / raw)
To: kernel test robot, Rob Herring, Krzysztof Kozlowski,
Matthias Brugger, Chun-Kuang Hu, AngeloGioacchino Del Regno
Cc: devicetree, moudy.ho, kbuild-all, Maxime Coquelin, David Airlie,
singo.chang, llvm, roy-cw.yeh, dri-devel, linux-kernel,
Project_Global_Chrome_Upstream_Group, Fabien Parent, nancy.lin,
linux-mediatek, Yongqiang Niu, hsinyi, Rex-BC Chen, linux-stm32,
linux-arm-kernel
Hi "kernel test robot",
Thanks for the reviews.
On Thu, 2022-04-14 at 23:31 +0800, kernel test robot wrote:
> Hi "jason-jh.lin",
>
> Thank you for the patch! Perhaps something to improve:
>
> [auto build test WARNING on drm/drm-next]
> [also build test WARNING on robh/for-next krzk/for-next linus/master
> v5.18-rc2 next-20220414]
> [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://urldefense.com/v3/__https://git-scm.com/docs/git-format-patch__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaGS9DHJM$
> ]
>
> url:
> https://urldefense.com/v3/__https://github.com/intel-lab-lkp/linux/commits/jason-jh-lin/Add-Mediatek-Soc-DRM-vdosys0-support-for-mt8195/20220412-183359__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaP5NY9rw$
>
> base: git://anongit.freedesktop.org/drm/drm drm-next
> config: arm64-buildonly-randconfig-r001-20220413 (
> https://urldefense.com/v3/__https://download.01.org/0day-ci/archive/20220414/202204142333.qXgcGMI1-lkp@intel.com/config__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaMatHhNq$
> )
> compiler: clang version 15.0.0 (
> https://urldefense.com/v3/__https://github.com/llvm/llvm-project__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaED4pz5K$
> $ 6b7e6ea489f6dd45a9b0da9ac20871560917b9b0)
> reproduce (this is a W=1 build):
> wget
> https://urldefense.com/v3/__https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaGF0WqUy$
> -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # install arm64 cross compiling tool for clang build
> # apt-get install binutils-aarch64-linux-gnu
> #
> https://urldefense.com/v3/__https://github.com/intel-lab-lkp/linux/commit/7c175317aa80bbc885609a730214448147a46b47__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaN_w7rr0$
>
> git remote add linux-review
> https://urldefense.com/v3/__https://github.com/intel-lab-lkp/linux__;!!CTRNKA9wMg0ARbw!1fp3H7fjidBzBzfT64rjyCv4-HN-czQi_52IEZTo18hsph4srdLAEfrGamKBaMen_Lno$
>
> git fetch --no-tags linux-review jason-jh-lin/Add-Mediatek-
> Soc-DRM-vdosys0-support-for-mt8195/20220412-183359
> git checkout 7c175317aa80bbc885609a730214448147a46b47
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross
> W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/gpu/drm/mediatek/
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> drivers/gpu/drm/mediatek/mtk_drm_drv.c:707:15: warning: cast to
> smaller integer type 'enum mtk_ddp_comp_type' from 'const void *' [-
> Wvoid-pointer-to-enum-cast]
> comp_type = (enum mtk_ddp_comp_type)of_id->data;
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I can't reproduce this build warning in my platform, but I'd found the
same warning fixed patch: 20210928154620.11181-4-cgzones@googlemail.com
It should not be fixed at this series. So I'll fixed this warning in
another fixed patch.
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:203:42: warning: unused
> > > variable 'mt2701_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt2701_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:218:42: warning: unused
> > > variable 'mt7623_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt7623_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:234:42: warning: unused
> > > variable 'mt2712_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt2712_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:246:42: warning: unused
> > > variable 'mt8167_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt8167_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:260:42: warning: unused
> > > variable 'mt8173_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt8173_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:274:42: warning: unused
> > > variable 'mt8183_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt8183_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:288:42: warning: unused
> > > variable 'mt8192_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt8192_mmsys_match_data =
> {
> ^
> > > drivers/gpu/drm/mediatek/mtk_drm_drv.c:305:42: warning: unused
> > > variable 'mt8195_mmsys_match_data' [-Wunused-const-variable]
>
> static const struct mtk_mmsys_match_data mt8195_mmsys_match_data =
> {
Oh, I forgot to replace the driver_data to match_data in mtk_drm_of_ids
table. I'll fix them at the next version.
Regards,
Jason-JH.Lin
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-15 6:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20220412103114.19922-8-jason-jh.lin@mediatek.com>
2022-04-14 15:31 ` [PATCH v18 07/10] drm/mediatek: add mediatek-drm of vdosys0 support for mt8195 kernel test robot
2022-04-15 6:27 ` Jason-JH Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).