Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Greg Kroah-Hartman
On Mon, Jul 31, 2017 at 07:00:32AM +, Jamet, Michael wrote: > This e-mail and any attachments may contain confidential material for > the sole use of the intended recipient(s). Any review or distribution > by others is strictly prohibited. If you are not the intended > recipient, please

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Greg Kroah-Hartman
On Mon, Jul 31, 2017 at 07:00:32AM +, Jamet, Michael wrote: > This e-mail and any attachments may contain confidential material for > the sole use of the intended recipient(s). Any review or distribution > by others is strictly prohibited. If you are not the intended > recipient, please

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Andy Shevchenko
> - > Intel Israel (74) Limited > > This e-mail and any attachments may contain confidential material for > the sole use of the intended recipient(s). Any review or distribution > by others is strictly prohibited. If you are not

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Andy Shevchenko
> - > Intel Israel (74) Limited > > This e-mail and any attachments may contain confidential material for > the sole use of the intended recipient(s). Any review or distribution > by others is strictly prohibited. If you are not

RE: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Jamet, Michael
ever <andreas.noe...@gmail.com>; > Jamet, Michael <michael.ja...@intel.com>; Greg Kroah-Hartman > <gre...@linuxfoundation.org> > Subject: Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in > icm_suspend() > > > > > Acked-by: Mika Westerberg <mika

RE: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-31 Thread Jamet, Michael
> -Original Message- > From: Rafael J. Wysocki [mailto:r...@rjwysocki.net] > Sent: Wednesday, July 26, 2017 20:05 > To: Mika Westerberg > Cc: LKML ; Linux PM p...@vger.kernel.org>; Andreas Noever ; > Jamet, Michael ; Greg Kroah-Hartman > > Subject: Re: [R

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-27 Thread Rafael J. Wysocki
On Thursday, July 27, 2017 11:50:11 AM Mika Westerberg wrote: > On Wed, Jul 26, 2017 at 07:05:01PM +0200, Rafael J. Wysocki wrote: > > On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > > > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > > > On Wednesday, July

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-27 Thread Rafael J. Wysocki
On Thursday, July 27, 2017 11:50:11 AM Mika Westerberg wrote: > On Wed, Jul 26, 2017 at 07:05:01PM +0200, Rafael J. Wysocki wrote: > > On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > > > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > > > On Wednesday, July

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-27 Thread Mika Westerberg
On Wed, Jul 26, 2017 at 07:05:01PM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > > > On Tue, Jul 25,

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-27 Thread Mika Westerberg
On Wed, Jul 26, 2017 at 07:05:01PM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > > > On Tue, Jul 25,

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Rafael J. Wysocki
On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > > > On Tuesday, July

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Rafael J. Wysocki
On Wednesday, July 26, 2017 04:12:59 PM Mika Westerberg wrote: > On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > > > On Tuesday, July

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Mika Westerberg
On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > > > On Tue, Jul 25, 2017

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Mika Westerberg
On Wed, Jul 26, 2017 at 02:48:54PM +0200, Rafael J. Wysocki wrote: > On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > > > On Tue, Jul 25, 2017

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Rafael J. Wysocki
On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > > > From: Rafael J.

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Rafael J. Wysocki
On Wednesday, July 26, 2017 11:32:44 AM Mika Westerberg wrote: > On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > > > From: Rafael J.

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Mika Westerberg
On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > > > On one of my test

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-26 Thread Mika Westerberg
On Tue, Jul 25, 2017 at 06:10:57PM +0200, Rafael J. Wysocki wrote: > On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > > > On one of my test machines nhi_mailbox_cmd() called

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-25 Thread Rafael J. Wysocki
On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > On one of my test machines nhi_mailbox_cmd() called from icm_suspend() > > times out and returnes an

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-25 Thread Rafael J. Wysocki
On Tuesday, July 25, 2017 01:00:12 PM Mika Westerberg wrote: > On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > On one of my test machines nhi_mailbox_cmd() called from icm_suspend() > > times out and returnes an error which then is

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-25 Thread Mika Westerberg
On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > On one of my test machines nhi_mailbox_cmd() called from icm_suspend() > times out and returnes an error which then is propagated to the > caller and causes the entire

Re: [RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-25 Thread Mika Westerberg
On Tue, Jul 25, 2017 at 01:31:00AM +0200, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > On one of my test machines nhi_mailbox_cmd() called from icm_suspend() > times out and returnes an error which then is propagated to the > caller and causes the entire system suspend to be aborted

[RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-24 Thread Rafael J. Wysocki
From: Rafael J. Wysocki On one of my test machines nhi_mailbox_cmd() called from icm_suspend() times out and returnes an error which then is propagated to the caller and causes the entire system suspend to be aborted which isn't very useful. Instead of aborting

[RFC][PATCH] thunderbolt: icm: Ignore mailbox errors in icm_suspend()

2017-07-24 Thread Rafael J. Wysocki
From: Rafael J. Wysocki On one of my test machines nhi_mailbox_cmd() called from icm_suspend() times out and returnes an error which then is propagated to the caller and causes the entire system suspend to be aborted which isn't very useful. Instead of aborting system suspend, print the error