Skip to contentSkip to conversation

Remove unused CONFIG_SYS_I2C_BUS[_SELECT]

PATCH New Michael Jones 2 participants 2 messages Jan 16, 2013

lore
Share & tools
lore

Michael Jones

author
#0 lore raw

PATCH 1/1 Remove unused CONFIG_SYS_I2C_BUS[_SELECT]

"CONFIG_SYS_I2C_BUS" and "CONFIG_SYS_I2C_BUS_SELECT" don't appear anywhere outside of config files.

Signed-off-by Michael Jones <[email protected]>

---
As far as I can tell, these CONFIG options never have existed outside of these config files - looking at omap3_beagle.h as an example, these lines were inserted with the creation of that file, and at that point in time there was also no occurence of them elsewhere. Can't we get rid of them?

 include/configs/am3517_crane.h     |    2 --
 include/configs/am3517_evm.h       |    2 --
 include/configs/cm_t35.h           |    2 --
 include/configs/devkit8000.h       |    2 --
 include/configs/dig297.h           |    2 --
 include/configs/igep00x0.h         |    2 --
 include/configs/mcx.h              |    1 -
 include/configs/omap3_beagle.h     |    2 --
 include/configs/omap3_evm_common.h |    2 --
 include/configs/omap3_logic.h      |    2 --
 include/configs/omap3_mvblx.h      |    2 --
 include/configs/omap3_pandora.h    |    2 --
 include/configs/omap3_sdp3430.h    |    2 --
 include/configs/omap3_zoom1.h      |    2 --
 include/configs/omap3_zoom2.h      |    2 --
 include/configs/omap4_common.h     |    2 --
 include/configs/tam3517-common.h   |    2 --
 include/configs/tricorder.h        |    2 --
 18 files changed, 0 insertions(+), 35 deletions(-)
include/configs/am3517_crane.h +0 -2
index e1ad1e5..bf9d63e 100644
@@ -156,8 +156,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 #undef CONFIG_CMD_NET
include/configs/am3517_evm.h +0 -2
index ba15325..17fe88d 100644
@@ -150,8 +150,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 #undef CONFIG_CMD_NET
include/configs/cm_t35.h +0 -2
index 7d07215..943b658 100644
@@ -157,8 +157,6 @@
 #define CONFIG_HARD_I2C
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C
 #define CONFIG_SYS_I2C_EEPROM_ADDR	0x50
 #define CONFIG_SYS_I2C_EEPROM_ADDR_LEN	1
include/configs/devkit8000.h +0 -2
index 83a8b5d..d926f74 100644
@@ -108,8 +108,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /* TWL4030 */
include/configs/dig297.h +0 -2
index 721b91c..e68654f 100644
@@ -141,8 +141,6 @@
 #define CONFIG_HARD_I2C
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /*
include/configs/igep00x0.h +0 -2
index f8131b1..e59f41e 100644
@@ -129,8 +129,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /*
include/configs/mcx.h +0 -1
index 185faa7..1f09947 100644
@@ -151,7 +151,6 @@
 #define CONFIG_HARD_I2C
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
 #define CONFIG_DRIVER_OMAP34XX_I2C
 
 /* RTC */
include/configs/omap3_beagle.h +0 -2
index d0daa45..59255c4 100644
@@ -178,8 +178,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_I2C_MULTI_BUS		1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 #define CONFIG_VIDEO_OMAP3	/* DSS Support			*/
include/configs/omap3_evm_common.h +0 -2
index 2ef3aaa..b1f11c0 100644
@@ -100,8 +100,6 @@
 
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 
 /*
  * PISMO support
include/configs/omap3_logic.h +0 -2
index b2457d0..629118e 100644
@@ -138,8 +138,6 @@
 
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_I2C_MULTI_BUS
 
 /*
include/configs/omap3_mvblx.h +0 -2
index 873f5b4..c7fecce 100644
@@ -142,8 +142,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		0
-#define CONFIG_SYS_I2C_BUS		0 /* This isn't used anywhere ?? */
-#define CONFIG_SYS_I2C_BUS_SELECT	1 /* This isn't used anywhere ?? */
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 #define CONFIG_I2C_MULTI_BUS		1
include/configs/omap3_pandora.h +0 -2
index 217f306..ee88841 100644
@@ -126,8 +126,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /*
include/configs/omap3_sdp3430.h +0 -2
index b02ec85..19c6a3d 100644
@@ -132,8 +132,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /* OMITTED:  single 1 Gbit MT29F1G NAND flash */
include/configs/omap3_zoom1.h +0 -2
index ee4cbd7..b48f21a 100644
@@ -136,8 +136,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /*
include/configs/omap3_zoom2.h +0 -2
index a6b48a8..27527ce 100644
@@ -156,8 +156,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /*
include/configs/omap4_common.h +0 -2
index a32369a..ca4e724 100644
@@ -91,8 +91,6 @@
 #define CONFIG_HARD_I2C			1
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 #define CONFIG_I2C_MULTI_BUS		1
include/configs/tam3517-common.h +0 -2
index 9f1fb9f..2af504b 100644
@@ -131,8 +131,6 @@
 #define CONFIG_HARD_I2C
 #define CONFIG_SYS_I2C_SPEED		400000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_SYS_I2C_EEPROM_ADDR	0x50		/* base address */
 #define CONFIG_SYS_I2C_EEPROM_ADDR_LEN	1		/* bytes of address */
 #define CONFIG_SYS_I2C_EEPROM_ADDR_OVERFLOW	0x07
include/configs/tricorder.h +0 -2
index bcb0350..1a665ac 100644
@@ -98,8 +98,6 @@
 #define CONFIG_HARD_I2C
 #define CONFIG_SYS_I2C_SPEED		100000
 #define CONFIG_SYS_I2C_SLAVE		1
-#define CONFIG_SYS_I2C_BUS		0
-#define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
 /* TWL4030 */
Signature · 6 lines
1.7.4.1


MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler
Registergericht: Amtsgericht Stuttgart, HRB 271090
Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier
#1 lore raw

PATCH 1/1 Remove unused CONFIG_SYS_I2C_BUS[_SELECT]

Quoted 4 lines · Michael Jones

"CONFIG_SYS_I2C_BUS" and "CONFIG_SYS_I2C_BUS_SELECT" don't appear anywhere outside of config files.

Signed-off-by Michael Jones <[email protected]>

Thanks for the patch!

Acked-by Igor Grinberg <[email protected]>
Quoted 5 lines

---
As far as I can tell, these CONFIG options never have existed outside of these config files - looking at omap3_beagle.h as an example, these lines were inserted with the creation of that file, and at that point in time there was also no occurence of them elsewhere. Can't we get rid of them?

Yes we can and should ;-)

Signature · 2 lines
Regards,
Igor.