linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] ARM: SPEAr13xx support for 3.5
Date: Tue, 1 May 2012 08:34:14 +0000	[thread overview]
Message-ID: <201205010834.14724.arnd@arndb.de> (raw)
In-Reply-To: <4F9F5B3F.1030502@st.com>

On Tuesday 01 May 2012, Viresh Kumar wrote:
> 
> On 4/30/2012 11:16 PM, Arnd Bergmann wrote:
> > Ok, I missed the compilation dependencies you mentioned here.  So the branch
> > you sent me doesn't actually build by itself, which is not good.
> 
> I thought, i must not add patches from Russell and Mike in my pull request, as they
> haven't made to linux-next till now. i.e. even if this branch doesn't compile by
> itself, it should get compiled when it is merged with your branch, where these
> dependencies must be present already. Isn't this the right approach?

It's more complicated than this. You are right that you cannot just add patches
to one tree that should go through a different tree. However, another requirement
is that all branches are bisectable and you have a working kernel at each
intermediate step. It takes some experience to get it all working.

> > We can work out something from here once I understand where the dependencies
> > are.
> 
> These two only:
> 
> - Mike's clk-next branch

The right solution for this would have been to take it as another base merged
with the other two branches and put the new patches on top, and then let me
pull in the clk-next branch as a dependency into linux-next that needs to
get upstream first before I send out your patches.

This is not as important for a new platform like spear13xx because there won't
be any regressions, but you should absolutely make sure that you don't break
existing platforms. I also noticed now that the pinctrl patches in the spear3xx
series don't work by themselves and have another dependency on the stuff that
I presume is in Linus' pinctrl tree.

> - Russell's: CLKDEV: Add helper routines to allocate and add clkdevs for ... 

I could not find out the status of that patch. Which branch is it going to get
merged through?

	Arnd

  reply	other threads:[~2012-05-01  8:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-30 16:10 [GIT PULL] ARM: SPEAr13xx support for 3.5 viresh kumar
2012-04-30 17:46 ` Arnd Bergmann
2012-05-01  3:40   ` Viresh Kumar
2012-05-01  8:34     ` Arnd Bergmann [this message]
2012-05-01  8:38       ` Viresh Kumar
2012-05-01  8:38       ` Russell King - ARM Linux
2012-05-01  8:42         ` Russell King - ARM Linux
2012-05-01  8:54           ` Arnd Bergmann

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=201205010834.14724.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.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).