Date: Mon, 25 Dec 2017 09:47:48 +0100
From: Markus Trippelsdorf <markus@trippelsdorf.de>
To: Pavel Machek <pavel@ucw.cz>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>, gregkh@linuxfoundation.org,
        kyle@kernel.org, akpm@linux-foundation.org, mmarek@suse.com,
        yamada.masahiro@socionext.com, dwmw2@infradead.org,
        torvalds@linux-foundation.org,
        kernel list <linux-kernel@vger.kernel.org>, len.brown@intel.com,
        linux-pm@vger.kernel.org
Subject: Re: [regression] Re: Thinkpad T40p: suspend to ram stopped working
 sometime before 4.14
Message-ID: <20171225084748.GA246@x4>
References: <20171224114921.GA9891@amd>
 <58258946.1WsWlzHmoT@aspire.rjw.lan>
 <20171224152232.GA30077@amd>
 <20171224153624.GA22377@amd>
 <20171224181740.GA26326@amd>
 <20171224223739.GA24747@amd>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20171224223739.GA24747@amd>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1306
Lines: 41

On 2017.12.24 at 23:37 +0100, Pavel Machek wrote:
> Hi!
> 
> > > > > > 4.15-rcX is broken, but that had other problems so lets not go there.
> > > > > > 
> > > > > > 4.14 is broken.
> > > > > 
> > > > > And what exactly does happen?
> > > > 
> > > > Suspend looks ok. I believe there's some reaction to power button, but
> > > > screen stays blank. I'll re-check that.. I'm in the middle of bisect
> > > > now.
> > > 
> > > Ok, so suspend looks ok. On resume, "moon" LED goes off and power /
> > > bluetooth LEDs go on, but screen is black and no response to
> > > > > > anything.
> > 
> > And it seems bad commit is
> > 
> > commit 5620a0d1aacd554ebebcff373e31107bb1ef7769
> > Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > Date:   Thu Sep 14 14:23:01 2017 -0700
> > 
> >     firmware: delete in-kernel firmware
> > 
> > Unfortuantely, it does not revert cleanly.
> 
> I can't, because there's df85b2d767aad90fd2746f993fcd66dd322768f8 on
> top of it, partialy reverting it. Seems like 5620 was causing problems
> for other people before.
> 
> And then there's huge mega-patch called "license cleanup" on top of
> that, which changes 11100 files (!) and makes work harder still.
> 
> Can we go for full revert?

No, it does not make any sense. 
Could you please post your config? 

-- 
Markus