[Precise][PATCH] (pre-stable) HID: bump maximum global item tag report size to 96 bytes

Chase Douglas chase.douglas at canonical.com
Mon Nov 21 23:45:51 UTC 2011

On 11/21/2011 07:11 AM, Chase Douglas wrote:
> On 11/21/2011 05:53 AM, Tim Gardner wrote:
>> On 11/17/2011 10:02 AM, Chase Douglas wrote:
>>> On 11/17/2011 03:58 AM, Tim Gardner wrote:
>>>> On 11/16/2011 01:37 PM, Chase Douglas wrote:
>>>>> This allows the latest N-Trig devices to function properly.
>>>>> BugLink: https://bugs.launchpad.net/bugs/724831
>>>>> Cc: stable at vger.kernel.org
>>>>> Signed-off-by: Chase Douglas<chase.douglas at canonical.com>
>>>>> Signed-off-by: Jiri Kosina<jkosina at suse.cz>
>>>>> ---
>>>>> This has been accepted into Jiri Kosina's git tree for 3.4. It will eventually
>>>>> come down through the stable queue in three or four months, but it should be
>>>>> applied to Precise in the meantime.
>>>> Surely you mean the 3.3 merge window ? Why won't Jiri accept this for
>>>> 3.2 ? There are still a number of rc candidates left, and this looks
>>>> like a minor bug fix patch suitable for an rc.
>>> I agree and am pestering him about it. I may be misinterpreting his
>>> tree. He appears to be using a different branch scheme than in the past.
>>> -- Chase
>> Any progress ?
> Still pestering. RTT is approximately 4 days... I can confirm that his
> "upstream" branch is stuff that will go into the next merge window, and
> his "upstream-fixes" branch is stuff that will go into the current rc
> releases. This patch is only in his "upstream" branch right now. I'm
> asking him why it shouldn't go into 3.3, and I'll update you when I find
> out.

I got word back from Jiri today. He doesn't want to push the change
because it has the potential to cause regressions in the HID layer.
There's nothing about this particular patch that he has highlighted as
an issue, but it is not a pure "fix" in the sense that it allows more
devices to pass the HID parser checks.

I can see his point, and if I were in his shoes I might make the same
decision. However, given that we are a community distribution with a
focus on consumer use cases, this patch allows a multitouch touchscreen
product to function without any changes other than device ID addition,
and this patch is merely a softening of a HID parsing check, I feel it
is reasonable to apply it as an SRU.

Either way, I think the patch should be applied to Precise since it will
come down from stable in a few months anyways.

-- Chase

More information about the kernel-team mailing list