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