aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/serial/imx.c
diff options
context:
space:
mode:
authorDaniel Glöckner <dg@emlix.com>2009-06-11 09:36:29 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2009-06-11 11:51:08 -0400
commit26bbb3ff1ff6163d6a233055766e26af8054a212 (patch)
treeba50f720a04c0fd3a34bb8a29c362d24c8585b03 /drivers/serial/imx.c
parentd3810cd4d7064b109574374f73c41b11b481dbf2 (diff)
imx: serial: fix one bit field type
"have_rtscts" is assigned 1, while it is declared int:1, two's complement, which can hold 0 and -1 only. The code works, as the upper bits are cut off, and tests are done against 0 only. Nonetheless, correctly declaring the bit field as unsigned int:1 renders the code more robust. Signed-off-by: Daniel Glöckner <dg@emlix.com> Signed-off-by: Oskar Schirmer <os@emlix.com> Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/serial/imx.c')
-rw-r--r--drivers/serial/imx.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/serial/imx.c b/drivers/serial/imx.c
index 76c8fa1884ec..6b8f12f4a705 100644
--- a/drivers/serial/imx.c
+++ b/drivers/serial/imx.c
@@ -211,7 +211,7 @@ struct imx_port {
211 struct timer_list timer; 211 struct timer_list timer;
212 unsigned int old_status; 212 unsigned int old_status;
213 int txirq,rxirq,rtsirq; 213 int txirq,rxirq,rtsirq;
214 int have_rtscts:1; 214 unsigned int have_rtscts:1;
215 struct clk *clk; 215 struct clk *clk;
216}; 216};
217 217