Linux-Next Archive on lore.kernel.org
 help / color / Atom feed
* linux-next: build warning after merge of the pinctrl tree
@ 2021-03-30  6:07 Stephen Rothwell
  2021-03-30  7:51 ` Linus Walleij
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2021-03-30  6:07 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Jonas Gorski, Álvaro Fernández Rojas,
	Linux Kernel Mailing List, Linux Next Mailing List


[-- Attachment #1: Type: text/plain, Size: 505 bytes --]

Hi all,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/pinctrl/bcm/pinctrl-bcm6362.c: In function 'bcm6362_set_gpio':
drivers/pinctrl/bcm/pinctrl-bcm6362.c:503:8: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  503 |        (uint32_t) desc->drv_data, 0);
      |        ^

Introduced by commit

  705791e23ecd ("pinctrl: add a pincontrol driver for BCM6362")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2021-03-30  6:07 linux-next: build warning after merge of the pinctrl tree Stephen Rothwell
@ 2021-03-30  7:51 ` Linus Walleij
  0 siblings, 0 replies; 16+ messages in thread
From: Linus Walleij @ 2021-03-30  7:51 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Jonas Gorski, Álvaro Fernández Rojas,
	Linux Kernel Mailing List, Linux Next Mailing List

On Tue, Mar 30, 2021 at 8:07 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> drivers/pinctrl/bcm/pinctrl-bcm6362.c: In function 'bcm6362_set_gpio':
> drivers/pinctrl/bcm/pinctrl-bcm6362.c:503:8: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
>   503 |        (uint32_t) desc->drv_data, 0);
>       |        ^
>
> Introduced by commit
>
>   705791e23ecd ("pinctrl: add a pincontrol driver for BCM6362")

Ha! The 0day robot didn't see it. Good that we have so many layers.

I bet Álvaro will have a patch to fix it in no time.

Yours,
Linus Walleij

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2020-04-17  2:27 Stephen Rothwell
@ 2020-04-17 10:23 ` Linus Walleij
  0 siblings, 0 replies; 16+ messages in thread
From: Linus Walleij @ 2020-04-17 10:23 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Andy Shevchenko

On Fri, Apr 17, 2020 at 4:28 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> WARNING: modpost: missing MODULE_LICENSE() in drivers/pinctrl/pinctrl-mcp23s08.o
> see include/linux/module.h for more information
>
> Introduced by commit
>
>   0f04a81784fe ("pinctrl: mcp23s08: Split to three parts: core, I²C, SPI")

Thanks I got a patch to fix it up and applied.

Yours,
Linus Walleij

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

* linux-next: build warning after merge of the pinctrl tree
@ 2020-04-17  2:27 Stephen Rothwell
  2020-04-17 10:23 ` Linus Walleij
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2020-04-17  2:27 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Andy Shevchenko


[-- Attachment #1: Type: text/plain, Size: 385 bytes --]

Hi all,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

WARNING: modpost: missing MODULE_LICENSE() in drivers/pinctrl/pinctrl-mcp23s08.o
see include/linux/module.h for more information

Introduced by commit

  0f04a81784fe ("pinctrl: mcp23s08: Split to three parts: core, I²C, SPI")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2018-10-11  3:17 Stephen Rothwell
@ 2018-10-11  7:51 ` Linus Walleij
  0 siblings, 0 replies; 16+ messages in thread
From: Linus Walleij @ 2018-10-11  7:51 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Linux-Next Mailing List, linux-kernel, Rafał Miłecki

On Thu, Oct 11, 2018 at 5:17 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> drivers/pinctrl/bcm/pinctrl-ns.c: In function 'ns_pinctrl_probe':
> drivers/pinctrl/bcm/pinctrl-ns.c:288:29: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
>   ns_pinctrl->chipset_flag = (unsigned int)of_id->data;
>                              ^
> drivers/pinctrl/bcm/pinctrl-ns.c:310:27: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
>    unsigned int chipsets = (unsigned int)src->drv_data;
>                            ^

I tried to fix it, stumbled, not I tried to fix it again.
Let's see if I got it right now!

Thanks,
Linus Walleij

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

* linux-next: build warning after merge of the pinctrl tree
@ 2018-10-11  3:17 Stephen Rothwell
  2018-10-11  7:51 ` Linus Walleij
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2018-10-11  3:17 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Rafał Miłecki


[-- Attachment #1: Type: text/plain, Size: 721 bytes --]

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/pinctrl/bcm/pinctrl-ns.c: In function 'ns_pinctrl_probe':
drivers/pinctrl/bcm/pinctrl-ns.c:288:29: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  ns_pinctrl->chipset_flag = (unsigned int)of_id->data;
                             ^
drivers/pinctrl/bcm/pinctrl-ns.c:310:27: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
   unsigned int chipsets = (unsigned int)src->drv_data;
                           ^

Introduced by commit

  c12fb1774dea ("pinctrl: bcm: add Northstar driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2017-06-30  6:59 ` Geert Uytterhoeven
@ 2017-06-30 13:48   ` Linus Walleij
  0 siblings, 0 replies; 16+ messages in thread
From: Linus Walleij @ 2017-06-30 13:48 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Stephen Rothwell, Jacopo Mondi, Linux-Next Mailing List,
	Linux Kernel Mailing List

On Fri, Jun 30, 2017 at 8:59 AM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Fri, Jun 30, 2017 at 6:56 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> After merging the pinctrl tree, today's linux-next build (x86_64
>> allmodconfig) produced this warning:
>>
>> drivers/pinctrl/pinctrl-rza1.c: In function 'rza1_pinctrl_probe':
>> drivers/pinctrl/pinctrl-rza1.c:1260:5: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
>>   if (ret)
>>      ^
>>
>> Introduced by commit
>>
>>   5a49b644b307 ("pinctrl: Renesas RZ/A1 pin and gpio controller")
>
> Doh, 0day doesn't use compilers that report uninitialized variables?

The real problem is that 0day is not working right now.

I only get timeouts from it for the moment.

Yours,
Linus Walleij

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2017-06-30  4:56 Stephen Rothwell
  2017-06-30  4:57 ` Stephen Rothwell
@ 2017-06-30  6:59 ` Geert Uytterhoeven
  2017-06-30 13:48   ` Linus Walleij
  1 sibling, 1 reply; 16+ messages in thread
From: Geert Uytterhoeven @ 2017-06-30  6:59 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Linus Walleij, Jacopo Mondi, Linux-Next Mailing List,
	Linux Kernel Mailing List

Hi Stephen,

On Fri, Jun 30, 2017 at 6:56 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> drivers/pinctrl/pinctrl-rza1.c: In function 'rza1_pinctrl_probe':
> drivers/pinctrl/pinctrl-rza1.c:1260:5: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
>   if (ret)
>      ^
>
> Introduced by commit
>
>   5a49b644b307 ("pinctrl: Renesas RZ/A1 pin and gpio controller")

Doh, 0day doesn't use compilers that report uninitialized variables?
Patch sent.

Thanks for reporting!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2017-06-30  4:56 Stephen Rothwell
@ 2017-06-30  4:57 ` Stephen Rothwell
  2017-06-30  6:59 ` Geert Uytterhoeven
  1 sibling, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2017-06-30  4:57 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Jacopo Mondi,
	Geert Uytterhoeven

Hi all,

[Just adding cc's]

On Fri, 30 Jun 2017 14:56:01 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
> 
> drivers/pinctrl/pinctrl-rza1.c: In function 'rza1_pinctrl_probe':
> drivers/pinctrl/pinctrl-rza1.c:1260:5: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
>   if (ret)
>      ^
> 
> Introduced by commit
> 
>   5a49b644b307 ("pinctrl: Renesas RZ/A1 pin and gpio controller")

-- 
Cheers,
Stephen Rothwell

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

* linux-next: build warning after merge of the pinctrl tree
@ 2017-06-30  4:56 Stephen Rothwell
  2017-06-30  4:57 ` Stephen Rothwell
  2017-06-30  6:59 ` Geert Uytterhoeven
  0 siblings, 2 replies; 16+ messages in thread
From: Stephen Rothwell @ 2017-06-30  4:56 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Linux-Next Mailing List, Linux Kernel Mailing List

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/pinctrl/pinctrl-rza1.c: In function 'rza1_pinctrl_probe':
drivers/pinctrl/pinctrl-rza1.c:1260:5: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
  if (ret)
     ^

Introduced by commit

  5a49b644b307 ("pinctrl: Renesas RZ/A1 pin and gpio controller")

-- 
Cheers,
Stephen Rothwell

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

* linux-next: build warning after merge of the pinctrl tree
@ 2017-01-03  4:37 Stephen Rothwell
  0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2017-01-03  4:37 UTC (permalink / raw)
  To: Linus Walleij; +Cc: linux-next, linux-kernel, Nehal Shah

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/pinctrl/pinctrl-amd.c: In function 'amd_gpio_dbg_show':
drivers/pinctrl/pinctrl-amd.c:210:3: warning: 'pin_num' may be used uninitialized in this function [-Wmaybe-uninitialized]
   for (; i < pin_num; i++) {
   ^
drivers/pinctrl/pinctrl-amd.c:172:21: warning: 'i' may be used uninitialized in this function [-Wmaybe-uninitialized]
  unsigned int bank, i, pin_num;
                     ^

Introduced by commit

  3bfd44306c65 ("pinctrl: amd: Add support for additional GPIO")

-- 
Cheers,
Stephen Rothwell

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2015-05-20  6:40 Stephen Rothwell
@ 2015-05-20  7:14 ` Linus Walleij
  0 siblings, 0 replies; 16+ messages in thread
From: Linus Walleij @ 2015-05-20  7:14 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Hongzhou Yang

On Wed, May 20, 2015 at 8:40 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi Linus,
>
> After merging the pinctrl tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
>
> warning: (PINCTRL_MT6397) selects PINCTRL_MTK_COMMON which has unmet direct dependencies (PINCTRL && (ARCH_MEDIATEK || COMPILE_TEST) && OF)
>
> Introduced by commit fc59e66c4284 ("pinctrl: mediatek: Add Pinctrl/GPIO
> driver for mt6397").

Ah too bad I didn't see it before pushing, fixed it now, thanks!

Yours,
Linus Walleij

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

* linux-next: build warning after merge of the pinctrl tree
@ 2015-05-20  6:40 Stephen Rothwell
  2015-05-20  7:14 ` Linus Walleij
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2015-05-20  6:40 UTC (permalink / raw)
  To: Linus Walleij; +Cc: linux-next, linux-kernel, Hongzhou Yang


[-- Attachment #1: Type: text/plain, Size: 429 bytes --]

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64 allmodconfig)
produced this warning:

warning: (PINCTRL_MT6397) selects PINCTRL_MTK_COMMON which has unmet direct dependencies (PINCTRL && (ARCH_MEDIATEK || COMPILE_TEST) && OF)

Introduced by commit fc59e66c4284 ("pinctrl: mediatek: Add Pinctrl/GPIO
driver for mt6397").
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* Re: linux-next: build warning after merge of the pinctrl tree
  2014-05-19  6:59 Stephen Rothwell
@ 2014-05-19  9:03 ` Maxime Ripard
  0 siblings, 0 replies; 16+ messages in thread
From: Maxime Ripard @ 2014-05-19  9:03 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: Linus Walleij, linux-next, linux-kernel


[-- Attachment #1: Type: text/plain, Size: 863 bytes --]

Hi Stephen,

On Mon, May 19, 2014 at 04:59:47PM +1000, Stephen Rothwell wrote:
> Hi Linus,
> 
> After merging the pinctrl tree, today's linux-next build (x86_64
> allmodconfig among others) produced this warning:
> 
> drivers/pinctrl/sunxi/Kconfig:3:error: recursive dependency detected!
> drivers/pinctrl/sunxi/Kconfig:3:	symbol PINCTRL_SUNXI is selected by PINCTRL_SUN4I_A10
> drivers/pinctrl/sunxi/Kconfig:9:	symbol PINCTRL_SUN4I_A10 default value contains PINCTRL_SUNXI
> 
> Introduced by commit 3b582f7e1542 ("pinctrl: sunxi: Enable the pinctrl
> Kconfig options by default").

This should be fixed with the patch I sent on saturday:
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-May/257411.html

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* linux-next: build warning after merge of the pinctrl tree
@ 2014-05-19  6:59 Stephen Rothwell
  2014-05-19  9:03 ` Maxime Ripard
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2014-05-19  6:59 UTC (permalink / raw)
  To: Linus Walleij; +Cc: linux-next, linux-kernel, Maxime Ripard


[-- Attachment #1: Type: text/plain, Size: 566 bytes --]

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig among others) produced this warning:

drivers/pinctrl/sunxi/Kconfig:3:error: recursive dependency detected!
drivers/pinctrl/sunxi/Kconfig:3:	symbol PINCTRL_SUNXI is selected by PINCTRL_SUN4I_A10
drivers/pinctrl/sunxi/Kconfig:9:	symbol PINCTRL_SUN4I_A10 default value contains PINCTRL_SUNXI

Introduced by commit 3b582f7e1542 ("pinctrl: sunxi: Enable the pinctrl
Kconfig options by default").

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

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

* linux-next: build warning after merge of the pinctrl tree
@ 2013-12-13  4:06 Stephen Rothwell
  0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2013-12-13  4:06 UTC (permalink / raw)
  To: Linus Walleij; +Cc: linux-next, linux-kernel, Alexandre Belloni


[-- Attachment #1: Type: text/plain, Size: 767 bytes --]

Hi Linus,

After merging the pinctrl tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/pinctrl/pinconf.c: In function 'pinconf_pins_show':
drivers/pinctrl/pinconf.c:299:28: warning: unused variable 'ops' [-Wunused-variable]
  const struct pinconf_ops *ops = pctldev->desc->confops;
                            ^
drivers/pinctrl/pinconf.c: In function 'pinconf_groups_show':
drivers/pinctrl/pinconf.c:346:28: warning: unused variable 'ops' [-Wunused-variable]
  const struct pinconf_ops *ops = pctldev->desc->confops;
                            ^

Introduced by commit c420619d5177 ("pinctrl: pinconf: remove checks on
ops->pin_config_get").

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

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

end of thread, back to index

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-30  6:07 linux-next: build warning after merge of the pinctrl tree Stephen Rothwell
2021-03-30  7:51 ` Linus Walleij
  -- strict thread matches above, loose matches on Subject: below --
2020-04-17  2:27 Stephen Rothwell
2020-04-17 10:23 ` Linus Walleij
2018-10-11  3:17 Stephen Rothwell
2018-10-11  7:51 ` Linus Walleij
2017-06-30  4:56 Stephen Rothwell
2017-06-30  4:57 ` Stephen Rothwell
2017-06-30  6:59 ` Geert Uytterhoeven
2017-06-30 13:48   ` Linus Walleij
2017-01-03  4:37 Stephen Rothwell
2015-05-20  6:40 Stephen Rothwell
2015-05-20  7:14 ` Linus Walleij
2014-05-19  6:59 Stephen Rothwell
2014-05-19  9:03 ` Maxime Ripard
2013-12-13  4:06 Stephen Rothwell

Linux-Next Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-next/0 linux-next/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-next linux-next/ https://lore.kernel.org/linux-next \
		linux-next@vger.kernel.org
	public-inbox-index linux-next

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-next


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git