Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1678472 > unrolled thread

linux-next: build warning after merge of the pinctrl tree

Started byStephen Rothwell <sfr@canb.auug.org.au>
First post2017-06-30 07:00 +0200
Last post2017-06-30 15:50 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  linux-next: build warning after merge of the pinctrl tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-30 07:00 +0200
    Re: linux-next: build warning after merge of the pinctrl tree Geert Uytterhoeven <geert@linux-m68k.org> - 2017-06-30 09:00 +0200
      Re: linux-next: build warning after merge of the pinctrl tree Linus Walleij <linus.walleij@linaro.org> - 2017-06-30 15:50 +0200

#1678472 — linux-next: build warning after merge of the pinctrl tree

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-30 07:00 +0200
Subjectlinux-next: build warning after merge of the pinctrl tree
Message-ID<tXWc9-KG-3@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1678534

FromGeert Uytterhoeven <geert@linux-m68k.org>
Date2017-06-30 09:00 +0200
Message-ID<tXY4h-23t-3@gated-at.bofh.it>
In reply to#1678472
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

[toc] | [prev] | [next] | [standalone]


#1678841

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-06-30 15:50 +0200
Message-ID<tY4t5-682-17@gated-at.bofh.it>
In reply to#1678534
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web