Message ID | 20200804014137.45444-1-yuehaibing@huawei.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2,-next] media: staging: tegra-vde: Mark tegra_vde_runtime_suspend/resume as __maybe_unused | expand |
04.08.2020 04:41, YueHaibing пишет: > If CONFIG_PM is not set, gcc warns: > > drivers/staging/media/tegra-vde/vde.c:916:12: > warning: 'tegra_vde_runtime_suspend' defined but not used [-Wunused-function] > > Make it __maybe_unused to fix this. > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > v2: both suspend and resume functions marked > --- > drivers/staging/media/tegra-vde/vde.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/media/tegra-vde/vde.c b/drivers/staging/media/tegra-vde/vde.c > index a3c24d96d5b9..28845b5bafaf 100644 > --- a/drivers/staging/media/tegra-vde/vde.c > +++ b/drivers/staging/media/tegra-vde/vde.c > @@ -913,7 +913,7 @@ static irqreturn_t tegra_vde_isr(int irq, void *data) > return IRQ_HANDLED; > } > > -static int tegra_vde_runtime_suspend(struct device *dev) > +static __maybe_unused int tegra_vde_runtime_suspend(struct device *dev) > { > struct tegra_vde *vde = dev_get_drvdata(dev); > int err; > @@ -929,7 +929,7 @@ static int tegra_vde_runtime_suspend(struct device *dev) > return 0; > } > > -static int tegra_vde_runtime_resume(struct device *dev) > +static __maybe_unused int tegra_vde_runtime_resume(struct device *dev) > { > struct tegra_vde *vde = dev_get_drvdata(dev); > int err; > Thanks! Reviewed-by: Dmitry Osipenko <digetx@gmail.com>
diff --git a/drivers/staging/media/tegra-vde/vde.c b/drivers/staging/media/tegra-vde/vde.c index a3c24d96d5b9..28845b5bafaf 100644 --- a/drivers/staging/media/tegra-vde/vde.c +++ b/drivers/staging/media/tegra-vde/vde.c @@ -913,7 +913,7 @@ static irqreturn_t tegra_vde_isr(int irq, void *data) return IRQ_HANDLED; } -static int tegra_vde_runtime_suspend(struct device *dev) +static __maybe_unused int tegra_vde_runtime_suspend(struct device *dev) { struct tegra_vde *vde = dev_get_drvdata(dev); int err; @@ -929,7 +929,7 @@ static int tegra_vde_runtime_suspend(struct device *dev) return 0; } -static int tegra_vde_runtime_resume(struct device *dev) +static __maybe_unused int tegra_vde_runtime_resume(struct device *dev) { struct tegra_vde *vde = dev_get_drvdata(dev); int err;
If CONFIG_PM is not set, gcc warns: drivers/staging/media/tegra-vde/vde.c:916:12: warning: 'tegra_vde_runtime_suspend' defined but not used [-Wunused-function] Make it __maybe_unused to fix this. Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- v2: both suspend and resume functions marked --- drivers/staging/media/tegra-vde/vde.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)