SUPERH platform development
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH 3/3] ARM: shmobile: r8a7791: add R-Car sound support on DTSI
Date: Tue, 17 Jun 2014 06:09:29 +0000	[thread overview]
Message-ID: <20140617060926.GA18186@verge.net.au> (raw)
In-Reply-To: <87k38menyv.wl%kuninori.morimoto.gx@renesas.com>

On Mon, Jun 16, 2014 at 12:12:18AM -0700, Kuninori Morimoto wrote:
> 
> Hi Simon
> 
> > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> > > 
> > > This patch support PIO transfer only at this point
> > 
> > I have marked this patch as an RFC in patchwork.
> > 
> > As discussed privately I think we should wait until we
> > can enable audio with DMA as my understanding is that
> > audio with PIO is not particularly usable.
> 
> Thank you for your help.
> 
> But, this patch adds "SoC" side sound support,
> not "board/platform" side.
> If you want to use sound, then, both "SoC + board" are needed.
> 
> When I sent r8a7790 sound support, I sent both "SoC" + "board",
> and, you ignored "board (= lager)" patch. This is OK for me.
> 
> This r8a7791 sound support patches, I sent "SoC" patch only,
> I didn't sent "board" side.
> So, koelsch board still cann't use sound :P
> I guess you can apply [3/3] patch ?
> 
> r8a7790 has "SoC" sound support patch on devel branch
> eed8d04e231ea32fa704e4afb5f304ec0e836c4f
> (ARM: shmobile: r8a7790: add R-Car sound support on DTSI)
> 
> Sorry for un-understandable / confusable patches.

Hi Morimoto-san,

sorry for my confusion.

I have re-marked this patch as "Under Review" in patchwork and
I intend to apply it once I have finished attending rebasing
the tree now that v3.16-rc1 has been released.

Please ping me if I have not queued up this patch by the
the end of this week.

  parent reply	other threads:[~2014-06-17  6:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-12  4:44 [PATCH 3/3] ARM: shmobile: r8a7791: add R-Car sound support on DTSI Kuninori Morimoto
2014-06-13  7:49 ` Simon Horman
2014-06-16  7:12 ` Kuninori Morimoto
2014-06-17  6:09 ` Simon Horman [this message]
2014-06-17 10:10 ` Kuninori Morimoto
2014-06-17 12:22 ` Simon Horman

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=20140617060926.GA18186@verge.net.au \
    --to=horms@verge.net.au \
    --cc=linux-sh@vger.kernel.org \
    /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