linux-kernel - Re: [PATCH] drivers: Adding driver support for "One by Wacom CTL472/672" (parameters transferred from CTL471/671 since only difference is the force resolution)

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] [day] [month] [year] [list]

Date:   Wed, 11 Oct 2017 08:11:20 -0700
From:   Jason Gerecke <killertofu@...il.com>
To:     Jiri Kosina <jikos@...nel.org>, "? ??" <jingmingxuan@...look.com>
Cc:     "benjamin.tissoires@...hat.com" <benjamin.tissoires@...hat.com>,
        "linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Ping Cheng <ping.cheng@...om.com>,
        Jason Gerecke <jason.gerecke@...om.com>
Subject: Re: [PATCH] drivers: Adding driver support for "One by Wacom
 CTL472/672" (parameters transferred from CTL471/671 since only difference is
 the force resolution)

On Wed, Oct 11, 2017 at 6:24 AM, Jiri Kosina <jikos@...nel.org> wrote:
> On Sun, 8 Oct 2017, ? ?? wrote:
>
>> Signed-off-by: Mx Jing <jingmingxuan@...look.com>
>
> Changelog is missing, please provide one.
>
> Adding Jason and Ping to CC.
>

A few inline nits:

>> ---
>>  drivers/hid/wacom_wac.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/hid/wacom_wac.c b/drivers/hid/wacom_wac.c
>> index aa692e28b2cd..920f3d6bbc71 100644
>> --- a/drivers/hid/wacom_wac.c
>> +++ b/drivers/hid/wacom_wac.c
>> @@ -4376,6 +4376,12 @@ static const struct wacom_features wacom_features_0x360 =
>>  static const struct wacom_features wacom_features_0x361 =
>>       { "Wacom Intuos Pro L", 62200, 43200, 8191, 63,
>>         INTUOSP2_BT, WACOM_INTUOS3_RES, WACOM_INTUOS3_RES, 9, .touch_max = 10 };
>> +static const struct wacom_features wacom_features_0x37A =
>> +     { "One by Wacom CTL472", 14720, 9225, 2047, 63,

Replace "CTL472" in name with "S". Resolution should be 15200 x 9500,
not 14720 x 9225.

>> +       BAMBOO_PEN, WACOM_INTUOS_RES, WACOM_INTUOS_RES };
>> +static const struct wacom_features wacom_features_0x37B =
>> +     { "One by Wacom CTL672", 21648, 13530, 2047, 63,

Replace "CTL672" in name with "M". Resolution should be 21600 x 13500,
not 21648 x 13530.

>> +       BAMBOO_PEN, WACOM_INTUOS_RES, WACOM_INTUOS_RES };
>>

Nits aside, this seems reasonable:

Acked-by: Jason Gerecke <jason.gerecke@...om.com>

Jason
---
Now instead of four in the eights place /
you’ve got three, ‘Cause you added one  /
(That is to say, eight) to the two,     /
But you can’t take seven from three,    /
So you look at the sixty-fours....

>>  static const struct wacom_features wacom_features_HID_ANY_ID =
>>       { "Wacom HID", .type = HID_GENERIC, .oVid = HID_ANY_ID, .oPid = HID_ANY_ID };
>> @@ -4544,6 +4550,8 @@ const struct hid_device_id wacom_ids[] = {
>>       { USB_DEVICE_WACOM(0x343) },
>>       { BT_DEVICE_WACOM(0x360) },
>>       { BT_DEVICE_WACOM(0x361) },
>> +     { USB_DEVICE_WACOM(0x37A) },
>> +     { USB_DEVICE_WACOM(0x37B) },
>>       { USB_DEVICE_WACOM(0x4001) },
>>       { USB_DEVICE_WACOM(0x4004) },
>>       { USB_DEVICE_WACOM(0x5000) },
>> --
>> 2.11.0
>>
>
> --
> Jiri Kosina
> SUSE Labs
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html