From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: dmaengine: rcar-dmac: Fix too early/late system suspend/resume callbacks From: Vinod Koul Message-Id: <20180404055420.GE15443@localhost> Date: Wed, 4 Apr 2018 11:24:21 +0530 To: Geert Uytterhoeven Cc: Dan Williams , Ulf Hansson , "Rafael J . Wysocki" , Yoshihiro Shimoda , Kuninori Morimoto , Niklas =?iso-8859-1?Q?S=F6derlund?= , dmaengine@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org List-ID: T24gVGh1LCBNYXIgMjksIDIwMTggYXQgMDY6NTM6MzJQTSArMDIwMCwgR2VlcnQgVXl0dGVyaG9l dmVuIHdyb3RlOgo+IElmIHNlcmlhbCBjb25zb2xlIHdha2UtdXAgaXMgZW5hYmxlZCAoImVjaG8g ZW5hYmxlZCA+Cj4gL3N5cy8uLi4vdHR5U0MwL3Bvd2VyL3dha2V1cCIpLCBhbmQgYW55IHNlcmlh bCBpbnB1dCBpcyByZWNlaXZlZCB3aGlsZQo+IHRoZSBzeXN0ZW0gaXMgc3VzcGVuZGVkLCBzZXJp YWwgcG9ydCBpbnB1dCBubyBsb25nZXIgd29ya3MgYWZ0ZXIgc3lzdGVtCj4gcmVzdW1lLgo+IAo+ IE5vdGUgdGhhdDoKPiAgIDEpIFRoZSBzeXN0ZW0gY2FuIHN0aWxsIGJlIHdva2VuIHVwIHVzaW5n IHRoZSBzZXJpYWwgY29uc29sZSwKPiAgIDIpIFNlcmlhbCBwb3J0IGlucHV0IGtlZXBzIHdvcmtp bmcgaWYgdGhlIHN5c3RlbSBpcyB3b2tlbiB1cCBpbiBzb21lCj4gICAgICBvdGhlciB3YXkgKGUu Zy4gV2FrZS1vbi1MQU4gb3IgZ3Bpby1rZXlzKSwgYW5kIG5vIHNlcmlhbCBpbnB1dCB3YXMKPiAg ICAgIHJlY2VpdmVkIHdoaWxlIHN1c3BlbmRlZC4KPiAKPiBUbyBmaXggdGhpcywgcmVwbGFjZSBT RVRfTEFURV9TWVNURU1fU0xFRVBfUE1fT1BTKCkgYnkKPiBTRVRfTk9JUlFfU1lTVEVNX1NMRUVQ X1BNX09QUygpLCBhcyB0aGUgY2FsbGJhY2tzIGluc3RhbGxlZCBieSB0aGUKPiBmb3JtZXIgaGFw cGVuIHRvbyBlYXJseSByZXNwLiBsYXRlIGluIHRoZSBzdXNwZW5kIHJlc3AuIHJlc3VtZSBwcm9j ZXNzLgoKQXBwbGllZCwgdGhhbmtzCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH] dmaengine: rcar-dmac: Fix too early/late system suspend/resume callbacks Date: Wed, 4 Apr 2018 11:24:21 +0530 Message-ID: <20180404055420.GE15443@localhost> References: <1522342412-10126-1-git-send-email-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1522342412-10126-1-git-send-email-geert+renesas@glider.be> Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Dan Williams , Ulf Hansson , "Rafael J . Wysocki" , Yoshihiro Shimoda , Kuninori Morimoto , Niklas =?iso-8859-1?Q?S=F6derlund?= , dmaengine@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-pm@vger.kernel.org On Thu, Mar 29, 2018 at 06:53:32PM +0200, Geert Uytterhoeven wrote: > If serial console wake-up is enabled ("echo enabled > > /sys/.../ttySC0/power/wakeup"), and any serial input is received while > the system is suspended, serial port input no longer works after system > resume. > > Note that: > 1) The system can still be woken up using the serial console, > 2) Serial port input keeps working if the system is woken up in some > other way (e.g. Wake-on-LAN or gpio-keys), and no serial input was > received while suspended. > > To fix this, replace SET_LATE_SYSTEM_SLEEP_PM_OPS() by > SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(), as the callbacks installed by the > former happen too early resp. late in the suspend resp. resume process. Applied, thanks -- ~Vinod