linux-kernel - RE: [PATCH 1/1] input: keyboard: Set configuration registers

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]

Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF178E1244B3@HQMAIL01.nvidia.com>
Date:	Tue, 31 Jan 2012 11:51:36 -0800
From:	Stephen Warren <swarren@...dia.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
CC:	Shridhar Rasal <srasal@...dia.com>,
	"rydberg@...omail.se" <rydberg@...omail.se>,
	Rakesh Iyer <riyer@...dia.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
	"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>
Subject: RE: [PATCH 1/1] input: keyboard: Set configuration registers

Dmitry Torokhov wrote at Tuesday, January 31, 2012 12:48 PM:
> On Tue, Jan 31, 2012 at 11:29:35AM -0800, Stephen Warren wrote:
> > Dmitry Torokhov wrote at Tuesday, January 31, 2012 12:09 PM:
> > > On Thu, Dec 29, 2011 at 07:25:59PM -0800, Dmitry Torokhov wrote:
> > > > On Fri, Dec 09, 2011 at 03:17:54PM -0800, Stephen Warren wrote:
...
> > > > > However, I suppose if we were to change the structure of the pdata to be:
> > > > >
> > > > > struct tegra_kbc_pin_cfg {
> > > > >     bool is_row;
> > > > >     u8 row_col_id;
> > > > >     u8 pin_id;
> > > > > };
> > > > >
> > > > > Then struct tegra_kbc_platform_data could indeed have a pointer to a
> > > > > variable-sized array of these, which would avoid the "en" member.
> > > > >
> > > >
> > > > How about we change bool to enum instead, like in the patch below?
...
> > I'd rather PIN_CFG_IGNORE was the first enum, so it gets value 0, or this
> > probably won't solve the original issue?
> 
> The benefit of current definition is that it is compatible with old
> ones using bool...

There are no in-tree users of this type yet, so compatibility isn't
really an issue. And if we don't make that change, anyone who does start
to use it is going to have to initialize every element of the array even
when they aren't used.

-- 
nvpublic

--
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/