2019-06-17 04:53:16

by Stephen Rothwell

[permalink] [raw]

Subject: linux-next: build warnings after merge of the clockevents tree

---

Hi all,

After merging the clockevents tree, today's linux-next build (arm
multi_v7_defconfig) produced these warnings:

In file included from arch/arm/kernel/vdso.c:30:
arch/arm/include/asm/arch_timer.h: In function 'arch_timer_set_evtstrm_feature':
arch/arm/include/asm/arch_timer.h:131:1: warning: no return statement in function returning non-void [-Wreturn-type]
}
^
In file included from drivers/clocksource/arm_arch_timer.c:31:
arch/arm/include/asm/arch_timer.h: In function 'arch_timer_set_evtstrm_feature':
arch/arm/include/asm/arch_timer.h:131:1: warning: no return statement in function returning non-void [-Wreturn-type]
}
^

Introduced by commit

11e34eca5d0a ("clocksource/arm_arch_timer: Extract elf_hwcap use to arch-helper")

Look like a missed "return".

--
Cheers,
Stephen Rothwell

---

Attachments:

(No filename) (499.00 B)
OpenPGP digital signature

2019-06-17 09:01:34

by Andrew Murray

[permalink] [raw]

Subject: Re: linux-next: build warnings after merge of the clockevents tree

---

On Mon, Jun 17, 2019 at 02:52:45PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the clockevents tree, today's linux-next build (arm
> multi_v7_defconfig) produced these warnings:
>
> In file included from arch/arm/kernel/vdso.c:30:
> arch/arm/include/asm/arch_timer.h: In function 'arch_timer_set_evtstrm_feature':
> arch/arm/include/asm/arch_timer.h:131:1: warning: no return statement in function returning non-void [-Wreturn-type]
> }
> ^
> In file included from drivers/clocksource/arm_arch_timer.c:31:
> arch/arm/include/asm/arch_timer.h: In function 'arch_timer_set_evtstrm_feature':
> arch/arm/include/asm/arch_timer.h:131:1: warning: no return statement in function returning non-void [-Wreturn-type]
> }
> ^
>
> Introduced by commit
>
> 11e34eca5d0a ("clocksource/arm_arch_timer: Extract elf_hwcap use to arch-helper")

This was also picked up by the kbuild robot, I'll sent a patch shortly - apologies.

Thanks,

Andrew Murray

>
> Look like a missed "return".
>
> --
> Cheers,
> Stephen Rothwell