From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pali =?ISO-8859-1?Q?Roh=E1r?= Date: Mon, 27 Feb 2012 19:40:07 +0100 Subject: [U-Boot] [PATCH 05/15] common/main.c: Fix function readline In-Reply-To: <201202262333.03150.vapier@gentoo.org> References: <1778406.ezGto4lKnu@pali-elitebook> <7289416.t84EK1VC5H@pali-elitebook> <201202262333.03150.vapier@gentoo.org> Message-ID: <1819996.OSAkOTJcMA@pali> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sunday 26 February 2012 23:33:01 Mike Frysinger wrote: > On Saturday 07 January 2012 03:51:16 Pali Roh?r wrote: > > On Friday 06 January 2012 15:15:29 Mike Frysinger wrote: > > > On Sunday 18 December 2011 16:34:01 Pali Roh?r wrote: > > > > * Ignore ANSI escape codes for moving cursor, which are generated by > > > > > > > > keyboard > > > > > > this probably should be behind an appropriate CONFIG knob. i don't > > > think > > > this is relevant to serial users (which are the majority of boards). > > > > I think this is also relevant to serial console too. I tried serial > > console > > (over USB) with Nokia N900 on computer (ANSI terminal) and if I pressed > > cursor key (UP/DOWN/LEFT/RIGHT) it moved cursor. But in readline function > > buffer was stored ANSI sequence. So when I pressed ENTER, u-boot tried to > > execute that buffer and showed me error. But it showed error message > > something like unknown command . ansi_seq was recognized in ansi > > terminal on computer and it garbaged output. (for example if asn_seq was > > move up, error message continued on previous line...) > > > > This patch ignore ansi codes which can be generated by curosor keys, so > > there will not be that bad error messages... > > do you have CONFIG_CMDLINE_EDITING enabled ? arrow keys work just fine when > that's enabled, and if it isn't, then i think it's expected behavior that > pressing the arrow keys would generate "junk" that u-boot wouldn't > interpret. -mike I had CONFIG_CMDLINE_EDITING disabled - I did not know this option. Anyway, when I enabled it arrow keys does not worked with cfb video driver and rx51 platform code. I will try to fix that problem and I then will delete this patch. -- Pali Roh?r pali.rohar at gmail.com -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part. URL: