Subject: [PATCH 1/3] video: fbdev: mmp: remove duplicated MMP_DISP dependency

This dependency is already present in higher level Kconfig file
(drivers/video/fbdev/mmp/Kconfig).

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

Index: b/drivers/video/fbdev/mmp/fb/Kconfig
===================================================================
--- a/drivers/video/fbdev/mmp/fb/Kconfig
+++ b/drivers/video/fbdev/mmp/fb/Kconfig
@@ -1,6 +1,4 @@
# SPDX-License-Identifier: GPL-2.0-only
-if MMP_DISP
-
config MMP_FB
tristate "fb driver for Marvell MMP Display Subsystem"
depends on FB
@@ -10,5 +8,3 @@ config MMP_FB
default y
help
fb driver for Marvell MMP Display Subsystem
-
-endif
Index: b/drivers/video/fbdev/mmp/hw/Kconfig
===================================================================
--- a/drivers/video/fbdev/mmp/hw/Kconfig
+++ b/drivers/video/fbdev/mmp/hw/Kconfig
@@ -1,6 +1,4 @@
# SPDX-License-Identifier: GPL-2.0-only
-if MMP_DISP
-
config MMP_DISP_CONTROLLER
bool "mmp display controller hw support"
depends on CPU_PXA910 || CPU_MMP2
@@ -16,5 +14,3 @@ config MMP_DISP_SPI
help
Marvell MMP display hw controller spi port support
will register as a spi master for panel usage
-
-endif


2019-08-20 09:57:29

by Andrzej Hajda

[permalink] [raw]
Subject: Re: [PATCH 1/3] video: fbdev: mmp: remove duplicated MMP_DISP dependency

On 27.06.2019 16:07, Bartlomiej Zolnierkiewicz wrote:
> This dependency is already present in higher level Kconfig file
> (drivers/video/fbdev/mmp/Kconfig).
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]>


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

 --
Regards
Andrzej


> ---
> drivers/video/fbdev/mmp/fb/Kconfig | 4 ----
> drivers/video/fbdev/mmp/hw/Kconfig | 4 ----
> 2 files changed, 8 deletions(-)
>
> Index: b/drivers/video/fbdev/mmp/fb/Kconfig
> ===================================================================
> --- a/drivers/video/fbdev/mmp/fb/Kconfig
> +++ b/drivers/video/fbdev/mmp/fb/Kconfig
> @@ -1,6 +1,4 @@
> # SPDX-License-Identifier: GPL-2.0-only
> -if MMP_DISP
> -
> config MMP_FB
> tristate "fb driver for Marvell MMP Display Subsystem"
> depends on FB
> @@ -10,5 +8,3 @@ config MMP_FB
> default y
> help
> fb driver for Marvell MMP Display Subsystem
> -
> -endif
> Index: b/drivers/video/fbdev/mmp/hw/Kconfig
> ===================================================================
> --- a/drivers/video/fbdev/mmp/hw/Kconfig
> +++ b/drivers/video/fbdev/mmp/hw/Kconfig
> @@ -1,6 +1,4 @@
> # SPDX-License-Identifier: GPL-2.0-only
> -if MMP_DISP
> -
> config MMP_DISP_CONTROLLER
> bool "mmp display controller hw support"
> depends on CPU_PXA910 || CPU_MMP2
> @@ -16,5 +14,3 @@ config MMP_DISP_SPI
> help
> Marvell MMP display hw controller spi port support
> will register as a spi master for panel usage
> -
> -endif
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel


Subject: Re: [PATCH 1/3] video: fbdev: mmp: remove duplicated MMP_DISP dependency


On 8/20/19 11:55 AM, Andrzej Hajda wrote:
> On 27.06.2019 16:07, Bartlomiej Zolnierkiewicz wrote:
>> This dependency is already present in higher level Kconfig file
>> (drivers/video/fbdev/mmp/Kconfig).
>>
>> 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/fb/Kconfig | 4 ----
>> drivers/video/fbdev/mmp/hw/Kconfig | 4 ----
>> 2 files changed, 8 deletions(-)
>>
>> Index: b/drivers/video/fbdev/mmp/fb/Kconfig
>> ===================================================================
>> --- a/drivers/video/fbdev/mmp/fb/Kconfig
>> +++ b/drivers/video/fbdev/mmp/fb/Kconfig
>> @@ -1,6 +1,4 @@
>> # SPDX-License-Identifier: GPL-2.0-only
>> -if MMP_DISP
>> -
>> config MMP_FB
>> tristate "fb driver for Marvell MMP Display Subsystem"
>> depends on FB
>> @@ -10,5 +8,3 @@ config MMP_FB
>> default y
>> help
>> fb driver for Marvell MMP Display Subsystem
>> -
>> -endif
>> Index: b/drivers/video/fbdev/mmp/hw/Kconfig
>> ===================================================================
>> --- a/drivers/video/fbdev/mmp/hw/Kconfig
>> +++ b/drivers/video/fbdev/mmp/hw/Kconfig
>> @@ -1,6 +1,4 @@
>> # SPDX-License-Identifier: GPL-2.0-only
>> -if MMP_DISP
>> -
>> config MMP_DISP_CONTROLLER
>> bool "mmp display controller hw support"
>> depends on CPU_PXA910 || CPU_MMP2
>> @@ -16,5 +14,3 @@ config MMP_DISP_SPI
>> help
>> Marvell MMP display hw controller spi port support
>> will register as a spi master for panel usage
>> -
>> -endif
>> _______________________________________________
>> dri-devel mailing list
>> [email protected]
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel