aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorJiri Slaby <jslaby@suse.cz>2012-04-02 07:53:52 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-04-09 14:16:54 -0400
commit37630f406338de4a5f72a42c0b6a034657dce383 (patch)
tree11212fa0f1b2481e60af871b0bb77e483adf5963 /drivers
parentce93d33cf45db9893cfc725a2b470da15bfe4435 (diff)
TTY: isdn, make some functions readable
Huh, this was a mess. * Remove the 5 indent levels by just returning from isdn_tty_try_read when the conditions there are not satisfied. * Use 'continue' in loop in isdn_tty_readmodem to save 2 levels. * Chain 'if's in isdn_tty_modem_escape. * Use 'continue' in loop in isdn_tty_carrier_timeout to save 1 level. Better to look at this patch with -w -b. Signed-off-by: Jiri Slaby <jslaby@suse.cz> Cc: Karsten Keil <isdn@linux-pingi.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/isdn/i4l/isdn_tty.c178
1 files changed, 93 insertions, 85 deletions
diff --git a/drivers/isdn/i4l/isdn_tty.c b/drivers/isdn/i4l/isdn_tty.c
index 1ff307276d2d..74b05188c18f 100644
--- a/drivers/isdn/i4l/isdn_tty.c
+++ b/drivers/isdn/i4l/isdn_tty.c
@@ -65,49 +65,54 @@ isdn_tty_try_read(modem_info *info, struct sk_buff *skb)
65 struct tty_struct *tty; 65 struct tty_struct *tty;
66 char last; 66 char last;
67 67
68 if (info->online) { 68 if (!info->online)
69 if ((tty = info->tty)) { 69 return 0;
70 if (info->mcr & UART_MCR_RTS) { 70
71 len = skb->len 71 tty = info->tty;
72 if (!tty)
73 return 0;
74
75 if (!(info->mcr & UART_MCR_RTS))
76 return 0;
77
78 len = skb->len
72#ifdef CONFIG_ISDN_AUDIO 79#ifdef CONFIG_ISDN_AUDIO
73 + ISDN_AUDIO_SKB_DLECOUNT(skb) 80 + ISDN_AUDIO_SKB_DLECOUNT(skb)
74#endif 81#endif
75 ; 82 ;
83
84 c = tty_buffer_request_room(tty, len);
85 if (c < len)
86 return 0;
76 87
77 c = tty_buffer_request_room(tty, len);
78 if (c >= len) {
79#ifdef CONFIG_ISDN_AUDIO
80 if (ISDN_AUDIO_SKB_DLECOUNT(skb)) {
81 int l = skb->len;
82 unsigned char *dp = skb->data;
83 while (--l) {
84 if (*dp == DLE)
85 tty_insert_flip_char(tty, DLE, 0);
86 tty_insert_flip_char(tty, *dp++, 0);
87 }
88 if (*dp == DLE)
89 tty_insert_flip_char(tty, DLE, 0);
90 last = *dp;
91 } else {
92#endif
93 if (len > 1)
94 tty_insert_flip_string(tty, skb->data, len - 1);
95 last = skb->data[len - 1];
96#ifdef CONFIG_ISDN_AUDIO 88#ifdef CONFIG_ISDN_AUDIO
97 } 89 if (ISDN_AUDIO_SKB_DLECOUNT(skb)) {
90 int l = skb->len;
91 unsigned char *dp = skb->data;
92 while (--l) {
93 if (*dp == DLE)
94 tty_insert_flip_char(tty, DLE, 0);
95 tty_insert_flip_char(tty, *dp++, 0);
96 }
97 if (*dp == DLE)
98 tty_insert_flip_char(tty, DLE, 0);
99 last = *dp;
100 } else {
98#endif 101#endif
99 if (info->emu.mdmreg[REG_CPPP] & BIT_CPPP) 102 if (len > 1)
100 tty_insert_flip_char(tty, last, 0xFF); 103 tty_insert_flip_string(tty, skb->data, len - 1);
101 else 104 last = skb->data[len - 1];
102 tty_insert_flip_char(tty, last, TTY_NORMAL); 105#ifdef CONFIG_ISDN_AUDIO
103 tty_flip_buffer_push(tty);
104 kfree_skb(skb);
105 return 1;
106 }
107 }
108 }
109 } 106 }
110 return 0; 107#endif
108 if (info->emu.mdmreg[REG_CPPP] & BIT_CPPP)
109 tty_insert_flip_char(tty, last, 0xFF);
110 else
111 tty_insert_flip_char(tty, last, TTY_NORMAL);
112 tty_flip_buffer_push(tty);
113 kfree_skb(skb);
114
115 return 1;
111} 116}
112 117
113/* isdn_tty_readmodem() is called periodically from within timer-interrupt. 118/* isdn_tty_readmodem() is called periodically from within timer-interrupt.
@@ -125,35 +130,39 @@ isdn_tty_readmodem(void)
125 modem_info *info; 130 modem_info *info;
126 131
127 for (i = 0; i < ISDN_MAX_CHANNELS; i++) { 132 for (i = 0; i < ISDN_MAX_CHANNELS; i++) {
128 if ((midx = dev->m_idx[i]) >= 0) { 133 midx = dev->m_idx[i];
129 info = &dev->mdm.info[midx]; 134 if (midx < 0)
130 if (info->online) { 135 continue;
131 r = 0; 136
137 info = &dev->mdm.info[midx];
138 if (!info->online)
139 continue;
140
141 r = 0;
132#ifdef CONFIG_ISDN_AUDIO 142#ifdef CONFIG_ISDN_AUDIO
133 isdn_audio_eval_dtmf(info); 143 isdn_audio_eval_dtmf(info);
134 if ((info->vonline & 1) && (info->emu.vpar[1])) 144 if ((info->vonline & 1) && (info->emu.vpar[1]))
135 isdn_audio_eval_silence(info); 145 isdn_audio_eval_silence(info);
136#endif 146#endif
137 if ((tty = info->tty)) { 147 tty = info->tty;
138 if (info->mcr & UART_MCR_RTS) { 148 if (tty) {
139 /* CISCO AsyncPPP Hack */ 149 if (info->mcr & UART_MCR_RTS) {
140 if (!(info->emu.mdmreg[REG_CPPP] & BIT_CPPP)) 150 /* CISCO AsyncPPP Hack */
141 r = isdn_readbchan_tty(info->isdn_driver, info->isdn_channel, tty, 0); 151 if (!(info->emu.mdmreg[REG_CPPP] & BIT_CPPP))
142 else 152 r = isdn_readbchan_tty(info->isdn_driver, info->isdn_channel, tty, 0);
143 r = isdn_readbchan_tty(info->isdn_driver, info->isdn_channel, tty, 1); 153 else
144 if (r) 154 r = isdn_readbchan_tty(info->isdn_driver, info->isdn_channel, tty, 1);
145 tty_flip_buffer_push(tty); 155 if (r)
146 } else 156 tty_flip_buffer_push(tty);
147 r = 1; 157 } else
148 } else 158 r = 1;
149 r = 1; 159 } else
150 if (r) { 160 r = 1;
151 info->rcvsched = 0; 161 if (r) {
152 resched = 1; 162 info->rcvsched = 0;
153 } else 163 resched = 1;
154 info->rcvsched = 1; 164 } else
155 } 165 info->rcvsched = 1;
156 }
157 } 166 }
158 if (!resched) 167 if (!resched)
159 isdn_timer_ctrl(ISDN_TIMER_MODEMREAD, 0); 168 isdn_timer_ctrl(ISDN_TIMER_MODEMREAD, 0);
@@ -3769,19 +3778,19 @@ isdn_tty_modem_escape(void)
3769 int midx; 3778 int midx;
3770 3779
3771 for (i = 0; i < ISDN_MAX_CHANNELS; i++) 3780 for (i = 0; i < ISDN_MAX_CHANNELS; i++)
3772 if (USG_MODEM(dev->usage[i])) 3781 if (USG_MODEM(dev->usage[i]) && (midx = dev->m_idx[i]) >= 0) {
3773 if ((midx = dev->m_idx[i]) >= 0) { 3782 modem_info *info = &dev->mdm.info[midx];
3774 modem_info *info = &dev->mdm.info[midx]; 3783 if (info->online) {
3775 if (info->online) { 3784 ton = 1;
3776 ton = 1; 3785 if ((info->emu.pluscount == 3) &&
3777 if ((info->emu.pluscount == 3) && 3786 time_after(jiffies,
3778 time_after(jiffies , info->emu.lastplus + PLUSWAIT2)) { 3787 info->emu.lastplus + PLUSWAIT2)) {
3779 info->emu.pluscount = 0; 3788 info->emu.pluscount = 0;
3780 info->online = 0; 3789 info->online = 0;
3781 isdn_tty_modem_result(RESULT_OK, info); 3790 isdn_tty_modem_result(RESULT_OK, info);
3782 }
3783 } 3791 }
3784 } 3792 }
3793 }
3785 isdn_timer_ctrl(ISDN_TIMER_MODEMPLUS, ton); 3794 isdn_timer_ctrl(ISDN_TIMER_MODEMPLUS, ton);
3786} 3795}
3787 3796
@@ -3839,15 +3848,14 @@ isdn_tty_carrier_timeout(void)
3839 3848
3840 for (i = 0; i < ISDN_MAX_CHANNELS; i++) { 3849 for (i = 0; i < ISDN_MAX_CHANNELS; i++) {
3841 modem_info *info = &dev->mdm.info[i]; 3850 modem_info *info = &dev->mdm.info[i];
3842 if (info->dialing) { 3851 if (!info->dialing)
3843 if (info->emu.carrierwait++ > info->emu.mdmreg[REG_WAITC]) { 3852 continue;
3844 info->dialing = 0; 3853 if (info->emu.carrierwait++ > info->emu.mdmreg[REG_WAITC]) {
3845 isdn_tty_modem_result(RESULT_NO_CARRIER, info); 3854 info->dialing = 0;
3846 isdn_tty_modem_hup(info, 1); 3855 isdn_tty_modem_result(RESULT_NO_CARRIER, info);
3847 } 3856 isdn_tty_modem_hup(info, 1);
3848 else 3857 } else
3849 ton = 1; 3858 ton = 1;
3850 }
3851 } 3859 }
3852 isdn_timer_ctrl(ISDN_TIMER_CARRIER, ton); 3860 isdn_timer_ctrl(ISDN_TIMER_CARRIER, ton);
3853} 3861}