USB: c67x00: add proper delays to HPI read/write
authorMax Filippov <jcmvbkbc@gmail.com>
Wed, 25 Dec 2013 12:01:30 +0000 (16:01 +0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jan 2014 00:15:39 +0000 (16:15 -0800)
According to CY7C67300 specification HPI read and write cycle duration
Tcyc must be at least 6T long, where T is 1/48MHz, which is 125ns.
Without this delay fast host processor cannot write to chip registers.
Add proper ndelay to hpi_{read,write}_reg.

Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
Acked-by: Peter Korsgaard <peter@korsgaard.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/c67x00/c67x00-ll-hpi.c

index 3a1ca4dfc83ac26aa4af5e0e6ea68a24962d0e1e..cd2c6ed67493ef5af6564338a474d6ffe3efe94c 100644 (file)
@@ -22,6 +22,7 @@
  */
 
 #include <asm/byteorder.h>
+#include <linux/delay.h>
 #include <linux/io.h>
 #include <linux/jiffies.h>
 #include <linux/usb/c67x00.h>
@@ -73,13 +74,22 @@ struct c67x00_lcp_int_data {
 #define HPI_ADDR       2
 #define HPI_STATUS     3
 
+/*
+ * According to CY7C67300 specification (tables 140 and 141) HPI read and
+ * write cycle duration Tcyc must be at least 6T long, where T is 1/48MHz,
+ * which is 125ns.
+ */
+#define HPI_T_CYC_NS   125
+
 static inline u16 hpi_read_reg(struct c67x00_device *dev, int reg)
 {
+       ndelay(HPI_T_CYC_NS);
        return __raw_readw(dev->hpi.base + reg * dev->hpi.regstep);
 }
 
 static inline void hpi_write_reg(struct c67x00_device *dev, int reg, u16 value)
 {
+       ndelay(HPI_T_CYC_NS);
        __raw_writew(value, dev->hpi.base + reg * dev->hpi.regstep);
 }