greybus: interface: add ES2 init-status quirk flag

Add ES2 init-status quirk flag instead of checking MID/PID directly.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

diff --git a/drivers/staging/greybus/interface.c b/drivers/staging/greybus/interface.c
index c7793a93c8027d0d2a8f9154ca6770e6b6a01e6b..767bf87b2dbd72bc2c09834caac4cfde6f9de458 100644 (file)
--- a/drivers/staging/greybus/interface.c
+++ b/drivers/staging/greybus/interface.c
@@ -78,6 +78,11 @@ static int gb_interface_read_dme(struct gb_interface *intf)
        if (ret)
                return ret;
 
+       if (intf->ddbl1_manufacturer_id == TOSHIBA_DMID &&
+                       intf->ddbl1_product_id == TOSHIBA_ES2_BRIDGE_DPID) {
+               intf->quirks |= GB_INTERFACE_QUIRK_NO_INIT_STATUS;
+       }
+
        return gb_interface_read_ara_dme(intf);
 }
 
@@ -155,17 +160,13 @@ static int gb_interface_read_and_clear_init_status(struct gb_interface *intf)
        u32 value;
        u16 attr;
        u8 init_status;
-       bool es2_bridge;
-
-       es2_bridge = intf->ddbl1_manufacturer_id == ES2_DDBL1_MFR_ID &&
-                       intf->ddbl1_product_id == ES2_DDBL1_PROD_ID;
 
        /*
         * ES2 bridges use T_TstSrcIncrement for the init status.
         *
         * FIXME: Remove ES2 support
         */
-       if (es2_bridge)
+       if (intf->quirks & GB_INTERFACE_QUIRK_NO_INIT_STATUS)
                attr = DME_ATTR_T_TST_SRC_INCREMENT;
        else
                attr = DME_ATTR_ES3_INIT_STATUS;
@@ -192,7 +193,7 @@ static int gb_interface_read_and_clear_init_status(struct gb_interface *intf)
         *
         * FIXME: Remove ES2 support
         */
-       if (es2_bridge)
+       if (intf->quirks & GB_INTERFACE_QUIRK_NO_INIT_STATUS)
                init_status = value & 0xff;
        else
                init_status = value >> 24;

diff --git a/drivers/staging/greybus/interface.h b/drivers/staging/greybus/interface.h
index 1a6ce5ce6e6c95179664957f74ba4da294ac4bba..94eb36e369754530d0829dacb6374da9e4e3dcb7 100644 (file)
--- a/drivers/staging/greybus/interface.h
+++ b/drivers/staging/greybus/interface.h
@@ -12,6 +12,7 @@
 
 #define GB_INTERFACE_QUIRK_NO_CPORT_FEATURES           BIT(0)
 #define GB_INTERFACE_QUIRK_NO_INTERFACE_VERSION                BIT(1)
+#define GB_INTERFACE_QUIRK_NO_INIT_STATUS              BIT(2)
 
 struct gb_interface {
        struct device dev;