llvm.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO
       [not found] <1234b2b7105443654c5f2bb97d25cf16408d6003.1756727747.git.christophe.leroy@csgroup.eu>
@ 2025-09-02 10:50 ` kernel test robot
  2025-09-02 17:28   ` Christophe Leroy
  0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2025-09-02 10:50 UTC (permalink / raw)
  To: Christophe Leroy; +Cc: llvm, oe-kbuild-all

Hi Christophe,

kernel test robot noticed the following build warnings:

[auto build test WARNING on robh/for-next]
[also build test WARNING on powerpc/next powerpc/fixes linus/master v6.17-rc4]
[cannot apply to next-20250902]
[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/Christophe-Leroy/soc-fsl-qe-Add-an-interrupt-controller-for-QUICC-Engine-Ports/20250901-202447
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link:    https://lore.kernel.org/r/1234b2b7105443654c5f2bb97d25cf16408d6003.1756727747.git.christophe.leroy%40csgroup.eu
patch subject: [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO
config: powerpc64-randconfig-003-20250902 (https://download.01.org/0day-ci/archive/20250902/202509021830.uakzVIlK-lkp@intel.com/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 2e122990391b2ba062e6308a12cfedf7206270ba)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250902/202509021830.uakzVIlK-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/202509021830.uakzVIlK-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from <built-in>:3:
   In file included from include/linux/compiler_types.h:171:
   include/linux/compiler-clang.h:37:9: warning: '__SANITIZE_THREAD__' macro redefined [-Wmacro-redefined]
      37 | #define __SANITIZE_THREAD__
         |         ^
   <built-in>:352:9: note: previous definition is here
     352 | #define __SANITIZE_THREAD__ 1
         |         ^
>> drivers/soc/fsl/qe/gpio.c:316:9: warning: cast to smaller integer type 'u32' (aka 'unsigned int') from 'const void *' [-Wvoid-pointer-to-int-cast]
     316 |         mask = (u32)of_device_get_match_data(dev);
         |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   2 warnings generated.


vim +316 drivers/soc/fsl/qe/gpio.c

   301	
   302	static int qe_gpio_probe(struct platform_device *ofdev)
   303	{
   304		struct device *dev = &ofdev->dev;
   305		struct device_node *np = dev->of_node;
   306		struct qe_gpio_chip *qe_gc;
   307		struct gpio_chip *gc;
   308		u32 mask;
   309	
   310		qe_gc = devm_kzalloc(dev, sizeof(*qe_gc), GFP_KERNEL);
   311		if (!qe_gc)
   312			return -ENOMEM;
   313	
   314		spin_lock_init(&qe_gc->lock);
   315	
 > 316		mask = (u32)of_device_get_match_data(dev);
   317		if (mask) {
   318			int i, j;
   319	
   320			for (i = 0, j = 0; i < ARRAY_SIZE(qe_gc->irq); i++)
   321				if (mask & PIN_MASK(i))
   322					qe_gc->irq[i] = irq_of_parse_and_map(np, j++);
   323		}
   324	
   325		gc = &qe_gc->gc;
   326	
   327		gc->base = -1;
   328		gc->ngpio = QE_PIO_PINS;
   329		gc->direction_input = qe_gpio_dir_in;
   330		gc->direction_output = qe_gpio_dir_out;
   331		gc->get = qe_gpio_get;
   332		gc->set = qe_gpio_set;
   333		gc->set_multiple = qe_gpio_set_multiple;
   334		gc->to_irq = qe_gpio_to_irq;
   335		gc->parent = dev;
   336		gc->owner = THIS_MODULE;
   337	
   338		gc->label = devm_kasprintf(dev, GFP_KERNEL, "%pOF", np);
   339		if (!gc->label)
   340			return -ENOMEM;
   341	
   342		qe_gc->regs = devm_of_iomap(dev, np, 0, NULL);
   343		if (IS_ERR(qe_gc->regs))
   344			return PTR_ERR(qe_gc->regs);
   345	
   346		qe_gpio_save_regs(qe_gc);
   347	
   348		return devm_gpiochip_add_data(dev, gc, qe_gc);
   349	}
   350	

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO
  2025-09-02 10:50 ` [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO kernel test robot
@ 2025-09-02 17:28   ` Christophe Leroy
  0 siblings, 0 replies; 2+ messages in thread
From: Christophe Leroy @ 2025-09-02 17:28 UTC (permalink / raw)
  To: kernel test robot; +Cc: llvm, oe-kbuild-all



Le 02/09/2025 à 12:50, kernel test robot a écrit :
> Hi Christophe,
> 
> kernel test robot noticed the following build warnings:
> 
> [auto build test WARNING on robh/for-next]
> [also build test WARNING on powerpc/next powerpc/fixes linus/master v6.17-rc4]
> [cannot apply to next-20250902]
> [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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit-scm.com%2Fdocs%2Fgit-format-patch%23_base_tree_information&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075803525%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=%2BniURYrufRzhFQ7KXwgpmnJIfeJWcTMavXkEndbtHYA%3D&reserved=0]
> 
> url:    https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fintel-lab-lkp%2Flinux%2Fcommits%2FChristophe-Leroy%2Fsoc-fsl-qe-Add-an-interrupt-controller-for-QUICC-Engine-Ports%2F20250901-202447&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075825139%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=v43MiEVTfpflIobzhrSvzs4IZcmfHuSrXyUJHafSLNk%3D&reserved=0
> base:   https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Frobh%2Flinux.git&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075840967%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=g1r1XaGeDG5nZqIU1b61fCZlve9VQaMIliHoufnXFLU%3D&reserved=0 for-next
> patch link:    https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.kernel.org%2Fr%2F1234b2b7105443654c5f2bb97d25cf16408d6003.1756727747.git.christophe.leroy%2540csgroup.eu&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075856717%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=BWYYNgbEu2UZWOS8XRlq%2FdGddmBDeQ%2BjDb%2FIorMXSmc%3D&reserved=0
> patch subject: [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO
> config: powerpc64-randconfig-003-20250902 (https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdownload.01.org%2F0day-ci%2Farchive%2F20250902%2F202509021830.uakzVIlK-lkp%40intel.com%2Fconfig&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075872388%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=M6TMS23JmgSVi3aQZDvjZcOn9vX7HyUc8%2BcoLOjfN6E%3D&reserved=0)
> compiler: clang version 22.0.0git (https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fllvm%2Fllvm-project&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075888080%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=3uIHxd8uhuDUbQRCQbcW%2BFnR1uvJZeN3AQWOR039L7A%3D&reserved=0 2e122990391b2ba062e6308a12cfedf7206270ba)
> reproduce (this is a W=1 build): (https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdownload.01.org%2F0day-ci%2Farchive%2F20250902%2F202509021830.uakzVIlK-lkp%40intel.com%2Freproduce&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075903628%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=y13g2bKm6FrjWSiErSO83akrf3BclbaIQ4142F9f5Tk%3D&reserved=0)
> 
> 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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.kernel.org%2Foe-kbuild-all%2F202509021830.uakzVIlK-lkp%40intel.com%2F&data=05%7C02%7Cchristophe.leroy%40csgroup.eu%7Ca3ddcbd613fd48271bba08ddea0eb403%7C8b87af7d86474dc78df45f69a2011bb5%7C0%7C0%7C638924071075921207%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=j%2BMDWBvsHHtgzjiHc83gr3OguP22jVJ41AmVgFtT6eM%3D&reserved=0
> 
> All warnings (new ones prefixed by >>):
> 
>     In file included from <built-in>:3:
>     In file included from include/linux/compiler_types.h:171:
>     include/linux/compiler-clang.h:37:9: warning: '__SANITIZE_THREAD__' macro redefined [-Wmacro-redefined]
>        37 | #define __SANITIZE_THREAD__
>           |         ^
>     <built-in>:352:9: note: previous definition is here
>       352 | #define __SANITIZE_THREAD__ 1
>           |         ^
>>> drivers/soc/fsl/qe/gpio.c:316:9: warning: cast to smaller integer type 'u32' (aka 'unsigned int') from 'const void *' [-Wvoid-pointer-to-int-cast]
>       316 |         mask = (u32)of_device_get_match_data(dev);
>           |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>     2 warnings generated.
> 


Will be fixed with the following change in the patch:

diff --git a/drivers/soc/fsl/qe/gpio.c b/drivers/soc/fsl/qe/gpio.c
index 8632b0d37255..7a23eb220217 100644
--- a/drivers/soc/fsl/qe/gpio.c
+++ b/drivers/soc/fsl/qe/gpio.c
@@ -305,7 +305,7 @@ static int qe_gpio_probe(struct platform_device *ofdev)
  	struct device_node *np = dev->of_node;
  	struct qe_gpio_chip *qe_gc;
  	struct gpio_chip *gc;
-	u32 mask;
+	unsigned long mask;

  	qe_gc = devm_kzalloc(dev, sizeof(*qe_gc), GFP_KERNEL);
  	if (!qe_gc)
@@ -313,7 +313,7 @@ static int qe_gpio_probe(struct platform_device *ofdev)

  	spin_lock_init(&qe_gc->lock);

-	mask = (u32)of_device_get_match_data(dev);
+	mask = (unsigned long)of_device_get_match_data(dev);
  	if (mask) {
  		int i, j;



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-09-02 17:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1234b2b7105443654c5f2bb97d25cf16408d6003.1756727747.git.christophe.leroy@csgroup.eu>
2025-09-02 10:50 ` [PATCH v5 4/7] soc: fsl: qe: Add support of IRQ in QE GPIO kernel test robot
2025-09-02 17:28   ` Christophe Leroy

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).