diff mbox series

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

Message ID d21a19ea-8c18-80df-ae79-76de7c5ee67c@samsung.com (mailing list archive)
State New, archived
Headers show
Series [1/3] video: fbdev: mmp: remove duplicated MMP_DISP dependency | expand

Commit Message

Bartlomiej Zolnierkiewicz June 27, 2019, 2:07 p.m. UTC
Add COMPILE_TEST support to mmp display subsystem for better compile
testing coverage.

Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
---
 drivers/video/fbdev/mmp/Kconfig    |    2 +-
 drivers/video/fbdev/mmp/hw/Kconfig |    3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Andrzej Hajda Aug. 20, 2019, 9:56 a.m. UTC | #1
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 <b.zolnierkie@samsung.com>


Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>

 --
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
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Bartlomiej Zolnierkiewicz Jan. 3, 2020, 11:38 a.m. UTC | #2
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 <b.zolnierkie@samsung.com>
> 
> 
> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>

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
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff mbox series

Patch

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