[MAVERICK SAUCE] UBUNTU: SAUCE: Add support for Intellimouse Mode in ALPS touchpad on Dell E2 series Laptops
manoj.iyer at canonical.com
manoj.iyer at canonical.com
Thu Sep 9 15:08:33 UTC 2010
I ran checkpatch.pl and it generated 3 warning, I am ignoring the "line
over 80 characters warning"...
WARNING: line over 80 characters
#106: FILE: drivers/input/mouse/alps.c:179:
+ rc =
alps_model_init_quirk_tbl[i].callback(psmouse);
WARNING: suspect code indent for conditional statements (24, 8)
#155: FILE: drivers/input/mouse/psmouse-base.c:661:
if (!set_properties || alps_init(psmouse) == 0)
+ /* If ALPS model quirk was applied, don't change the settings */
WARNING: line over 80 characters
#158: FILE: drivers/input/mouse/psmouse-base.c:663:
+ return psmouse->type ? psmouse->type :
PSMOUSE_ALPS;
total: 0 errors, 3 warnings, 119 lines checked
./0001-UBUNTU-SAUCE-Add-support-for-Intellimouse-Mode-in-AL.patch has
style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
I fixed warning #158, and pushed to remote repo. Modified patch attached.
The following changes since commit
4e90adae43f52db35c99bcaccb2e2a912b8299d5:
stanley.miao (1):
DSS2: Don't power off a panel twice
are available in the git repository at:
git://kernel.ubuntu.com/git/manjo/ubuntu-maverick.git lp632884
Manoj Iyer (1):
UBUNTU: SAUCE: Add support for Intellimouse Mode in ALPS touchpad on
Dell E2 series Laptops
drivers/input/mouse/alps.c | 81
++++++++++++++++++++++++++++++++++++
drivers/input/mouse/alps.h | 5 ++
drivers/input/mouse/psmouse-base.c | 3 +-
3 files changed, 88 insertions(+), 1 deletions(-)
Cheers
--- manjo
On Thu, 9 Sep 2010, Tim Gardner wrote:
> On 09/08/2010 07:09 PM, Manoj Iyer wrote:
>> git://kernel.ubuntu.com/git/manjo/ubuntu-maverick.git lp632884
>>
>
> Other then some minor formatting the patch looks self contained. Can you run
> it through checkpatch.pl first? Then perhaps send it back to Rez with the
> formatting corrections? Otherwise,
>
> Acked-by: Tim Gardner <tim.gardner at canonical.com>
>
> --
> Tim Gardner tim.gardner at canonical.com
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-UBUNTU-SAUCE-Add-support-for-Intellimouse-Mode-in-AL.patch
Type: text/x-diff
Size: 5798 bytes
Desc: patch attached
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20100909/4f6e2284/attachment.patch>
More information about the kernel-team
mailing list