diff mbox series

[U-Boot] video: kconfig: remove DM_I2C dependency of I2C_EDID

Message ID 20180704064000.13499-1-luaraneda@gmail.com
State Accepted
Commit ba71ab8246d1d1c4778ebe5dd892d569c8d6219c
Delegated to: Anatolij Gustschin
Headers show
Series [U-Boot] video: kconfig: remove DM_I2C dependency of I2C_EDID | expand

Commit Message

Luis Araneda July 4, 2018, 6:40 a.m. UTC
Drop the DM_I2C dependency, as the library only implements
the parsing of EDID data and doesn't depend on any driver

One user of this library, the i2c command, implements
support for legacy and DM I2C drivers

Tested on a Zynq board, whose I2C driver is not ported
yet to DM

Signed-off-by: Luis Araneda <luaraneda@gmail.com>
---
 drivers/video/Kconfig | 1 -
 1 file changed, 1 deletion(-)

Comments

Luis Araneda July 30, 2018, 1:11 p.m. UTC | #1
Hi,

On Wed, Jul 4, 2018 at 2:40 AM Luis Araneda <luaraneda@gmail.com> wrote:
>
> Drop the DM_I2C dependency, as the library only implements
> the parsing of EDID data and doesn't depend on any driver
>
> One user of this library, the i2c command, implements
> support for legacy and DM I2C drivers
>
> Tested on a Zynq board, whose I2C driver is not ported
> yet to DM
>
> Signed-off-by: Luis Araneda <luaraneda@gmail.com>
> ---
>  drivers/video/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> index 5ee9032dc9..a2d7e10938 100644
> --- a/drivers/video/Kconfig
> +++ b/drivers/video/Kconfig
> @@ -410,7 +410,6 @@ config VIDEO_MVEBU
>
>  config I2C_EDID
>         bool "Enable EDID library"
> -       depends on DM_I2C
>         default n
>         help
>            This enables library for accessing EDID data from an LCD panel.

Ping, can I get this reviewed please, and merged if it's ok.
I'm moving some legacy board config options to kconfig and some of
them depend on removing this dependency.

Thanks,

Luis Araneda.
Anatolij Gustschin Aug. 1, 2018, 12:56 p.m. UTC | #2
On Mon, 30 Jul 2018 09:11:31 -0400
Luis Araneda luaraneda@gmail.com wrote:

> Hi,
> 
> On Wed, Jul 4, 2018 at 2:40 AM Luis Araneda <luaraneda@gmail.com> wrote:
> >
> > Drop the DM_I2C dependency, as the library only implements
> > the parsing of EDID data and doesn't depend on any driver
> >
> > One user of this library, the i2c command, implements
> > support for legacy and DM I2C drivers
> >
> > Tested on a Zynq board, whose I2C driver is not ported
> > yet to DM
> >
> > Signed-off-by: Luis Araneda <luaraneda@gmail.com>
> > ---
> >  drivers/video/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)

Applied to u-boot-video/master, thanks!

--
Anatolij
diff mbox series

Patch

diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index 5ee9032dc9..a2d7e10938 100644
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -410,7 +410,6 @@  config VIDEO_MVEBU
 
 config I2C_EDID
 	bool "Enable EDID library"
-	depends on DM_I2C
 	default n
 	help
 	   This enables library for accessing EDID data from an LCD panel.