* [PATCH v1 0/3] Tegra VDE: Minor code clean up
@ 2019-12-15 18:07 Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 1/3] media: staging: tegra-vde: Select IOVA unconditionally in Kconfig Dmitry Osipenko
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Dmitry Osipenko @ 2019-12-15 18:07 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, Mauro Carvalho Chehab, Hans Verkuil
Cc: linux-tegra, devel, linux-kernel, linux-media
Hello,
This small series just makes a minor clean up of the driver's code,
addressing COMPILE_TEST build problem that was reported sometime ago.
Dmitry Osipenko (3):
media: staging: tegra-vde: Select IOVA unconditionally in Kconfig
media: staging: tegra-vde: Sort headers alphabetically
media: staging: tegra-vde: Use __maybe_unused attribute instead of
ifdef
drivers/staging/media/tegra-vde/Kconfig | 2 +-
drivers/staging/media/tegra-vde/vde.c | 6 ++----
drivers/staging/media/tegra-vde/vde.h | 2 +-
3 files changed, 4 insertions(+), 6 deletions(-)
--
2.24.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v1 1/3] media: staging: tegra-vde: Select IOVA unconditionally in Kconfig
2019-12-15 18:07 [PATCH v1 0/3] Tegra VDE: Minor code clean up Dmitry Osipenko
@ 2019-12-15 18:07 ` Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 2/3] media: staging: tegra-vde: Sort headers alphabetically Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 3/3] media: staging: tegra-vde: Use __maybe_unused attribute instead of ifdef Dmitry Osipenko
2 siblings, 0 replies; 4+ messages in thread
From: Dmitry Osipenko @ 2019-12-15 18:07 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, Mauro Carvalho Chehab, Hans Verkuil
Cc: linux-tegra, devel, linux-kernel, linux-media
There were several reports of building failures due to IOVA being selected
inconsistently by different drivers. All drivers that are using IOVA API
should select it unconditionally in order to avoid the compilation
problems, tegra-vde is one of those drivers.
Link: https://patchwork.kernel.org/patch/11057831/
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
drivers/staging/media/tegra-vde/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/tegra-vde/Kconfig b/drivers/staging/media/tegra-vde/Kconfig
index ba49ea50b8c0..0dc78afd09e0 100644
--- a/drivers/staging/media/tegra-vde/Kconfig
+++ b/drivers/staging/media/tegra-vde/Kconfig
@@ -3,7 +3,7 @@ config TEGRA_VDE
tristate "NVIDIA Tegra Video Decoder Engine driver"
depends on ARCH_TEGRA || COMPILE_TEST
select DMA_SHARED_BUFFER
- select IOMMU_IOVA if (IOMMU_SUPPORT || COMPILE_TEST)
+ select IOMMU_IOVA
select SRAM
help
Say Y here to enable support for the NVIDIA Tegra video decoder
--
2.24.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v1 2/3] media: staging: tegra-vde: Sort headers alphabetically
2019-12-15 18:07 [PATCH v1 0/3] Tegra VDE: Minor code clean up Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 1/3] media: staging: tegra-vde: Select IOVA unconditionally in Kconfig Dmitry Osipenko
@ 2019-12-15 18:07 ` Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 3/3] media: staging: tegra-vde: Use __maybe_unused attribute instead of ifdef Dmitry Osipenko
2 siblings, 0 replies; 4+ messages in thread
From: Dmitry Osipenko @ 2019-12-15 18:07 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, Mauro Carvalho Chehab, Hans Verkuil
Cc: linux-tegra, devel, linux-kernel, linux-media
Keep the alphabetical order for consistency.
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
drivers/staging/media/tegra-vde/vde.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/tegra-vde/vde.h b/drivers/staging/media/tegra-vde/vde.h
index d369f1466bc7..5561291b0c88 100644
--- a/drivers/staging/media/tegra-vde/vde.h
+++ b/drivers/staging/media/tegra-vde/vde.h
@@ -10,11 +10,11 @@
#include <linux/completion.h>
#include <linux/dma-direction.h>
+#include <linux/iova.h>
#include <linux/list.h>
#include <linux/miscdevice.h>
#include <linux/mutex.h>
#include <linux/types.h>
-#include <linux/iova.h>
struct clk;
struct dma_buf;
--
2.24.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v1 3/3] media: staging: tegra-vde: Use __maybe_unused attribute instead of ifdef
2019-12-15 18:07 [PATCH v1 0/3] Tegra VDE: Minor code clean up Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 1/3] media: staging: tegra-vde: Select IOVA unconditionally in Kconfig Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 2/3] media: staging: tegra-vde: Sort headers alphabetically Dmitry Osipenko
@ 2019-12-15 18:07 ` Dmitry Osipenko
2 siblings, 0 replies; 4+ messages in thread
From: Dmitry Osipenko @ 2019-12-15 18:07 UTC (permalink / raw)
To: Thierry Reding, Jonathan Hunter, Mauro Carvalho Chehab, Hans Verkuil
Cc: linux-tegra, devel, linux-kernel, linux-media
Replace #ifdef with __maybe_unused attribute just to keep code cleaner a
tad.
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
---
drivers/staging/media/tegra-vde/vde.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/media/tegra-vde/vde.c b/drivers/staging/media/tegra-vde/vde.c
index 3466daddf663..e18fd48981da 100644
--- a/drivers/staging/media/tegra-vde/vde.c
+++ b/drivers/staging/media/tegra-vde/vde.c
@@ -1150,8 +1150,7 @@ static int tegra_vde_remove(struct platform_device *pdev)
return 0;
}
-#ifdef CONFIG_PM_SLEEP
-static int tegra_vde_pm_suspend(struct device *dev)
+static __maybe_unused int tegra_vde_pm_suspend(struct device *dev)
{
struct tegra_vde *vde = dev_get_drvdata(dev);
int err;
@@ -1165,7 +1164,7 @@ static int tegra_vde_pm_suspend(struct device *dev)
return 0;
}
-static int tegra_vde_pm_resume(struct device *dev)
+static __maybe_unused int tegra_vde_pm_resume(struct device *dev)
{
struct tegra_vde *vde = dev_get_drvdata(dev);
int err;
@@ -1178,7 +1177,6 @@ static int tegra_vde_pm_resume(struct device *dev)
return 0;
}
-#endif
static const struct dev_pm_ops tegra_vde_pm_ops = {
SET_RUNTIME_PM_OPS(tegra_vde_runtime_suspend,
--
2.24.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-12-15 18:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-15 18:07 [PATCH v1 0/3] Tegra VDE: Minor code clean up Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 1/3] media: staging: tegra-vde: Select IOVA unconditionally in Kconfig Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 2/3] media: staging: tegra-vde: Sort headers alphabetically Dmitry Osipenko
2019-12-15 18:07 ` [PATCH v1 3/3] media: staging: tegra-vde: Use __maybe_unused attribute instead of ifdef Dmitry Osipenko
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).