Messages in this thread | | | Date | Thu, 29 Sep 2011 08:47:10 -0700 | Subject | Re: [PATCH v5 2/3] arm/tegra: add support for ventana pinmuxing | From | Olof Johansson <> |
| |
On Thu, Sep 29, 2011 at 8:27 AM, Peter De Schrijver <pdeschrijver@nvidia.com> wrote: > On Thu, Sep 29, 2011 at 11:29:51AM +0200, Russell King - ARM Linux wrote: >> On Thu, Sep 29, 2011 at 01:29:35AM +0300, Peter De Schrijver wrote: >> > diff --git a/arch/arm/mach-tegra/Makefile b/arch/arm/mach-tegra/Makefile >> > index f11b910..91a07e1 100644 >> > --- a/arch/arm/mach-tegra/Makefile >> > +++ b/arch/arm/mach-tegra/Makefile >> > @@ -31,6 +31,7 @@ obj-${CONFIG_MACH_SEABOARD} += board-seaboard-pinmux.o >> > >> > obj-${CONFIG_MACH_TEGRA_DT} += board-dt.o >> > obj-${CONFIG_MACH_TEGRA_DT} += board-harmony-pinmux.o >> > +obj-${CONFIG_MACH_TEGRA_DT} += board-seaboard-pinmux.o >> >> Since when did Tegra start using braces instead of parens in its Makefile? >> I note some of Tegra's makefile is using braces and some of it parens. >> What's the reason for this alternative style to the rest of kbuild? > > I have no idea. I just followed the style used for the other CONFIG_MACH_TEGRA_DT entries. Colin or Olof, any idea?
From git logs, it looks like the first harmony board file that came from Colin used braces, and from there it's likely that everyone just copied and pasted -- I know that's what I did.
Thanks for catching it, Russell, it had obviously completely gone unnoticed by me and others.
I'll fix up the rest of the file. Peter -- no need to respin, I'll fix up when I apply.
-Olof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |