2021-12-01 23:28:21

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build failure after merge of the pm tree

Hi all,

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

drivers/powercap/dtpm.c: In function 'init_dtpm':
drivers/powercap/dtpm.c:466:21: error: unused variable 'dtpm_descr' [-Werror=unused-variable]
466 | struct dtpm_descr *dtpm_descr;
| ^~~~~~~~~~
cc1: all warnings being treated as errors

Caused by commit

f751db8adaea ("powercap/drivers/dtpm: Disable DTPM at boot time")

I have used the pm tree from next-20211201 for today.

--
Cheers,
Stephen Rothwell

Attachments:
(No filename) (488.00 B)
OpenPGP digital signature

2021-12-02 10:05:41

by Daniel Lezcano

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the pm tree


On 02/12/2021 00:28, Stephen Rothwell wrote:
> Hi all,
>
> After merging the pm tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
>
> drivers/powercap/dtpm.c: In function 'init_dtpm':
> drivers/powercap/dtpm.c:466:21: error: unused variable 'dtpm_descr' [-Werror=unused-variable]
> 466 | struct dtpm_descr *dtpm_descr;
> | ^~~~~~~~~~
> cc1: all warnings being treated as errors
>
> Caused by commit
>
> f751db8adaea ("powercap/drivers/dtpm: Disable DTPM at boot time")

Rafael, will you drop the patch from your tree and I send a new one with
the local variable also, or shall I send a change on top of?

>
> I have used the pm tree from next-20211201 for today.
>


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

2021-12-03 16:50:41

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the pm tree

On Thursday, December 2, 2021 11:04:45 AM CET Daniel Lezcano wrote:
>
> On 02/12/2021 00:28, Stephen Rothwell wrote:
> > Hi all,
> >
> > After merging the pm tree, today's linux-next build (x86_64 allmodconfig)
> > failed like this:
> >
> > drivers/powercap/dtpm.c: In function 'init_dtpm':
> > drivers/powercap/dtpm.c:466:21: error: unused variable 'dtpm_descr' [-Werror=unused-variable]
> > 466 | struct dtpm_descr *dtpm_descr;
> > | ^~~~~~~~~~
> > cc1: all warnings being treated as errors
> >
> > Caused by commit
> >
> > f751db8adaea ("powercap/drivers/dtpm: Disable DTPM at boot time")
>
> Rafael, will you drop the patch from your tree and I send a new one with
> the local variable also, or shall I send a change on top of?

I'll fix this.




2021-12-03 17:22:12

by Daniel Lezcano

[permalink] [raw]
Subject: Re: linux-next: build failure after merge of the pm tree

On 03/12/2021 17:50, Rafael J. Wysocki wrote:
> On Thursday, December 2, 2021 11:04:45 AM CET Daniel Lezcano wrote:
>>
>> On 02/12/2021 00:28, Stephen Rothwell wrote:
>>> Hi all,
>>>
>>> After merging the pm tree, today's linux-next build (x86_64 allmodconfig)
>>> failed like this:
>>>
>>> drivers/powercap/dtpm.c: In function 'init_dtpm':
>>> drivers/powercap/dtpm.c:466:21: error: unused variable 'dtpm_descr' [-Werror=unused-variable]
>>> 466 | struct dtpm_descr *dtpm_descr;
>>> | ^~~~~~~~~~
>>> cc1: all warnings being treated as errors
>>>
>>> Caused by commit
>>>
>>> f751db8adaea ("powercap/drivers/dtpm: Disable DTPM at boot time")
>>
>> Rafael, will you drop the patch from your tree and I send a new one with
>> the local variable also, or shall I send a change on top of?
>
> I'll fix this.

Ok, thanks. Sorry for the inconvenience


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog