All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Louis-Alexis Eyraud <louisalexis.eyraud@collabora.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [PATCH v11 09/11] drm/mediatek: Introduce HDMI/DDC v2 for MT8195/MT8188
Date: Sun, 26 Oct 2025 03:04:52 +0800	[thread overview]
Message-ID: <202510260233.dr87FI0R-lkp@intel.com> (raw)
In-Reply-To: <20251023-mediatek-drm-hdmi-v2-v11-9-7873ec4a1edf@collabora.com>

Hi Louis-Alexis,

kernel test robot noticed the following build warnings:

[auto build test WARNING on a92c761bcac3d5042559107fa7679470727a4bcb]

url:    https://github.com/intel-lab-lkp/linux/commits/Louis-Alexis-Eyraud/drm-mediatek-mtk_hdmi-Drop-redundant-clock-retrieval-in-mtk_hdmi_get_cec_dev/20251023-183509
base:   a92c761bcac3d5042559107fa7679470727a4bcb
patch link:    https://lore.kernel.org/r/20251023-mediatek-drm-hdmi-v2-v11-9-7873ec4a1edf%40collabora.com
patch subject: [PATCH v11 09/11] drm/mediatek: Introduce HDMI/DDC v2 for MT8195/MT8188
config: microblaze-allmodconfig (https://download.01.org/0day-ci/archive/20251026/202510260233.dr87FI0R-lkp@intel.com/config)
compiler: microblaze-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251026/202510260233.dr87FI0R-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202510260233.dr87FI0R-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from ./arch/microblaze/include/generated/asm/div64.h:1,
                    from include/linux/math.h:6,
                    from include/linux/kernel.h:27,
                    from include/linux/clk.h:13,
                    from drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c:12:
   drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c: In function 'mtk_ddcm_read_hdmi':
   include/asm-generic/div64.h:183:35: warning: comparison of distinct pointer types lacks a cast [-Wcompare-distinct-pointer-types]
     183 |         (void)(((typeof((n)) *)0) == ((uint64_t *)0));  \
         |                                   ^~
   drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   In file included from include/linux/build_bug.h:5,
                    from include/linux/bitfield.h:10,
                    from drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c:11:
>> include/asm-generic/div64.h:195:32: warning: right shift count >= width of type [-Wshift-count-overflow]
     195 |         } else if (likely(((n) >> 32) == 0)) {          \
         |                                ^~
   include/linux/compiler.h:76:45: note: in definition of macro 'likely'
      76 | # define likely(x)      __builtin_expect(!!(x), 1)
         |                                             ^
   drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   include/asm-generic/div64.h:199:36: error: passing argument 1 of '__div64_32' from incompatible pointer type [-Wincompatible-pointer-types]
     199 |                 __rem = __div64_32(&(n), __base);       \
         |                                    ^~~~
         |                                    |
         |                                    u16 * {aka short unsigned int *}
   drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   include/asm-generic/div64.h:174:38: note: expected 'uint64_t *' {aka 'long long unsigned int *'} but argument is of type 'u16 *' {aka 'short unsigned int *'}
     174 | extern uint32_t __div64_32(uint64_t *dividend, uint32_t divisor);
         |                            ~~~~~~~~~~^~~~~~~~
--
   In file included from arch/microblaze/include/generated/asm/div64.h:1,
                    from include/linux/math.h:6,
                    from include/linux/kernel.h:27,
                    from include/linux/clk.h:13,
                    from mtk_hdmi_ddc_v2.c:12:
   mtk_hdmi_ddc_v2.c: In function 'mtk_ddcm_read_hdmi':
   include/asm-generic/div64.h:183:35: warning: comparison of distinct pointer types lacks a cast [-Wcompare-distinct-pointer-types]
     183 |         (void)(((typeof((n)) *)0) == ((uint64_t *)0));  \
         |                                   ^~
   mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   In file included from include/linux/build_bug.h:5,
                    from include/linux/bitfield.h:10,
                    from mtk_hdmi_ddc_v2.c:11:
>> include/asm-generic/div64.h:195:32: warning: right shift count >= width of type [-Wshift-count-overflow]
     195 |         } else if (likely(((n) >> 32) == 0)) {          \
         |                                ^~
   include/linux/compiler.h:76:45: note: in definition of macro 'likely'
      76 | # define likely(x)      __builtin_expect(!!(x), 1)
         |                                             ^
   mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   include/asm-generic/div64.h:199:36: error: passing argument 1 of '__div64_32' from incompatible pointer type [-Wincompatible-pointer-types]
     199 |                 __rem = __div64_32(&(n), __base);       \
         |                                    ^~~~
         |                                    |
         |                                    u16 * {aka short unsigned int *}
   mtk_hdmi_ddc_v2.c:128:23: note: in expansion of macro 'do_div'
     128 |                 rem = do_div(loop_counter, temp_length);
         |                       ^~~~~~
   include/asm-generic/div64.h:174:38: note: expected 'uint64_t *' {aka 'long long unsigned int *'} but argument is of type 'u16 *' {aka 'short unsigned int *'}
     174 | extern uint32_t __div64_32(uint64_t *dividend, uint32_t divisor);
         |                            ~~~~~~~~~~^~~~~~~~


vim +195 include/asm-generic/div64.h

^1da177e4c3f41 Linus Torvalds     2005-04-16  176  
^1da177e4c3f41 Linus Torvalds     2005-04-16  177  /* The unnecessary pointer compare is there
^1da177e4c3f41 Linus Torvalds     2005-04-16  178   * to check for type safety (n must be 64bit)
^1da177e4c3f41 Linus Torvalds     2005-04-16  179   */
^1da177e4c3f41 Linus Torvalds     2005-04-16  180  # define do_div(n,base) ({				\
^1da177e4c3f41 Linus Torvalds     2005-04-16  181  	uint32_t __base = (base);			\
^1da177e4c3f41 Linus Torvalds     2005-04-16  182  	uint32_t __rem;					\
^1da177e4c3f41 Linus Torvalds     2005-04-16 @183  	(void)(((typeof((n)) *)0) == ((uint64_t *)0));	\
911918aa7ef6f8 Nicolas Pitre      2015-11-02  184  	if (__builtin_constant_p(__base) &&		\
911918aa7ef6f8 Nicolas Pitre      2015-11-02  185  	    is_power_of_2(__base)) {			\
911918aa7ef6f8 Nicolas Pitre      2015-11-02  186  		__rem = (n) & (__base - 1);		\
911918aa7ef6f8 Nicolas Pitre      2015-11-02  187  		(n) >>= ilog2(__base);			\
c747ce4706190e Geert Uytterhoeven 2021-08-11  188  	} else if (__builtin_constant_p(__base) &&	\
461a5e51060c93 Nicolas Pitre      2015-10-30  189  		   __base != 0) {			\
461a5e51060c93 Nicolas Pitre      2015-10-30  190  		uint32_t __res_lo, __n_lo = (n);	\
461a5e51060c93 Nicolas Pitre      2015-10-30  191  		(n) = __div64_const32(n, __base);	\
461a5e51060c93 Nicolas Pitre      2015-10-30  192  		/* the remainder can be computed with 32-bit regs */ \
461a5e51060c93 Nicolas Pitre      2015-10-30  193  		__res_lo = (n);				\
461a5e51060c93 Nicolas Pitre      2015-10-30  194  		__rem = __n_lo - __res_lo * __base;	\
911918aa7ef6f8 Nicolas Pitre      2015-11-02 @195  	} else if (likely(((n) >> 32) == 0)) {		\
^1da177e4c3f41 Linus Torvalds     2005-04-16  196  		__rem = (uint32_t)(n) % __base;		\
^1da177e4c3f41 Linus Torvalds     2005-04-16  197  		(n) = (uint32_t)(n) / __base;		\
c747ce4706190e Geert Uytterhoeven 2021-08-11  198  	} else {					\
^1da177e4c3f41 Linus Torvalds     2005-04-16  199  		__rem = __div64_32(&(n), __base);	\
c747ce4706190e Geert Uytterhoeven 2021-08-11  200  	}						\
^1da177e4c3f41 Linus Torvalds     2005-04-16  201  	__rem;						\
^1da177e4c3f41 Linus Torvalds     2005-04-16  202   })
^1da177e4c3f41 Linus Torvalds     2005-04-16  203  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  parent reply	other threads:[~2025-10-25 19:06 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-23 10:32 [PATCH v11 00/11] Add support for MT8195/88 HDMIv2 and DDCv2 Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 01/11] drm/mediatek: mtk_hdmi: Drop redundant clock retrieval in mtk_hdmi_get_cec_dev Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 02/11] drm/mediatek: mtk_hdmi: Improve mtk_hdmi_get_all_clk() flexibility Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 03/11] drm/mediatek: mtk_hdmi: Add HDMI IP version configuration to pdata Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 04/11] drm/mediatek: mtk_hdmi: Split driver and add common probe function Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 05/11] drm/mediatek: mtk_hdmi_common: Make CEC support optional Louis-Alexis Eyraud
2025-10-23 15:59   ` Chun-Kuang Hu
2025-10-23 17:05     ` Louis-Alexis Eyraud
2025-10-24  0:35       ` Chun-Kuang Hu
2025-10-23 10:32 ` [PATCH v11 06/11] drm/mediatek: mtk_hdmi_common: Assign DDC adapter pointer to bridge Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 07/11] drm/mediatek: mtk_hdmi_common: Add OP_HDMI if helper funcs assigned Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 08/11] drm/mediatek: mtk_hdmi_common: Add var to enable interlaced modes Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 09/11] drm/mediatek: Introduce HDMI/DDC v2 for MT8195/MT8188 Louis-Alexis Eyraud
2025-10-24  1:05   ` Chun-Kuang Hu
2025-10-24  7:38     ` AngeloGioacchino Del Regno
2025-10-25 19:04   ` kernel test robot [this message]
2025-10-23 10:32 ` [PATCH v11 10/11] drm/mediatek: mtk_hdmi_v2: Add debugfs ops and implement ABIST Louis-Alexis Eyraud
2025-10-23 10:32 ` [PATCH v11 11/11] drm/mediatek: mtk_hdmi_common: Defer probe when ddc i2c bus isn't available yet Louis-Alexis Eyraud
2025-10-24 12:39 ` [PATCH v11 00/11] Add support for MT8195/88 HDMIv2 and DDCv2 Chun-Kuang Hu

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=202510260233.dr87FI0R-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=louisalexis.eyraud@collabora.com \
    --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.