usb: chipidea: operate on otgsc register in a general way
authorLi Jun <b47624@freescale.com>
Wed, 23 Apr 2014 07:56:38 +0000 (15:56 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 24 Apr 2014 19:56:34 +0000 (12:56 -0700)
Use a more general way to read and write otgsc register.

Signed-off-by: Peter Chen <peter.chen@freescale.com>
Signed-off-by: Li Jun <b47624@freescale.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/chipidea/core.c
drivers/usb/chipidea/otg.c
drivers/usb/chipidea/otg.h
drivers/usb/chipidea/udc.c

index 1cd5d0ba587c8ebaf49c876a086570be4cabd8c9..f0cfa5b64bf1ac377d241ee95967f248e6dba902 100644 (file)
@@ -393,7 +393,7 @@ static irqreturn_t ci_irq(int irq, void *data)
        u32 otgsc = 0;
 
        if (ci->is_otg)
-               otgsc = hw_read(ci, OP_OTGSC, ~0);
+               otgsc = hw_read_otgsc(ci, ~0);
 
        /*
         * Handle id change interrupt, it indicates device/host function
@@ -401,7 +401,8 @@ static irqreturn_t ci_irq(int irq, void *data)
         */
        if (ci->is_otg && (otgsc & OTGSC_IDIE) && (otgsc & OTGSC_IDIS)) {
                ci->id_event = true;
-               ci_clear_otg_interrupt(ci, OTGSC_IDIS);
+               /* Clear ID change irq status */
+               hw_write_otgsc(ci, OTGSC_IDIS, OTGSC_IDIS);
                disable_irq_nosync(ci->irq);
                queue_work(ci->wq, &ci->work);
                return IRQ_HANDLED;
@@ -413,7 +414,8 @@ static irqreturn_t ci_irq(int irq, void *data)
         */
        if (ci->is_otg && (otgsc & OTGSC_BSVIE) && (otgsc & OTGSC_BSVIS)) {
                ci->b_sess_valid_event = true;
-               ci_clear_otg_interrupt(ci, OTGSC_BSVIS);
+               /* Clear BSV irq */
+               hw_write_otgsc(ci, OTGSC_BSVIS, OTGSC_BSVIS);
                disable_irq_nosync(ci->irq);
                queue_work(ci->wq, &ci->work);
                return IRQ_HANDLED;
@@ -535,8 +537,9 @@ static void ci_get_otg_capable(struct ci_hdrc *ci)
                                        == (DCCPARAMS_DC | DCCPARAMS_HC));
        if (ci->is_otg) {
                dev_dbg(ci->dev, "It is OTG capable controller\n");
-               ci_disable_otg_interrupt(ci, OTGSC_INT_EN_BITS);
-               ci_clear_otg_interrupt(ci, OTGSC_INT_STATUS_BITS);
+               /* Disable and clear all OTG irq */
+               hw_write_otgsc(ci, OTGSC_INT_EN_BITS | OTGSC_INT_STATUS_BITS,
+                                                       OTGSC_INT_STATUS_BITS);
        }
 }
 
@@ -648,7 +651,8 @@ static int ci_hdrc_probe(struct platform_device *pdev)
                         */
                        mdelay(2);
                        ci->role = ci_otg_role(ci);
-                       ci_enable_otg_interrupt(ci, OTGSC_IDIE);
+                       /* Enable ID change irq */
+                       hw_write_otgsc(ci, OTGSC_IDIE, OTGSC_IDIE);
                } else {
                        /*
                         * If the controller is not OTG capable, but support
index 39bd7ec8bf75f53a61e072c3b32090e8cec834c5..c6943401a0b3e42062e0d4d5663ac35277985265 100644 (file)
 #include "bits.h"
 #include "otg.h"
 
+/**
+ * hw_read_otgsc returns otgsc register bits value.
+ * @mask: bitfield mask
+ */
+u32 hw_read_otgsc(struct ci_hdrc *ci, u32 mask)
+{
+       return hw_read(ci, OP_OTGSC, mask);
+}
+
+/**
+ * hw_write_otgsc updates target bits of OTGSC register.
+ * @mask: bitfield mask
+ * @data: to be written
+ */
+void hw_write_otgsc(struct ci_hdrc *ci, u32 mask, u32 data)
+{
+       hw_write(ci, OP_OTGSC, mask | OTGSC_INT_STATUS_BITS, data);
+}
+
 /**
  * ci_otg_role - pick role based on ID pin state
  * @ci: the controller
  */
 enum ci_role ci_otg_role(struct ci_hdrc *ci)
 {
-       u32 sts = hw_read(ci, OP_OTGSC, ~0);
-       enum ci_role role = sts & OTGSC_ID
+       enum ci_role role = hw_read_otgsc(ci, OTGSC_ID)
                ? CI_ROLE_GADGET
                : CI_ROLE_HOST;
 
@@ -39,14 +57,10 @@ enum ci_role ci_otg_role(struct ci_hdrc *ci)
 
 void ci_handle_vbus_change(struct ci_hdrc *ci)
 {
-       u32 otgsc;
-
        if (!ci->is_otg)
                return;
 
-       otgsc = hw_read(ci, OP_OTGSC, ~0);
-
-       if (otgsc & OTGSC_BSV)
+       if (hw_read_otgsc(ci, OTGSC_BSV))
                usb_gadget_vbus_connect(&ci->gadget);
        else
                usb_gadget_vbus_disconnect(&ci->gadget);
@@ -115,6 +129,7 @@ void ci_hdrc_otg_destroy(struct ci_hdrc *ci)
                flush_workqueue(ci->wq);
                destroy_workqueue(ci->wq);
        }
-       ci_disable_otg_interrupt(ci, OTGSC_INT_EN_BITS);
-       ci_clear_otg_interrupt(ci, OTGSC_INT_STATUS_BITS);
+       /* Disable all OTG irq and clear status */
+       hw_write_otgsc(ci, OTGSC_INT_EN_BITS | OTGSC_INT_STATUS_BITS,
+                                               OTGSC_INT_STATUS_BITS);
 }
index 449bee07f4fe00a9e9f4f0e2f8b297f1d1518090..7349267541130c9388b67466e0fe15bdf7595bec 100644 (file)
 #ifndef __DRIVERS_USB_CHIPIDEA_OTG_H
 #define __DRIVERS_USB_CHIPIDEA_OTG_H
 
-static inline void ci_clear_otg_interrupt(struct ci_hdrc *ci, u32 bits)
-{
-       /* Only clear request bits */
-       hw_write(ci, OP_OTGSC, OTGSC_INT_STATUS_BITS, bits);
-}
-
-static inline void ci_enable_otg_interrupt(struct ci_hdrc *ci, u32 bits)
-{
-       hw_write(ci, OP_OTGSC, bits | OTGSC_INT_STATUS_BITS, bits);
-}
-
-static inline void ci_disable_otg_interrupt(struct ci_hdrc *ci, u32 bits)
-{
-       hw_write(ci, OP_OTGSC, bits | OTGSC_INT_STATUS_BITS, 0);
-}
-
+u32 hw_read_otgsc(struct ci_hdrc *ci, u32 mask);
+void hw_write_otgsc(struct ci_hdrc *ci, u32 mask, u32 data);
 int ci_hdrc_otg_init(struct ci_hdrc *ci);
 void ci_hdrc_otg_destroy(struct ci_hdrc *ci);
 enum ci_role ci_otg_role(struct ci_hdrc *ci);
index 7739c64ef2590a1dbdd0daf8e631b90cf99af628..798943b6ef7b5371988e124c83e127906a19caea 100644 (file)
@@ -1843,21 +1843,22 @@ void ci_hdrc_gadget_destroy(struct ci_hdrc *ci)
 
 static int udc_id_switch_for_device(struct ci_hdrc *ci)
 {
-       if (ci->is_otg) {
-               ci_clear_otg_interrupt(ci, OTGSC_BSVIS);
-               ci_enable_otg_interrupt(ci, OTGSC_BSVIE);
-       }
+       if (ci->is_otg)
+               /* Clear and enable BSV irq */
+               hw_write_otgsc(ci, OTGSC_BSVIS | OTGSC_BSVIE,
+                                       OTGSC_BSVIS | OTGSC_BSVIE);
 
        return 0;
 }
 
 static void udc_id_switch_for_host(struct ci_hdrc *ci)
 {
-       if (ci->is_otg) {
-               /* host doesn't care B_SESSION_VALID event */
-               ci_clear_otg_interrupt(ci, OTGSC_BSVIS);
-               ci_disable_otg_interrupt(ci, OTGSC_BSVIE);
-       }
+       /*
+        * host doesn't care B_SESSION_VALID event
+        * so clear and disbale BSV irq
+        */
+       if (ci->is_otg)
+               hw_write_otgsc(ci, OTGSC_BSVIE | OTGSC_BSVIS, OTGSC_BSVIS);
 }
 
 /**