public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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

      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