From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Andr=c3=a9_Przywara?= Date: Mon, 1 Apr 2019 01:04:02 +0100 Subject: [U-Boot] [PATCH 7/8] usb: kbd: Properly translate up/down arrow keys In-Reply-To: References: <20190323013002.27117-1-andre.przywara@arm.com> <20190323013002.27117-8-andre.przywara@arm.com> Message-ID: <74563db4-0e2c-9fbf-70e7-3987b9c8a059@arm.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 30/03/2019 21:18, Simon Glass wrote: > On Fri, 22 Mar 2019 at 19:32, Andre Przywara wrote: Hi Simon, many thanks for the review of all those patches, much appreciated! >> So far arrows key pressed on an USB keyboard got translated to some >> low ASCII control sequences (Ctrl+N, Ctrl+P). Some programs understand >> these codes, but the standard for those keys is to use ANSI control > > Which standard? The only real standard for encoding arrow keys seems to be "ANSI terminal" escape sequences, I think ECMA-48 is the official name(?) Also since our very own U-Boot boot menu requires this ... Cheers, Andre >> sequences for cursor movement (ESC [ A). >> Our own boot menu is a victim of this, currently we cannot change the >> selection with an USB keyboard due to this. >> >> Since we already implement a queue for USB key codes, we can just insert >> the three character ANSI sequence into the key buffer. This fixes the >> bootmenu, and is more universal for other users (UEFI) as well. >> >> Signed-off-by: Andre Przywara >> --- >> common/usb_kbd.c | 24 +++++++++++++++++++++++- >> 1 file changed, 23 insertions(+), 1 deletion(-) > > Reviewed-by: Simon Glass >