From: Greg KH <greg@kroah.com>
To: Ike Panhc <ike.pan@canonical.com>
Cc: linux-kernel@vger.kernel.org,
platform-driver-x86@vger.kernel.org, stable@kernel.org,
Matthew Garrett <mjg@redhat.com>
Subject: Re: [stable] [PATCH] [stable-37] ideapad: read brightness setting when have brightness key notify
Date: Mon, 7 Mar 2011 21:07:59 -0800 [thread overview]
Message-ID: <20110308050759.GA12184@kroah.com> (raw)
In-Reply-To: <4D759675.8020600@canonical.com>
On Tue, Mar 08, 2011 at 10:37:41AM +0800, Ike Panhc wrote:
> On 03/08/2011 12:18 AM, Greg KH wrote:
> > On Mon, Mar 07, 2011 at 06:35:25PM +0800, Ike Panhc wrote:
> >> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=25922
> >>
> >> On ideapad Y530, the brightness key notify will be blocked if the last notify
> >> is not responsed by getting the brightness value. Read value when we get the
> >> notify shall fix the problem and will not have any difference on other ideapads.
> >>
> >> This patch is made against 2.6.37.y and this bug impacts kernel after 2.6.37.
> >>
> >> The equivalent-fix patch is not in upstream yet. Since the policy of stable tree
> >> is changed and .38 kernel is going to finalize.
> >
> > What "policy of the stable tree is changed"? Huh?
>
> I thought we will have stable release on lastest kernel only[1]. Since the .38 is
> going to release soon. It seems I do not have much time for .37 stable release.
>
> I hope I do not miss something.
>
> [1] https://lkml.org/lkml/2010/12/2/388
I think you miss the fact that if by the time your patch makes it into
Linus's tree, and we have moved on to not supporting .37 anymore, then
no one will care about .37 so it doesn't matter at all.
> >> I think to post this patch for stable kernel now should be a good
> >> idea.
> >
> > Have you not read the Documentation/stable_kernel_rules.txt and seen
> > that this is totally and completly not allowed? Why is it suddenly not
> > true?
> >
> > confused,
>
> Sorry about the confusing. The same patch for upstream is posted[2] but not merged
> yet. It looks like a now-or-never situation to me. That's why I post and write the
> concerns in comment.
You need to follow the documented and well explained rules, they are
there for a very good reason, please don't try to subvert them,
otherwise you will just end up in my kill file.
thanks,
greg k-h
prev parent reply other threads:[~2011-03-08 5:09 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-07 10:35 [PATCH] [stable-37] ideapad: read brightness setting when have brightness key notify Ike Panhc
2011-03-07 16:18 ` [stable] " Greg KH
2011-03-08 2:37 ` Ike Panhc
2011-03-08 5:07 ` Greg KH [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110308050759.GA12184@kroah.com \
--to=greg@kroah.com \
--cc=ike.pan@canonical.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg@redhat.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=stable@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox