From: Tony Lindgren <tony@atomide.com>
To: Peter Robinson <pbrobinson@gmail.com>
Cc: linux-omap@vger.kernel.org, arnd@arndb.de, robh+dt@kernel.org
Subject: Re: [PATCH] ARM: OMAP dts: Only build dtb if associated SoC is built
Date: Mon, 12 May 2014 09:03:04 -0700 [thread overview]
Message-ID: <20140512160303.GI31772@atomide.com> (raw)
In-Reply-To: <CALeDE9PjS=zPzsUHiHLJc4Q9hYm_-49+1bRjqRO_a-r6_1LLQw@mail.gmail.com>
* Peter Robinson <pbrobinson@gmail.com> [140509 14:28]:
> On Tue, May 6, 2014 at 6:18 PM, Tony Lindgren <tony@atomide.com> wrote:
> > * Peter Robinson <pbrobinson@gmail.com> [140503 17:12]:
> >> With ARCH_OMAP2PLUS being separated out into OMAP2/3/4/5 etc all the TI device
> >> tree blobs are built no matter the combination of SoCs that are enabled. This
> >> often causes a bunch of irrelevant .dts to be built on a multi platform kernel,
> >> this enables the building of just the ones relevant to the SoCs that are
> >> actually enabled. It also orders the dts file alphabetically.
> >
> > Sure, makes sense and grouping + sorting also helps to avoid trivial
> > merge conflicts.
> >
> > I've changed the ordering a bit as am335x is based on omap3, and
> > omap43x based on omap4. And 3517 is an omap3 with the faster
> > Ethernet and few other changes.
>
> Makes sense.
>
> > I'll apply the following patch into omap-for-v3.16/dt. BTW, looks
> > like you missed a Signed-off-by in your original patch.
>
> Apologies, I was sure I added it but then I crafted the patch a couple
> of times as it's my first first upstream kernel patch so I must have
> missed it for the real one. Can I just ack it now with the below? If
> so...
No problem :) Would add it thanks..
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
..but I had to apply it already already as I wanted to have it
applied before other patches adding things to the Makefile. Having
just my Signed-off-by should be just fine in this case as I had to
edit the patch anyways, and it's just sorting the Makefile and not
adding any new code.
Regards,
Tony
prev parent reply other threads:[~2014-05-12 16:03 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-04 0:11 [PATCH] ARM: OMAP dts: Only build dtb if associated SoC is built Peter Robinson
2014-05-06 17:18 ` Tony Lindgren
2014-05-09 21:28 ` Peter Robinson
2014-05-12 16:03 ` Tony Lindgren [this message]
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=20140512160303.GI31772@atomide.com \
--to=tony@atomide.com \
--cc=arnd@arndb.de \
--cc=linux-omap@vger.kernel.org \
--cc=pbrobinson@gmail.com \
--cc=robh+dt@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.