All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Jones <pjones@redhat.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: [PATCH] Make CTRL and ALT keys work as expected on EFI systems (version 5).
Date: Mon, 12 Oct 2015 10:19:09 -0400	[thread overview]
Message-ID: <20151012141909.GA19125@redhat.com> (raw)
In-Reply-To: <56195D9A.2010306@gmail.com>

On Sat, Oct 10, 2015 at 09:48:58PM +0300, Andrei Borzenkov wrote:
> 26.02.2014 02:12, Peter Jones пишет:
> >This is version 4.
> >
> >Changes from version 1:
> >- handles SHIFT as a modifier
> >- handles F11 and F12 keys
> >- uses the handle provided by the system table to find our _EX protocol.
> >
> >Changes from version 2:
> >- eliminate duplicate keycode translation.
> >
> >Changes from version 3:
> >- Do not add the shift modifier for any ascii character between space
> >   (0x20) and DEL (0x7f); the combination of the modifier and many of the
> >   keys causes it not to be recognized at all.  Specifically, if we
> >   include the modifier on any querty punctuation character, i.e.
> >   anything the string "~!@#$%^&*()_+{}|:\"<>?" represents in C, it stops
> >   being recognized whatsoever.
> >
> >Changes from version 4:
> >- Always initialize term->data from locate protocol (i.e. make it
> >   unconditional.)
> >
> 
> Are there open issues with this patch? Is it used by Fedora? The part about
> SHIFT state bothers me, what happens for non-ASCII printable characters?
> UEFI spec is extremely vague here.
> 
> As currently there is no way to actually input Ctrl-X or similar this is
> needed. It may also allow us to actually implement keystatus on EFI.

Fedora is using it, and has been for 18 months or so.  As far as I know,
there aren't any known issues.

-- 
        Peter


  reply	other threads:[~2015-10-12 14:19 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-25 22:12 [PATCH] Make CTRL and ALT keys work as expected on EFI systems (version 5) Peter Jones
2014-02-26  2:58 ` Mroczek, Joseph T
2014-02-26 18:51   ` Vladimir 'φ-coder/phcoder' Serbinenko
2015-10-10 18:48 ` Andrei Borzenkov
2015-10-12 14:19   ` Peter Jones [this message]
2015-10-12 14:47   ` Peter Jones
2015-10-13  3:50     ` Andrei Borzenkov
2015-10-25 15:41 ` Vladimir 'φ-coder/phcoder' Serbinenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151012141909.GA19125@redhat.com \
    --to=pjones@redhat.com \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.