Skip to contentSkip to conversation

video: kconfig: remove DM_I2C dependency of I2C_EDID

PATCH New Luis Araneda 2 participants 3 messages Jul 4, 2018 – Aug 1, 2018 Framebuffer layer

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 video: kconfig: remove DM_I2C dependency of I2C_EDID

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

---
 drivers/video/Kconfig | 1 -
 1 file changed, 1 deletion(-)
drivers/video/Kconfig +0 -1
index 5ee9032dc9..a2d7e10938 100644
@@ -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.
Signature · 1 line
2.18.0
#1 lore raw

PATCH 1/1 video: kconfig: remove DM_I2C dependency of I2C_EDID

Hi,

Quoted 20 lines · Luis Araneda

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

---
 drivers/video/Kconfig | 1 -
 1 file changed, 1 deletion(-)
drivers/video/Kconfig +0 -0
index 5ee9032dc9..a2d7e10938 100644

drivers/video/Kconfig config VIDEO_MVEBU · line 415

Quoted 7 lines
drivers/video/Kconfig +0 -1
@@ -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.

#2 lore raw

PATCH 1/1 video: kconfig: remove DM_I2C dependency of I2C_EDID

Quoted 17 lines · On Mon, 30 Jul 2018 09:11:31 -0400 Luis Araneda luaraneda at gmail.com wrote:

Hi,

Quoted 14 lines · Luis Araneda

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

---
 drivers/video/Kconfig | 1 -
 1 file changed, 1 deletion(-)

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

--
Anatolij