Re: pls backport two thinkpad-acpi patches for X1 carbon 2nd generation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Apr 13, 2014 at 09:41:08PM +0800, Shuduo Sang wrote:
> On Sat, Apr 12, 2014 at 11:05 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Sat, Apr 12, 2014 at 03:26:41PM +0800, Shuduo Sang wrote:
> >> On Fri, Apr 11, 2014 at 11:44 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> >> > On Fri, Apr 11, 2014 at 11:33:41PM +0800, Shuduo Sang wrote:
> >> >> Hi stable tree maintainer,
> >> >>
> >> >> Pls backport two patches for Thinkpad X1 carbon 2nd generation's fancy
> >> >> adaptive keyboard:
> >> >>
> >> >> 330947b84382479459e5296a0024c670367b0b57 save and restore adaptive
> >> >> keyboard mode for suspend and resume
> >> >>
> >> >> 3a9d20bda1d6daae9d81a4cc4cc67238c5574d31 support Thinkpad X1 Carbon
> >> >> 2nd generation's adaptive keyboard
> >> >
> >> > That's a really "big" patch for a stable kernel, do you think it matches
> >> > the Documentation/stable_kernel_rules.txt rules?
> >> >
> >>
> >> Above patches are well tested on Ubuntu 14.04 Beta with 3.13 kernel and Debian
> >> with Linus tree.
> >>
> >> Sorry 3a9d20bda1d6daae9d81a4cc4cc67238c5574d31 is about 150 lines.
> >> But it's fundamental to initially support adaptive keyboard. It's hard to split.
> >
> > It looks like this is a "support new hardware" request, which doesn't
> > happen in stable kernel releases, unless it is just a "simple" quirk or
> > device id addition.  At 150 lines, that does not meet these rules,
> > sorry.
> 
> Sorry X1 carbon's adaptive keyboard has so rich functions that need
> 150 lines code
> to support. It can't be simple quirk or only new id. Is it possible to
> be accepted if the
> patch be split to two patches less 150 lines?

So, you want to game the system?  No, we don't add new hardware support
to old kernels, that's not what the stable kernel releases are about.

> If stable tree has no these patches, some linux users who purchased X1 carbon
> need hack kernel by themselves to make the fancy adaptive keyboard
> working. I know
> many X1 carbon's users are suffering on it.

Then just use a new kernel version.  Or get your distro to backport the
patch.  Or even better, run your own kernel with the patch in it.

What makes this keyboard so special from the thousands of other new
devices we support with every new kernel?  The rules about what we
accept into the stable kernels are there for a reason.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]