linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH] ARM: shmobile: r8a7790: add audio clocks
Date: Fri, 10 Jan 2014 10:47:09 +0000	[thread overview]
Message-ID: <2087710.fPFk1GzNhU@avalon> (raw)
In-Reply-To: <8738l1jzfv.wl%kuninori.morimoto.gx@gmail.com>

Hi Morimoto-san,

On Thursday 09 January 2014 16:51:52 Kuninori Morimoto wrote:
> Hi Laurent
> 
> Thank you for your help
> 
> > 1. Add the old style clocks to Lager
> > 2. Add the new style clocks to BockW
> > 3. Add the new style clocks to Lager
> > 4. Modify the ADG driver to use the new style clocks
> > 5. Remove the old style clocks from BockW
> > 6. Remove the old style clocks from Lager
> 
> Yes it become clean.
> But hmm,
> I guess #5, #6 (#4 too ?) are happen after merge anyway ?
> If so, this is more clean for me.
> 
> 1. Add the old style clocks to Lager
> 2. Modify the ADG driver to use the new style clocks.
>    BockW/Lager clocls are fixed in same patch.

My point is that you need to add the new style clocks to platform code before 
you modify the ADG driver. If adding old style clocks will get merged during 
the same merge window as the new style clocks and the ADG driver changes, I 
don't think there's a point to add them.

> Here, #2 will be done on ALSA branch side after merge,
> and I guess Simon can use it for topic branch for us ?
> 
> # This kind of merge timing something
> # is headache for us :P

-- 
Regards,

Laurent Pinchart


  parent reply	other threads:[~2014-01-10 10:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-06  8:18 [PATCH] ARM: shmobile: r8a7790: add audio clocks Kuninori Morimoto
2014-01-07  9:34 ` Laurent Pinchart
2014-01-08  0:22 ` Kuninori Morimoto
2014-01-08  0:40 ` Laurent Pinchart
2014-01-08  2:07 ` Kuninori Morimoto
2014-01-08  2:39 ` Kuninori Morimoto
2014-01-08  8:04 ` Laurent Pinchart
2014-01-08  9:11 ` Kuninori Morimoto
2014-01-09 18:32 ` Laurent Pinchart
2014-01-10  0:19 ` Kuninori Morimoto
2014-01-10  0:24 ` Laurent Pinchart
2014-01-10  0:51 ` Kuninori Morimoto
2014-01-10  1:08 ` Simon Horman
2014-01-10 10:47 ` Laurent Pinchart [this message]
2014-01-14  0:15 ` Kuninori Morimoto

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=2087710.fPFk1GzNhU@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --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;
as well as URLs for NNTP newsgroup(s).