linux-kernel - RE: [RFC PATCH 0/1] input/touchscreen: Synaptics Touchscreen Driver

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, 25 Aug 2010 11:05:09 -0700
From:	William Manson <WManson@...aptics.com>
To:	'Naveen Kumar GADDIPATI' <naveen.gaddipati@...ricsson.com>,
	Christopher Heiny <Cheiny@...aptics.com>,
	Allie Xiong <axiong@...aptics.com>,
	"j.de.gram@...il.com" <j.de.gram@...il.com>
CC:	"khali@...ux-fr.org" <khali@...ux-fr.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
	Linus WALLEIJ <linus.walleij@...ricsson.com>,
	Pradeepkumar Kumar SUJUAR <pradeep.kumar.sujuar@...ricsson.com>,
	Sundar R IYER <sundar.iyer@...ricsson.com>
Subject: RE: [RFC PATCH 0/1] input/touchscreen: Synaptics Touchscreen Driver

Hi Naveen -

Thanks for the feedback. We didn't know about defconfig usually being sent as a separate patch as this is our first driver submission. Yes - I agree we should move the RMI specific files to a dir under drivers/input. Yeah - we have some older legacy C coding standard that was originally used and it called for using Hungarian notation and it looks like Kernel.org doesn't like that.

Cheers!

Bill Manson

-----Original Message-----
From: Naveen Kumar GADDIPATI [mailto:naveen.gaddipati@...ricsson.com]
Sent: Wednesday, August 25, 2010 2:30 AM
To: Christopher Heiny; Allie Xiong; William Manson; j.de.gram@...il.com
Cc: khali@...ux-fr.org; linux-kernel@...r.kernel.org; linux-input@...r.kernel.org; Linus WALLEIJ; Pradeepkumar Kumar SUJUAR; Sundar R IYER
Subject: Re: [RFC PATCH 0/1] input/touchscreen: Synaptics Touchscreen Driver

Hi Christopher,

Some generic review comments for this patch

1. Normally, defconfig should be a separate patch. You need not
post the entire defconfig, but only the changes to the Makefiles and
the Kconfig files.

2. All RMI specific files can be moved to a separate driver folder
in input/ like drivers/input/rmi/*.

3. Lots of Hungarian notation in the code. Please refer this snippet.

>------------------------------------
>CODE CONTAINS HUNGARIAN NOTATION
>
>Some coding guidlines at some companies require that you type the name
>of your variables using the infamous hungarian notation:
>http://en.wikipedia.org/wiki/Hungarian_notation
>
>We have an internal coding guideline that touches on the
>subject:
>http://swadvice.lud.stericsson.com/guideline.aspx?nr=57&ver=latest
>
>According to the Linux kernel inventor Linus Torvalds in his document
>Documentation/CodingStyle in the Linux kernel:
>
>"Encoding the type of a function into the name (so-called Hungarian
>notation) is brain damaged-the compiler knows the types anyway and can
>check those, and it only confuses the programmer."
>
>Remove all instances of hungariang notation in the code please.
>Includes prefixing pointers with p_*, naming structs with s_*, enums
>with e_*, suffixes like *32b etc etc.
>-----------------------------------
>

We are also using the Synaptics RMI4 touch pad on our U8500 platform.
We had to make some changes to these posted drivers to make it work on
our platform. Also, we modified the touch screen driver to be compliant
to the kernel coding guidelines, but as a stand alone driver though.

I will soon post out an RFC for our patch which we got working on our board.
Please do have a look and we would like you guys to incorporate these changes
into your final patch sets as well, so that we can avoid any re-works later on
when your patch set gets merged into the mainline kernel.

Cheers!
Naveen
--
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/