From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v1 2/2] drm/bridge: add refactored it6505 driver
Date: Mon, 10 Aug 2020 20:53:00 +0800 [thread overview]
Message-ID: <202008102021.WKLURcQx%lkp@intel.com> (raw)
In-Reply-To: <1597054312-25538-3-git-send-email-allen.chen@ite.com.tw>
[-- Attachment #1: Type: text/plain, Size: 8917 bytes --]
Hi allen,
I love your patch! Yet something to improve:
[auto build test ERROR on robh/for-next]
[also build test ERROR on drm-intel/for-linux-next linus/master v5.8 next-20200810]
[cannot apply to drm-tip/drm-tip drm/drm-next]
[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/0day-ci/linux/commits/allen/IT6505-cover-letter/20200810-181446
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
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
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All error/warnings (new ones prefixed by >>):
drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_calc_video_info':
drivers/gpu/drm/bridge/ite-it6505.c:662:14: warning: variable 'vdes' set but not used [-Wunused-but-set-variable]
662 | int vtotal, vdes, vdew, vfph, vsyncw;
| ^~~~
drivers/gpu/drm/bridge/ite-it6505.c:661:14: warning: variable 'hdes' set but not used [-Wunused-but-set-variable]
661 | int htotal, hdes, hdew, hfph, hsyncw;
| ^~~~
drivers/gpu/drm/bridge/ite-it6505.c:660:28: warning: variable 'interlaced' set but not used [-Wunused-but-set-variable]
660 | int hsync_pol, vsync_pol, interlaced;
| ^~~~~~~~~~
drivers/gpu/drm/bridge/ite-it6505.c:660:17: warning: variable 'vsync_pol' set but not used [-Wunused-but-set-variable]
660 | int hsync_pol, vsync_pol, interlaced;
| ^~~~~~~~~
drivers/gpu/drm/bridge/ite-it6505.c:660:6: warning: variable 'hsync_pol' set but not used [-Wunused-but-set-variable]
660 | int hsync_pol, vsync_pol, interlaced;
| ^~~~~~~~~
In file included from drivers/gpu/drm/bridge/ite-it6505.c:32:
drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_setup_sha1_input':
>> drivers/gpu/drm/bridge/ite-it6505.c:1935:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
1935 | DRM_DEV_DEBUG_DRIVER(dev, "binfo:0x%*ph", ARRAY_SIZE(binfo), binfo);
| ^~~~~~~~~~~~~~
include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER'
386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
| ^~~
drivers/gpu/drm/bridge/ite-it6505.c:1935:38: note: format string is defined here
1935 | DRM_DEV_DEBUG_DRIVER(dev, "binfo:0x%*ph", ARRAY_SIZE(binfo), binfo);
| ~^~
| |
| int
In file included from drivers/gpu/drm/bridge/ite-it6505.c:32:
drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_hdcp_work':
drivers/gpu/drm/bridge/ite-it6505.c:2070:28: warning: field width specifier '*' expects argument of type 'int', but argument 5 has type 'long unsigned int' [-Wformat=]
2070 | DRM_DEV_DEBUG_DRIVER(dev, "ret: %d link_status: %*ph", ret,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
2071 | sizeof(link_status), link_status);
| ~~~~~~~~~~~~~~~~~~~
| |
| long unsigned int
include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER'
386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
| ^~~
drivers/gpu/drm/bridge/ite-it6505.c:2070:51: note: format string is defined here
2070 | DRM_DEV_DEBUG_DRIVER(dev, "ret: %d link_status: %*ph", ret,
| ~^~
| |
| int
In file included from drivers/gpu/drm/bridge/ite-it6505.c:32:
drivers/gpu/drm/bridge/ite-it6505.c:2087:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
2087 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph",
| ^~~~~~~~~~~~~~~
include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER'
386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
| ^~~
drivers/gpu/drm/bridge/ite-it6505.c:2087:39: note: format string is defined here
2087 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph",
| ~^~
| |
| int
drivers/gpu/drm/bridge/ite-it6505.c: At top level:
>> drivers/gpu/drm/bridge/ite-it6505.c:2099:6: warning: no previous prototype for 'it6505_show_hdcp_info' [-Wmissing-prototypes]
2099 | void it6505_show_hdcp_info(struct it6505 *it6505)
| ^~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/bridge/ite-it6505.c:32:
drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_show_hdcp_info':
drivers/gpu/drm/bridge/ite-it6505.c:2107:28: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
2107 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", ARRAY_SIZE(it6505->bksvs),
| ^~~~~~~~~~~~~~~
include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER'
386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
| ^~~
drivers/gpu/drm/bridge/ite-it6505.c:2107:39: note: format string is defined here
2107 | DRM_DEV_DEBUG_DRIVER(dev, "bksv = 0x%*ph", ARRAY_SIZE(it6505->bksvs),
| ~^~
| |
| int
In file included from drivers/gpu/drm/bridge/ite-it6505.c:32:
drivers/gpu/drm/bridge/ite-it6505.c:2113:29: warning: field width specifier '*' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
2113 | DRM_DEV_DEBUG_DRIVER(dev, "sha1_input: 0x%*ph",
| ^~~~~~~~~~~~~~~~~~~~
include/drm/drm_print.h:386:34: note: in definition of macro 'DRM_DEV_DEBUG_DRIVER'
386 | drm_dev_dbg(dev, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
| ^~~
drivers/gpu/drm/bridge/ite-it6505.c:2113:45: note: format string is defined here
2113 | DRM_DEV_DEBUG_DRIVER(dev, "sha1_input: 0x%*ph",
| ~^~
| |
| int
drivers/gpu/drm/bridge/ite-it6505.c: At top level:
>> drivers/gpu/drm/bridge/ite-it6505.c:2965:16: error: initialization of 'enum drm_mode_status (*)(struct drm_bridge *, const struct drm_display_mode *)' from incompatible pointer type 'enum drm_mode_status (*)(struct drm_bridge *, const struct drm_display_info *, const struct drm_display_mode *)' [-Werror=incompatible-pointer-types]
2965 | .mode_valid = it6505_bridge_mode_valid,
| ^~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/bridge/ite-it6505.c:2965:16: note: (near initialization for 'it6505_bridge_funcs.mode_valid')
drivers/gpu/drm/bridge/ite-it6505.c: In function 'it6505_i2c_probe':
drivers/gpu/drm/bridge/ite-it6505.c:3140:31: warning: variable 'pdata' set but not used [-Wunused-but-set-variable]
3140 | struct it6505_platform_data *pdata;
| ^~~~~
cc1: some warnings being treated as errors
vim +2965 drivers/gpu/drm/bridge/ite-it6505.c
2961
2962 static const struct drm_bridge_funcs it6505_bridge_funcs = {
2963 .attach = it6505_bridge_attach,
2964 .detach = it6505_bridge_detach,
> 2965 .mode_valid = it6505_bridge_mode_valid,
2966 .mode_set = it6505_bridge_mode_set,
2967 .enable = it6505_bridge_enable,
2968 .disable = it6505_bridge_disable,
2969 };
2970
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 69750 bytes --]
next prev parent reply other threads:[~2020-08-10 12:53 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-10 10:11 [PATCH v1 0/2] IT6505 cover letter allen
2020-08-10 10:11 ` allen
2020-08-10 10:11 ` allen
2020-08-10 10:11 ` allen
2020-08-10 10:11 ` [PATCH v1 1/2] ite-it6505 change trigger conditions allen
2020-08-10 10:11 ` allen
2020-08-24 22:32 ` Rob Herring
2020-08-24 22:32 ` Rob Herring
2020-08-10 10:11 ` [PATCH v1 2/2] drm/bridge: add refactored it6505 driver allen
2020-08-10 10:11 ` allen
2020-08-10 10:11 ` allen
2020-08-10 10:11 ` allen
2020-08-10 12:53 ` kernel test robot [this message]
2020-08-10 14:46 ` Randy Dunlap
2020-08-10 14:46 ` Randy Dunlap
2020-08-10 14:46 ` Randy Dunlap
2020-08-10 14:46 ` Randy Dunlap
-- strict thread matches above, loose matches on Subject: below --
2020-08-10 9:53 [PATCH v1 0/2] IT6505 cover letter allen
2020-08-10 9:53 ` [PATCH v1 2/2] drm/bridge: add refactored it6505 driver allen
2020-08-10 9:53 ` allen
2020-08-10 9:53 ` allen
2020-08-10 9:53 ` allen
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=202008102021.WKLURcQx%lkp@intel.com \
--to=lkp@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 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.