servod: Disable ID eeprom access on mimic flex.

Bug in id_eeprom control for the onboard 128Byte EEPROM on mimic
flexes w/ the PCA9500 chip.  As we don't use this EEPROM and the
PCA9500 has been deprecated for future flexes due to leakage issue I'm
removing controls.

Additionally added exceptions if controls are re-added without
addressing the side-effect.

BUG=chromium:294248
TEST=manual
  1. sudo servod -b <board w/ mimic flex>
  2. dut-control spi2_vref:pp1800 spi1_vref:pp1800
  3. dut-control spi2_buf_on_flex_en:off spi1_buf_on_flex_en:off
  3. dut-control | grep spi[12]_buf_on_flex_en

returns
  spi2_buf_on_flex_en:off
  spi1_buf_on_flex_en:off

Change-Id: I094431934bffa648d0682d6ebd30814266a15276
Reviewed-on: https://chromium-review.googlesource.com/169872
Reviewed-by: Simran Basi <sbasi@chromium.org>
Commit-Queue: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
diff --git a/servo/data/servoflex_v1_r1.xml b/servo/data/servoflex_v1_r1.xml
index c0c5fe9..766b464 100644
--- a/servo/data/servoflex_v1_r1.xml
+++ b/servo/data/servoflex_v1_r1.xml
@@ -51,17 +51,4 @@
     <params interface="2" servo_v3_interface="4" drv="pca9500" subtype="gpio"
     slv="0x27" offset="7" map="onoff_i"></params>
   </control>
-  <control>
-    <name>id_eeprom_baddr</name>
-    <doc>Set ID EEPROM byte address to write to</doc>
-    <params cmd="set" interface="2" servo_v3_interface="4" drv="pca9500"
-    subtype="byte_addr" slv="0x57"></params>
-  </control>
-  <control>
-    <name>id_eeprom</name>
-    <doc>ID EEPROM (256bytes). Write starting at
-    id_eeprom_baddr or read entire EEPROM</doc>
-    <params interface="2" servo_v3_interface="4" drv="pca9500" subtype="eeprom"
-    slv="0x57" input_type="str"></params>
-  </control>
 </root>
diff --git a/servo/data/servoflex_v2_r0_p50.xml b/servo/data/servoflex_v2_r0_p50.xml
index 14f6890..381b5ab 100644
--- a/servo/data/servoflex_v2_r0_p50.xml
+++ b/servo/data/servoflex_v2_r0_p50.xml
@@ -18,17 +18,4 @@
     <params interface="2" servo_v3_interface="4" drv="pca95xx" subtype="gpio"
     slv="0x26" offset="7" map="onoff_i" clobber_ok=""></params>
   </control>
-  <control>
-    <name>id_eeprom_baddr</name>
-    <doc>Set ID EEPROM byte address to write to</doc>
-    <params cmd="set" interface="2" servo_v3_interface="4" drv="pca95xx"
-    subtype="byte_addr" slv="0x26"></params>
-  </control>
-  <control>
-    <name>id_eeprom</name>
-    <doc>ID EEPROM (256bytes). Write starting at id_eeprom_baddr or
-    read entire EEPROM</doc>
-    <params interface="2" servo_v3_interface="4" drv="pca95xx" subtype="eeprom"
-    slv="0x26" input_type="str"></params>
-  </control>
 </root>
diff --git a/servo/drv/pca9500.py b/servo/drv/pca9500.py
index 0a5bcb1..d273f9b 100644
--- a/servo/drv/pca9500.py
+++ b/servo/drv/pca9500.py
@@ -133,7 +133,7 @@
       pca9500Error: if byte_addr > EEPROM_BYTES
 
     """
-    self._logger.debug('')
+    raise pca9500Error, 'Fix crbug.com/294248'
     if byte_addr > EEPROM_BYTES:
       raise pca9500Error, 'Byte address not valid'
     pca9500._byte_addr = byte_addr
@@ -163,7 +163,7 @@
       pca9500Error: if I2c write failed to complete successfully
 
     """
-    self._logger.debug('')
+    raise pca9500Error, 'Fix crbug.com/294248'
     byte_list = [int(byte_str, 0) for byte_str in value.split()]
     self._write_byte_addr(pca9500._byte_addr)
 
@@ -203,7 +203,7 @@
     Raises:
       pca9500Error: if I2c read failed to complete successfully
     """
-    self._logger.debug('')
+    raise pca9500Error, 'Fix crbug.com/294248'
     error = False
     self._write_byte_addr(0)
     try: