[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <65625987-9b8b-d0bc-5413-b8cd1f8ed14f@roeck-us.net>
Date: Mon, 18 Oct 2021 07:28:12 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Thierry Reding <thierry.reding@...il.com>,
Arnd Bergmann <arnd@...nel.org>
Cc: Laxman Dewangan <ldewangan@...dia.com>,
Mark Brown <broonie@...nel.org>,
Jonathan Hunter <jonathanh@...dia.com>,
Arnd Bergmann <arnd@...db.de>,
Dmitry Osipenko <digetx@...il.com>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Zhang Qilong <zhangqilong3@...wei.com>,
Yang Yingliang <yangyingliang@...wei.com>,
linux-spi@...r.kernel.org, linux-tegra@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] spi: tegra20: fix build with CONFIG_PM_SLEEP=n
On 10/18/21 6:57 AM, Thierry Reding wrote:
> On Wed, Oct 13, 2021 at 04:45:23PM +0200, Arnd Bergmann wrote:
>> From: Arnd Bergmann <arnd@...db.de>
>>
>> There is another one of these warnings:
>>
>> drivers/spi/spi-tegra20-slink.c:1197:12: error: 'tegra_slink_runtime_resume' defined but not used [-Werror=unused-function]
>> 1197 | static int tegra_slink_runtime_resume(struct device *dev)
>> | ^~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> Give it the same treatment as the other functions in this file.
>>
>> Fixes: efafec27c565 ("spi: Fix tegra20 build with CONFIG_PM=n")
>> Fixes: 2bab94090b01 ("spi: tegra20-slink: Declare runtime suspend and resume functions conditionally")
>> Signed-off-by: Arnd Bergmann <arnd@...db.de>
>> ---
>> drivers/spi/spi-tegra20-slink.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Acked-by: Thierry Reding <treding@...dia.com>
>
The problem has now been (re-)introduced into the mainline kernel
and causes various test build failures there.
Guenter
Powered by blists - more mailing lists