* [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM
@ 2016-03-04 1:02 Krzysztof Kozlowski
2016-03-04 7:03 ` Holger Schurig
2016-03-04 11:02 ` Sylwester Nawrocki
0 siblings, 2 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-03-04 1:02 UTC (permalink / raw)
To: Sylwester Nawrocki, Mauro Carvalho Chehab, linux-media,
linux-arm-kernel, linux-samsung-soc, linux-kernel
Cc: Arnd Bergmann, Krzysztof Kozlowski
The MFD_SYSCON depends on HAS_IOMEM so when selecting it avoid unmet
direct dependencies.
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
---
Changes since v1:
1. Fix comments from Arnd: VIDEO_OMAP3 does not require it, however
VIDEO_S5P_FIMC still needs it.
---
drivers/media/platform/exynos4-is/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/exynos4-is/Kconfig b/drivers/media/platform/exynos4-is/Kconfig
index 57d42c6172c5..c4317b99d257 100644
--- a/drivers/media/platform/exynos4-is/Kconfig
+++ b/drivers/media/platform/exynos4-is/Kconfig
@@ -17,6 +17,7 @@ config VIDEO_S5P_FIMC
tristate "S5P/EXYNOS4 FIMC/CAMIF camera interface driver"
depends on I2C
depends on HAS_DMA
+ depends on HAS_IOMEM # For MFD_SYSCON
select VIDEOBUF2_DMA_CONTIG
select V4L2_MEM2MEM_DEV
select MFD_SYSCON
--
2.5.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM
2016-03-04 1:02 [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM Krzysztof Kozlowski
@ 2016-03-04 7:03 ` Holger Schurig
2016-03-04 7:05 ` Krzysztof Kozlowski
2016-03-04 11:02 ` Sylwester Nawrocki
1 sibling, 1 reply; 5+ messages in thread
From: Holger Schurig @ 2016-03-04 7:03 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Sylwester Nawrocki, Mauro Carvalho Chehab, linux-media,
linux-arm-kernel, linux-samsung-soc, linux-kernel, Arnd Bergmann
Krzysztof Kozlowski <k.kozlowski@samsung.com> writes:
> + depends on HAS_IOMEM # For MFD_SYSCON
^^^^^^^^^^^^^^^^
I think this comment is not necessary, it's also highly unusual. On
other words: other patches like this don't add such comments.
You can always use "git blame" to find out why some line has changed the
way it changed ...
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM
2016-03-04 7:03 ` Holger Schurig
@ 2016-03-04 7:05 ` Krzysztof Kozlowski
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-03-04 7:05 UTC (permalink / raw)
To: Holger Schurig
Cc: Sylwester Nawrocki, Mauro Carvalho Chehab, linux-media,
linux-arm-kernel, linux-samsung-soc, linux-kernel, Arnd Bergmann
On 04.03.2016 16:03, Holger Schurig wrote:
> Krzysztof Kozlowski <k.kozlowski@samsung.com> writes:
>
>> + depends on HAS_IOMEM # For MFD_SYSCON
> ^^^^^^^^^^^^^^^^
>
> I think this comment is not necessary, it's also highly unusual. On
> other words: other patches like this don't add such comments.
>
> You can always use "git blame" to find out why some line has changed the
> way it changed ...
No problem, I can remove it. I thought it might be useful since this
dependency is not for the driver but for selected item.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM
2016-03-04 1:02 [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM Krzysztof Kozlowski
2016-03-04 7:03 ` Holger Schurig
@ 2016-03-04 11:02 ` Sylwester Nawrocki
2016-03-05 4:28 ` Krzysztof Kozlowski
1 sibling, 1 reply; 5+ messages in thread
From: Sylwester Nawrocki @ 2016-03-04 11:02 UTC (permalink / raw)
To: Krzysztof Kozlowski, linux-arm-kernel
Cc: Mauro Carvalho Chehab, linux-media, linux-samsung-soc,
linux-kernel, Arnd Bergmann
On 03/04/2016 02:02 AM, Krzysztof Kozlowski wrote:
> The MFD_SYSCON depends on HAS_IOMEM so when selecting
> it avoid unmet direct dependencies.
> diff --git a/drivers/media/platform/exynos4-is/Kconfig
> b/drivers/media/platform/exynos4-is/Kconfig
> index 57d42c6172c5..c4317b99d257 100644
> --- a/drivers/media/platform/exynos4-is/Kconfig
> +++ b/drivers/media/platform/exynos4-is/Kconfig
> @@ -17,6 +17,7 @@ config VIDEO_S5P_FIMC
> tristate "S5P/EXYNOS4 FIMC/CAMIF camera interface driver"
> depends on I2C
> depends on HAS_DMA
> + depends on HAS_IOMEM # For MFD_SYSCON
> select VIDEOBUF2_DMA_CONTIG
> select V4L2_MEM2MEM_DEV
> select MFD_SYSCON
While we are already at it, shouldn't "depends on HAS_IOMEM"
be instead added at the top level entry in this Kconfig file,
i.e. "config VIDEO_SAMSUNG_EXYNOS4_IS" ? For things like
devm_ioremap_resource() depending on HAS_IOMEM and used in all
the sub-drivers, enabled by VIDEO_SAMSUNG_EXYNOS4_IS?
--
Thanks,
Sylwester
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM
2016-03-04 11:02 ` Sylwester Nawrocki
@ 2016-03-05 4:28 ` Krzysztof Kozlowski
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-03-05 4:28 UTC (permalink / raw)
To: Sylwester Nawrocki
Cc: linux-arm-kernel, linux-samsung-soc, linux-media, linux-kernel,
Arnd Bergmann, Mauro Carvalho Chehab, Krzysztof Kozlowski,
Krzysztof Kozlowski
2016-03-04 20:02 GMT+09:00 Sylwester Nawrocki <s.nawrocki@samsung.com>:
> On 03/04/2016 02:02 AM, Krzysztof Kozlowski wrote:
>> The MFD_SYSCON depends on HAS_IOMEM so when selecting
>> it avoid unmet direct dependencies.
>
>> diff --git a/drivers/media/platform/exynos4-is/Kconfig
>> b/drivers/media/platform/exynos4-is/Kconfig
>> index 57d42c6172c5..c4317b99d257 100644
>> --- a/drivers/media/platform/exynos4-is/Kconfig
>> +++ b/drivers/media/platform/exynos4-is/Kconfig
>> @@ -17,6 +17,7 @@ config VIDEO_S5P_FIMC
>> tristate "S5P/EXYNOS4 FIMC/CAMIF camera interface driver"
>> depends on I2C
>> depends on HAS_DMA
>> + depends on HAS_IOMEM # For MFD_SYSCON
>> select VIDEOBUF2_DMA_CONTIG
>> select V4L2_MEM2MEM_DEV
>> select MFD_SYSCON
>
> While we are already at it, shouldn't "depends on HAS_IOMEM"
> be instead added at the top level entry in this Kconfig file,
> i.e. "config VIDEO_SAMSUNG_EXYNOS4_IS" ? For things like
> devm_ioremap_resource() depending on HAS_IOMEM and used in all
> the sub-drivers, enabled by VIDEO_SAMSUNG_EXYNOS4_IS?
Indeed that would make sense... but now after some more looking at it,
even this patch is not needed. All these drivers are children of
MEDIA_SUPPORT which already depends on HAS_IOMEM.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-03-05 4:28 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-04 1:02 [PATCH v2] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM Krzysztof Kozlowski
2016-03-04 7:03 ` Holger Schurig
2016-03-04 7:05 ` Krzysztof Kozlowski
2016-03-04 11:02 ` Sylwester Nawrocki
2016-03-05 4:28 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox