2020-07-30 02:58:27

by Stephen Rothwell

[permalink] [raw]

Subject: linux-next: build warning after merge of the pm tree

---

Hi all,

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

drivers/acpi/processor_idle.c: In function 'acpi_idle_enter_s2idle':
drivers/acpi/processor_idle.c:666:4: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
666 | return;
| ^~~~~~
drivers/acpi/processor_idle.c:657:12: note: declared here
657 | static int acpi_idle_enter_s2idle(struct cpuidle_device *dev,
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/acpi/processor_idle.c:670:4: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
670 | return;
| ^~~~~~
drivers/acpi/processor_idle.c:657:12: note: declared here
657 | static int acpi_idle_enter_s2idle(struct cpuidle_device *dev,
| ^~~~~~~~~~~~~~~~~~~~~~

Introduced by commit

efe9711214e6 ("cpuidle: change enter_s2idle() prototype")

--
Cheers,
Stephen Rothwell

---

Attachments:

(No filename) (499.00 B)
OpenPGP digital signature

2020-07-30 03:11:27

by Neal Liu

[permalink] [raw]

Subject: Re: linux-next: build warning after merge of the pm tree

---

This warning should be fixed also.
Should I send another patch?

Thanks !

On Thu, 2020-07-30 at 12:55 +1000, Stephen Rothwell wrote:
> Hi all,
>
> After merging the pm tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
>
> drivers/acpi/processor_idle.c: In function 'acpi_idle_enter_s2idle':
> drivers/acpi/processor_idle.c:666:4: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
> 666 | return;
> | ^~~~~~
> drivers/acpi/processor_idle.c:657:12: note: declared here
> 657 | static int acpi_idle_enter_s2idle(struct cpuidle_device *dev,
> | ^~~~~~~~~~~~~~~~~~~~~~
> drivers/acpi/processor_idle.c:670:4: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
> 670 | return;
> | ^~~~~~
> drivers/acpi/processor_idle.c:657:12: note: declared here
> 657 | static int acpi_idle_enter_s2idle(struct cpuidle_device *dev,
> | ^~~~~~~~~~~~~~~~~~~~~~
>
> Introduced by commit
>
> efe9711214e6 ("cpuidle: change enter_s2idle() prototype")
>