Skip to contentSkip to conversation

tty: serial: msm: Fix mask value of RFR level

PATCH In review Pramod Gurav 2 participants 2 messages Apr 7, 2015

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tty: serial: msm: Fix mask value of RFR level

According to documents The RFR_LEVEL1 in UART_DM_MR1 can be programmed in bits 31:8 but the masks only bits 17:8. Correct the same.

---
 drivers/tty/serial/msm_serial.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/tty/serial/msm_serial.h +1 -1
index 8f7806d..5ff9ebf 100644
@@ -19,7 +19,7 @@
 #define UART_MR1			0x0000
 
 #define UART_MR1_AUTO_RFR_LEVEL0	0x3F
-#define UART_MR1_AUTO_RFR_LEVEL1	0x3FF00
+#define UART_MR1_AUTO_RFR_LEVEL1	0xFFFFFF00
 #define UART_MR1_RX_RDY_CTL    		(1 << 7)
 #define UART_MR1_CTS_CTL       		(1 << 6)
 
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
#1 lore raw
Quoted 1 line · Pramod Gurav

According to documents The RFR_LEVEL1 in UART_DM_MR1 can be

That is UART_DM_MR1

Quoted 13 lines

programmed in bits 31:8 but the masks only bits 17:8. Correct the same.

---
 drivers/tty/serial/msm_serial.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/tty/serial/msm_serial.h +0 -0
index 8f7806d..5ff9ebf 100644

drivers/tty/serial/msm_serial.h line 22

Quoted 5 lines
drivers/tty/serial/msm_serial.h +1 -1
@@ -19,7 +19,7 @@
 #define UART_MR1			0x0000
 
 #define UART_MR1_AUTO_RFR_LEVEL0	0x3F
-#define UART_MR1_AUTO_RFR_LEVEL1	0x3FF00
+#define UART_MR1_AUTO_RFR_LEVEL1	0xFFFFFF00

But this is UART_MR1. The two hardware cores share the same driver because the DM hardware is based on the non-DM hardware. I suppose we need to make another #define UART_DM_MR1_AUTO_RFR_LEVEL1 and then have an if (msm_port->is_uartdm) in msm_startup() path to handle the differences here.

Quoted 3 lines
drivers/tty/serial/msm_serial.h +0 -0
 #define UART_MR1_RX_RDY_CTL    		(1 << 7)
 #define UART_MR1_CTS_CTL       		(1 << 6)
 
Signature · 2 lines
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project