Thread (9 messages) 9 messages, 5 authors, 2012-01-18

RE: [PATCH 2/2] Kbuild: Use dtc's -d (dependency) option

From: Mark Salter <hidden>
Date: 2012-01-17 18:27:24
Also in: linux-arm-kernel, linux-devicetree, linux-kbuild, lkml

On Tue, 2012-01-17 at 09:20 -0800, Stephen Warren wrote:
Michal Marek wrote at Saturday, January 14, 2012 3:51 PM:
quoted
On 11.1.2012 20:36, Mark Salter wrote:
quoted
On Mon, 2012-01-09 at 11:38 -0700, Stephen Warren wrote:
quoted
This hooks dtc into Kbuild's dependency system.

Thus, for example, "make dtbs" will rebuild tegra-harmony.dtb if only
tegra20.dtsi has changed yet tegra-harmony.dts has not. The previous
lack of this feature recently caused me to have very confusing "git
bisect" results.

For ARM, it's obvious what to add to $(targets). I'm not familiar enough
with other architectures to know what to add there. Powerpc appears to
already add various .dtb files into $(targets), but the other archs may
need something added to $(targets) to work.

Signed-off-by: Stephen Warren <redacted>
Acked-by: Mark Salter <redacted>

For the arch/c6x bit.
As the C6X port has been merged in this merge window, I can't easily
apply this patch to my kbuild branch and at the same time please Linus
by basing the pull request on a tagged release. So I'm going to drop the
arch/c6x part to keep the kbuild branch "pretty" and let either Mark or
Stephen send the patch to Linus directly, once he has merged the kbuild
branch. Would that work?
I was assuming my patches were for 3.4 rather than 3.3, since it's pretty
late for new features in 3.3. In that case, there wouldn't be an issue,
right?

If you do want to go ahead and push my patches into 3.3, your proposal
looks fine to me. Mark, do you want me to push the patch to Linus, or
do you want to pick it up?
It would probably be best to keep it all together, but if there's a need
to break out c6x individually, I don't mind picking it up.

--Mark
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help