Subject: [PATCH 2/3] video: fbdev: mmp: add COMPILE_TEST support

Add COMPILE_TEST support to mmp display subsystem for better compile
testing coverage.

Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
---
drivers/video/fbdev/mmp/Kconfig | 2 +-
drivers/video/fbdev/mmp/hw/Kconfig | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)

Index: b/drivers/video/fbdev/mmp/Kconfig
===================================================================
--- a/drivers/video/fbdev/mmp/Kconfig
+++ b/drivers/video/fbdev/mmp/Kconfig
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: GPL-2.0-only
menuconfig MMP_DISP
tristate "Marvell MMP Display Subsystem support"
- depends on CPU_PXA910 || CPU_MMP2
+ depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
help
Marvell Display Subsystem support.

Index: b/drivers/video/fbdev/mmp/hw/Kconfig
===================================================================
--- a/drivers/video/fbdev/mmp/hw/Kconfig
+++ b/drivers/video/fbdev/mmp/hw/Kconfig
@@ -1,7 +1,8 @@
# SPDX-License-Identifier: GPL-2.0-only
config MMP_DISP_CONTROLLER
bool "mmp display controller hw support"
- depends on CPU_PXA910 || CPU_MMP2
+ depends on HAVE_CLK && HAS_IOMEM
+ depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
help
Marvell MMP display hw controller support
this controller is used on Marvell PXA910 and


2019-08-20 09:59:01

by Andrzej Hajda

[permalink] [raw]
Subject: Re: [PATCH 2/3] video: fbdev: mmp: add COMPILE_TEST support

On 27.06.2019 16:07, Bartlomiej Zolnierkiewicz wrote:
> Add COMPILE_TEST support to mmp display subsystem for better compile
> testing coverage.
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>


Reviewed-by: Andrzej Hajda <[email protected]>

 --
Regards
Andrzej


> ---
> drivers/video/fbdev/mmp/Kconfig | 2 +-
> drivers/video/fbdev/mmp/hw/Kconfig | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> Index: b/drivers/video/fbdev/mmp/Kconfig
> ===================================================================
> --- a/drivers/video/fbdev/mmp/Kconfig
> +++ b/drivers/video/fbdev/mmp/Kconfig
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> menuconfig MMP_DISP
> tristate "Marvell MMP Display Subsystem support"
> - depends on CPU_PXA910 || CPU_MMP2
> + depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
> help
> Marvell Display Subsystem support.
>
> Index: b/drivers/video/fbdev/mmp/hw/Kconfig
> ===================================================================
> --- a/drivers/video/fbdev/mmp/hw/Kconfig
> +++ b/drivers/video/fbdev/mmp/hw/Kconfig
> @@ -1,7 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0-only
> config MMP_DISP_CONTROLLER
> bool "mmp display controller hw support"
> - depends on CPU_PXA910 || CPU_MMP2
> + depends on HAVE_CLK && HAS_IOMEM
> + depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
> help
> Marvell MMP display hw controller support
> this controller is used on Marvell PXA910 and
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel


Subject: Re: [PATCH 2/3] video: fbdev: mmp: add COMPILE_TEST support


On 8/20/19 11:56 AM, Andrzej Hajda wrote:
> On 27.06.2019 16:07, Bartlomiej Zolnierkiewicz wrote:
>> Add COMPILE_TEST support to mmp display subsystem for better compile
>> testing coverage.
>>
>> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>
>
>
> Reviewed-by: Andrzej Hajda <[email protected]>

Thanks, I've queued the patch for v5.6.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

>  --
> Regards
> Andrzej
>
>
>> ---
>> drivers/video/fbdev/mmp/Kconfig | 2 +-
>> drivers/video/fbdev/mmp/hw/Kconfig | 3 ++-
>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>
>> Index: b/drivers/video/fbdev/mmp/Kconfig
>> ===================================================================
>> --- a/drivers/video/fbdev/mmp/Kconfig
>> +++ b/drivers/video/fbdev/mmp/Kconfig
>> @@ -1,7 +1,7 @@
>> # SPDX-License-Identifier: GPL-2.0-only
>> menuconfig MMP_DISP
>> tristate "Marvell MMP Display Subsystem support"
>> - depends on CPU_PXA910 || CPU_MMP2
>> + depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
>> help
>> Marvell Display Subsystem support.
>>
>> Index: b/drivers/video/fbdev/mmp/hw/Kconfig
>> ===================================================================
>> --- a/drivers/video/fbdev/mmp/hw/Kconfig
>> +++ b/drivers/video/fbdev/mmp/hw/Kconfig
>> @@ -1,7 +1,8 @@
>> # SPDX-License-Identifier: GPL-2.0-only
>> config MMP_DISP_CONTROLLER
>> bool "mmp display controller hw support"
>> - depends on CPU_PXA910 || CPU_MMP2
>> + depends on HAVE_CLK && HAS_IOMEM
>> + depends on CPU_PXA910 || CPU_MMP2 || COMPILE_TEST
>> help
>> Marvell MMP display hw controller support
>> this controller is used on Marvell PXA910 and
>> _______________________________________________
>> dri-devel mailing list
>> [email protected]
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel