LCDproc development and user support list

Text archives Help


[Lcdproc] [PATCH] include asm/termbits.h instead of termios.h in hd44780-serial.c


Chronological Thread 
  • From: foxcore AT gmail.com (FoX)
  • Subject: [Lcdproc] [PATCH] include asm/termbits.h instead of termios.h in hd44780-serial.c
  • Date: Fri Apr 27 15:24:01 2007

2007/4/27, FoX
<foxcore AT gmail.com>:
And I found the reason inside bits/termios.h from uClibc, included by
<termios.h> :

#if 0 /* limited on uClibc, keep in sync w/ cfsetspeed.c */
#define B230400 0010003
#define B460800 0010004
#define B500000 0010005
#define B576000 0010006
#define B921600 0010007
#define B1000000 0010010
#define B1152000 0010011
#define B1500000 0010012
#define B2000000 0010013
#define B2500000 0010014
#define B3000000 0010015
#define B3500000 0010016
#define B4000000 0010017
#define __MAX_BAUD B4000000
#else
#define __MAX_BAUD B115200
#endif

So I think that the special geexbox case can be enlarged to every
uClibc crosscompilation.

This simple patch solves the problem.
Is it acceptable for being included in trunk ?

--
Guillaume LECERF
GeeXboX developer
--- lcdproc-0.5.2/server/drivers/hd44780-serial.c.orig	2007-04-27 17:19:25.000000000 +0200
+++ lcdproc-0.5.2/server/drivers/hd44780-serial.c	2007-04-27 17:19:56.000000000 +0200
@@ -66,8 +66,10 @@
 	{ 19200, B19200 },
 	{ 38400, B38400 },
 	{ 57600, B57600 },
-	{ 115200, B115200 },
-	{ 230400, B230400 }
+	{ 115200, B115200 }
+#if defined(B230400)
+	, { 230400, B230400 }
+#endif
 #if defined(B460800)	
 	, { 460800, B460800 }
 #endif



Archive powered by MHonArc 2.6.18.

Top of page