public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Porcel <nicolasporcel06@gmail.com>
To: Kieran Clancy <clancy.kieran@gmail.com>
Cc: "D. G. Jansen" <d.g.jansen@gmail.com>,
	Stefan Biereigel <stefan@biereigel.de>,
	Juan Manuel Cabo <juanmanuel.cabo@gmail.com>,
	Stefan Biereigel <security@biereigel-wb.de>,
	Lan Tianyu <lantianyu1986@gmail.com>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"linux-kernel@vger kernel org" <linux-kernel@vger.kernel.org>,
	Len Brown <lenb@kernel.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	San Zamoyski <san@plusnet.pl>, "D. Jansen" <dennis.jansen@web.de>,
	Maurizio D'Addona <mauritiusdadd@gmail.com>
Subject: Re: [REGRESSION 3.14-rc6] Samsung N150 lid does not "open" after suspend to RAM.
Date: Tue, 01 Apr 2014 20:18:43 +0800	[thread overview]
Message-ID: <533AAEA3.6070703@gmail.com> (raw)
In-Reply-To: <CAGf84QJyzDTjd3=XJBDANJgwh4mT5yxPLoCrsMRuq5ZA8XJkJg@mail.gmail.com>

Sure, I understand that you need to test it, that's why I am asking. I 
don't want an untested patch to be included in the kernel either. We 
also have to make sure that the patch fix the regression for all the 
affected Samsung laptops.

For the EC_SC code, I am using the first version of the patch without 
the pr_info line, so I need to recompile the kernel. I'll let you know 
as soon as it's done.

Best regards,

Nicolas Porcel

On 01/04/2014 19:58, Kieran Clancy wrote:
> On Tue, Apr 1, 2014 at 10:06 PM, Nicolas Porcel
> <nicolasporcel06@gmail.com> wrote:
>> Do you have any idea of how much time will it take to merge it to the
>> mainline ?
> Apologies for the breakage. The key here is that we want a patch which
> works 100% on all Samsung systems. If we rush a fix for this
> regression, we may risk breaking things on other systems.
>
> Fortunately, the work-around in the case of this regression is very
> simple (just repeat whatever event was ignored the first time, be it
> lid open, ac plug, etc) compared to the bug which the original patch
> fixed (which had no real workaround and caused near-permanent
> malfunction).
>
> This is why I am asking people with Samsung systems to test the patch
> (with the debug output I added in the kernel bugzilla attachment) and
> confirm that it is working. I am also asking them to report EC_SC data
> because we may be able to create a more targeted patch with this
> information.
>
> Speaking of which, can you please let me know the EC_SC bytes you see
> when you test it on your system, and if it is always the same?
>
> Once I hear back from users on a range of Samsung systems, I will
> submit a patch. It should have enough time to land before 3.15. I will
> also ask for it to be included in other stable branches.
>
> Kind regards,
> Kieran

  reply	other threads:[~2014-04-01 12:18 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-24  7:50 [REGRESSION 3.14-rc6] Samsung N150 lid does not "open" after suspend to RAM Stefan Biereigel
2014-03-24  9:30 ` Lan Tianyu
2014-03-24 11:19   ` Stefan Biereigel
2014-03-25  9:34     ` Lan Tianyu
2014-03-25  9:43       ` Stefan Biereigel
2014-03-25 13:23       ` Kieran Clancy
2014-03-25 13:53         ` Juan Manuel Cabo
2014-03-25 16:07           ` Stefan Biereigel
2014-03-25 16:38         ` Stefan Biereigel
2014-03-25 20:24         ` Stefan Biereigel
2014-03-25 20:41           ` Juan Manuel Cabo
2014-03-25 22:56             ` Juan Manuel Cabo
2014-03-26 10:42               ` Stefan Biereigel
2014-03-26 14:38                 ` Kieran Clancy
2014-03-26 15:01                   ` Rafael J. Wysocki
2014-03-26 19:56                   ` Stefan Biereigel
2014-03-26 22:36                     ` Kieran Clancy
2014-03-26 22:41                       ` Stefan Biereigel
     [not found]                   ` <CAM6oVw2v9ptr0c08uPiB_3z3e41VO+Vp3OhoHXiBxagAOfPBZA@mail.gmail.com>
2014-04-01  9:53                     ` Kieran Clancy
2014-04-01 11:36                       ` Nicolas Porcel
2014-04-01 11:58                         ` Kieran Clancy
2014-04-01 12:18                           ` Nicolas Porcel [this message]
2014-04-01 18:02                             ` Nicolas Porcel

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=533AAEA3.6070703@gmail.com \
    --to=nicolasporcel06@gmail.com \
    --cc=clancy.kieran@gmail.com \
    --cc=d.g.jansen@gmail.com \
    --cc=dennis.jansen@web.de \
    --cc=juanmanuel.cabo@gmail.com \
    --cc=lantianyu1986@gmail.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mauritiusdadd@gmail.com \
    --cc=rjw@rjwysocki.net \
    --cc=san@plusnet.pl \
    --cc=security@biereigel-wb.de \
    --cc=stefan@biereigel.de \
    /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