linux-kernel - RE: [PATCH 1/1] input: tegra-kbc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 12 Jan 2011 14:13:00 -0800
From:	Rakesh Iyer <riyer@...dia.com>
To:	'Pavel Machek' <pavel@....cz>
CC:	"tsoni@...eaurora.org" <tsoni@...eaurora.org>,
	"dmitry.torokhov@...il.com" <dmitry.torokhov@...il.com>,
	"shubhrajyoti@...com" <shubhrajyoti@...com>,
	"ccross@...roid.com" <ccross@...roid.com>,
	"konkers@...roid.com" <konkers@...roid.com>,
	"olof@...om.net" <olof@...om.net>, Andrew Chew <AChew@...dia.com>,
	"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>
Subject: RE: [PATCH 1/1] input: tegra-kbc - Add tegra keyboard driver

Here are the list of changes. I was not sure of the protocol of adding this into the patch email.

a) Addressed all of Trilok Soni's comments excluding the tegra clk programming which is necessary.
Use correct labels in tegra_kbc_probe error paths. 
Identify and if necessary document all register accesses without using magic numbers.
Modify suspend to check for valid users before proceeding to close.
 
b) Removed the Fn keymap and its references. This obviated the need for some helper functions.

c) Setup the input device's keycodexx parameters.


Pavel's query

> But now I'm confused; does hardware have Fx keys (F10 above) or not?
> 									Pavel

The function key for this keyboard is separate from the F1 to F12 keys, where the latter are always available.

The function key serves just like a SHIFT key to overload the use of certain keys e.g. Fn+PgUP-> "Home", Fn+PgDn -> "End"

Regards
Rakesh
> -----Original Message-----
> From: Pavel Machek [mailto:pavel@....cz]
> Sent: Wednesday, January 12, 2011 2:03 PM
> To: Rakesh Iyer
> Cc: tsoni@...eaurora.org; dmitry.torokhov@...il.com; shubhrajyoti@...com;
> ccross@...roid.com; konkers@...roid.com; olof@...om.net; Andrew Chew; linux-
> tegra@...r.kernel.org; linux-kernel@...r.kernel.org; linux-input@...r.kernel.org
> Subject: Re: [PATCH 1/1] input: tegra-kbc - Add tegra keyboard driver
> 
> On Wed 2011-01-12 13:59:45, Rakesh Iyer wrote:
> > Hi all.
> >
> > I apologize for not specifying the PATCH indicator as "PATCH v4".
> >
> > Sorry for the inconvenience caused.
> 
> Well, that is not huge problem, but short note of changes from last
> version would be welcome.
> 
> AFAICT, FN handling was reworked...?
> 
> > > +     KEY_F10, KEY_F9, KEY_BACKSPACE, KEY_3,
> > > +             KEY_2, KEY_UP, KEY_PRINT, KEY_PAUSE,
> 
> But now I'm confused; does hardware have Fx keys (F10 above) or not?
> 									Pavel
> 
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/