From: Daniel Mack <zonque@gmail.com>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: linux-usb@vger.kernel.org, linux-omap@vger.kernel.org,
neumann@teufel.de, vinod.koul@intel.com,
dan.j.williams@intel.com, balbi@ti.com
Subject: Re: [PATCH 0/3] dma: cppi41: more suspend/resume patches
Date: Wed, 09 Oct 2013 09:23:38 +0200 [thread overview]
Message-ID: <5255047A.5010609@gmail.com> (raw)
In-Reply-To: <20131009064124.GA3529@linutronix.de>
On 09.10.2013 08:41, Sebastian Andrzej Siewior wrote:
> * Daniel Mack | 2013-10-01 15:31:08 [+0200]:
>
>> Patch #1 restores more registers on resume time.
>>
>> Patch #2 is a cosmetic cleanup that emerged while digging through the
>> driver and gaining a basic idea of how it's implemented. Nothing fancy.
>
> I'm fine with those two.
>
>>
>> Patch #3, however, gives me headaches. I can't fully explain what's
>> going on, but I can tell for sure that if fixes a problem that I stared
>> on for many hours.
>
> I'm still trying to verify if it breaks something or not. So I haven't
> forgotten about this.
Ok, thank you very much for the update :) I can of course test
alternative patches if you have any.
Could you actually reproduce the issue I described by sending your board
to suspend?
Daniel
next prev parent reply other threads:[~2013-10-09 7:23 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-01 13:31 [PATCH 0/3] dma: cppi41: more suspend/resume patches Daniel Mack
[not found] ` <1380634271-27588-1-git-send-email-zonque-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-10-01 13:31 ` [PATCH 1/3] dma: cppi41: restore more registers Daniel Mack
2013-10-01 13:31 ` [PATCH 2/3] dma: cppi41: use cppi41_pop_desc() where possible Daniel Mack
2013-10-01 13:31 ` [PATCH 3/3] dma: cppi41: move -EAGAIN in tear_down Daniel Mack
[not found] ` <1380634271-27588-4-git-send-email-zonque-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-10-02 8:29 ` Sebastian Andrzej Siewior
2013-10-02 9:19 ` Daniel Mack
2013-10-02 10:25 ` Sebastian Andrzej Siewior
2013-10-02 11:38 ` Daniel Mack
2013-10-02 10:20 ` [PATCH 0/3] dma: cppi41: more suspend/resume patches Sebastian Andrzej Siewior
[not found] ` <20131002102033.GB16680-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>
2013-10-02 11:07 ` Daniel Mack
2013-10-02 12:57 ` Sebastian Andrzej Siewior
2013-10-09 6:41 ` Sebastian Andrzej Siewior
2013-10-09 7:23 ` Daniel Mack [this message]
[not found] ` <5255047A.5010609-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-10-09 7:28 ` Sebastian Andrzej Siewior
2013-10-09 7:31 ` Daniel Mack
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=5255047A.5010609@gmail.com \
--to=zonque@gmail.com \
--cc=balbi@ti.com \
--cc=bigeasy@linutronix.de \
--cc=dan.j.williams@intel.com \
--cc=linux-omap@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=neumann@teufel.de \
--cc=vinod.koul@intel.com \
/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;
as well as URLs for NNTP newsgroup(s).