* [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused
@ 2019-01-08 14:37 Anders Roxell
2019-01-10 8:16 ` Thierry Reding
0 siblings, 1 reply; 4+ messages in thread
From: Anders Roxell @ 2019-01-08 14:37 UTC (permalink / raw)
To: jassisinghbrar, thierry.reding, jonathanh
Cc: linux-kernel, linux-tegra, Anders Roxell
Without CONFIG_PM_SLEEP, we get annoying warnings about unused
functions:
drivers/mailbox/tegra-hsp.c:782:12: warning: ‘tegra_hsp_resume’ defined but not used [-Wunused-function]
static int tegra_hsp_resume(struct device *dev)
^~~~~~~~~~~~~~~~
Mark them as __maybe_unused to shut up the warning and silently drop the
functions without having to add ugly #ifdefs.
Fixes: 9a63f0f40599 ("mailbox: tegra-hsp: Add suspend/resume support")
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
drivers/mailbox/tegra-hsp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mailbox/tegra-hsp.c b/drivers/mailbox/tegra-hsp.c
index e443f6a2ec4b..b9ce823f79ab 100644
--- a/drivers/mailbox/tegra-hsp.c
+++ b/drivers/mailbox/tegra-hsp.c
@@ -779,7 +779,7 @@ static int tegra_hsp_probe(struct platform_device *pdev)
return 0;
}
-static int tegra_hsp_resume(struct device *dev)
+static __maybe_unused int tegra_hsp_resume(struct device *dev)
{
struct tegra_hsp *hsp = dev_get_drvdata(dev);
unsigned int i;
--
2.19.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused
2019-01-08 14:37 [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused Anders Roxell
@ 2019-01-10 8:16 ` Thierry Reding
2019-06-19 7:56 ` Anders Roxell
0 siblings, 1 reply; 4+ messages in thread
From: Thierry Reding @ 2019-01-10 8:16 UTC (permalink / raw)
To: Anders Roxell; +Cc: jassisinghbrar, jonathanh, linux-kernel, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 755 bytes --]
On Tue, Jan 08, 2019 at 03:37:13PM +0100, Anders Roxell wrote:
> Without CONFIG_PM_SLEEP, we get annoying warnings about unused
> functions:
>
> drivers/mailbox/tegra-hsp.c:782:12: warning: ‘tegra_hsp_resume’ defined but not used [-Wunused-function]
> static int tegra_hsp_resume(struct device *dev)
> ^~~~~~~~~~~~~~~~
>
> Mark them as __maybe_unused to shut up the warning and silently drop the
> functions without having to add ugly #ifdefs.
>
> Fixes: 9a63f0f40599 ("mailbox: tegra-hsp: Add suspend/resume support")
> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> ---
> drivers/mailbox/tegra-hsp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Thierry Reding <treding@nvidia.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused
2019-01-10 8:16 ` Thierry Reding
@ 2019-06-19 7:56 ` Anders Roxell
2019-06-19 7:57 ` Anders Roxell
0 siblings, 1 reply; 4+ messages in thread
From: Anders Roxell @ 2019-06-19 7:56 UTC (permalink / raw)
To: Thierry Reding
Cc: jassisinghbrar, jonathanh, Linux Kernel Mailing List, linux-tegra
On Thu, 10 Jan 2019 at 09:16, Thierry Reding <thierry.reding@gmail.com> wrote:
>
> On Tue, Jan 08, 2019 at 03:37:13PM +0100, Anders Roxell wrote:
> > Without CONFIG_PM_SLEEP, we get annoying warnings about unused
> > functions:
> >
> > drivers/mailbox/tegra-hsp.c:782:12: warning: ‘tegra_hsp_resume’ defined but not used [-Wunused-function]
> > static int tegra_hsp_resume(struct device *dev)
> > ^~~~~~~~~~~~~~~~
> >
> > Mark them as __maybe_unused to shut up the warning and silently drop the
> > functions without having to add ugly #ifdefs.
> >
> > Fixes: 9a63f0f40599 ("mailbox: tegra-hsp: Add suspend/resume support")
> > Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> > ---
> > drivers/mailbox/tegra-hsp.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Acked-by: Thierry Reding <treding@nvidia.com>
Will this be picked up ?
Cheers,
Anders
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused
2019-06-19 7:56 ` Anders Roxell
@ 2019-06-19 7:57 ` Anders Roxell
0 siblings, 0 replies; 4+ messages in thread
From: Anders Roxell @ 2019-06-19 7:57 UTC (permalink / raw)
To: Thierry Reding
Cc: jassisinghbrar, jonathanh, Linux Kernel Mailing List, linux-tegra
On Wed, 19 Jun 2019 at 09:56, Anders Roxell <anders.roxell@linaro.org> wrote:
>
> On Thu, 10 Jan 2019 at 09:16, Thierry Reding <thierry.reding@gmail.com> wrote:
> >
> > On Tue, Jan 08, 2019 at 03:37:13PM +0100, Anders Roxell wrote:
> > > Without CONFIG_PM_SLEEP, we get annoying warnings about unused
> > > functions:
> > >
> > > drivers/mailbox/tegra-hsp.c:782:12: warning: ‘tegra_hsp_resume’ defined but not used [-Wunused-function]
> > > static int tegra_hsp_resume(struct device *dev)
> > > ^~~~~~~~~~~~~~~~
> > >
> > > Mark them as __maybe_unused to shut up the warning and silently drop the
> > > functions without having to add ugly #ifdefs.
> > >
> > > Fixes: 9a63f0f40599 ("mailbox: tegra-hsp: Add suspend/resume support")
> > > Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> > > ---
> > > drivers/mailbox/tegra-hsp.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Acked-by: Thierry Reding <treding@nvidia.com>
>
> Will this be picked up ?
its already solved.
Sorry for the noice. =/
Anders
>
> Cheers,
> Anders
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-19 7:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-08 14:37 [PATCH] mailbox: tegra-hsp: mark PM functions as __maybe_unused Anders Roxell
2019-01-10 8:16 ` Thierry Reding
2019-06-19 7:56 ` Anders Roxell
2019-06-19 7:57 ` Anders Roxell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).