From: khilman@linaro.org (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v12 07/08] ARM: dts: enable clock binding on Hi3620
Date: Tue, 17 Dec 2013 16:20:07 -0800 [thread overview]
Message-ID: <878uvjnh6w.fsf@linaro.org> (raw)
In-Reply-To: <1386748497-11290-8-git-send-email-haojian.zhuang@gmail.com> (Haojian Zhuang's message of "Wed, 11 Dec 2013 15:54:56 +0800")
Haojian Zhuang <haojian.zhuang@linaro.org> writes:
> Enable clock binding for Hi3620 common clock driver.
>
> Signed-off-by: Haojian Zhuang <haojian.zhuang@gmail.com>
> ---
> arch/arm/boot/dts/hi3620.dtsi | 79 ++++++++++++++++++++++++-------------------
> arch/arm/boot/dts/hi4511.dts | 3 +-
> 2 files changed, 46 insertions(+), 36 deletions(-)
>
> diff --git a/arch/arm/boot/dts/hi3620.dtsi b/arch/arm/boot/dts/hi3620.dtsi
> index e311937..ab1116d 100644
> --- a/arch/arm/boot/dts/hi3620.dtsi
> +++ b/arch/arm/boot/dts/hi3620.dtsi
> @@ -11,7 +11,8 @@
> * publishhed by the Free Software Foundation.
> */
>
> -/include/ "skeleton.dtsi"
> +#include "skeleton.dtsi"
> +#include <dt-bindings/clock/hi3620-clock.h>
This file doesn't exist in this series, so fails to build. This appears
to depend on the clock series that Mike has merged into his clk-next
branch, so I tested the build with clk-next and it works.
In the future, please be sure to mention external dependencies like
this. Maintainers don't like to be surprised by this kind of thing.
I'll just merge this as is now, but builds for hi3xxx_defconfig will be
broken until everything hits -next.
Kevin
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
To: Haojian Zhuang <haojian.zhuang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org,
arnd-r2nGTMty4D4@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
patches-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
Haojian Zhuang
<haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"Turquette,
Mike" <mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH v12 07/08] ARM: dts: enable clock binding on Hi3620
Date: Tue, 17 Dec 2013 16:20:07 -0800 [thread overview]
Message-ID: <878uvjnh6w.fsf@linaro.org> (raw)
In-Reply-To: <1386748497-11290-8-git-send-email-haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> (Haojian Zhuang's message of "Wed, 11 Dec 2013 15:54:56 +0800")
Haojian Zhuang <haojian.zhuang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> writes:
> Enable clock binding for Hi3620 common clock driver.
>
> Signed-off-by: Haojian Zhuang <haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> arch/arm/boot/dts/hi3620.dtsi | 79 ++++++++++++++++++++++++-------------------
> arch/arm/boot/dts/hi4511.dts | 3 +-
> 2 files changed, 46 insertions(+), 36 deletions(-)
>
> diff --git a/arch/arm/boot/dts/hi3620.dtsi b/arch/arm/boot/dts/hi3620.dtsi
> index e311937..ab1116d 100644
> --- a/arch/arm/boot/dts/hi3620.dtsi
> +++ b/arch/arm/boot/dts/hi3620.dtsi
> @@ -11,7 +11,8 @@
> * publishhed by the Free Software Foundation.
> */
>
> -/include/ "skeleton.dtsi"
> +#include "skeleton.dtsi"
> +#include <dt-bindings/clock/hi3620-clock.h>
This file doesn't exist in this series, so fails to build. This appears
to depend on the clock series that Mike has merged into his clk-next
branch, so I tested the build with clk-next and it works.
In the future, please be sure to mention external dependencies like
this. Maintainers don't like to be surprised by this kind of thing.
I'll just merge this as is now, but builds for hi3xxx_defconfig will be
broken until everything hits -next.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2013-12-18 0:20 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-11 7:54 [PATCH v12 00/08] enable hisilicon hi3xxx SoC Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-11 7:54 ` [PATCH v12 01/08] ARM: hi3xxx: add board support with device tree Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-18 0:06 ` Kevin Hilman
2013-12-18 0:06 ` Kevin Hilman
2013-12-11 7:54 ` [PATCH v12 02/08] ARM: dts: enable hi4511 " Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-11 7:54 ` [PATCH v12 03/08] ARM: config: enable hi3xxx in multi_v7_defconfig Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-11 7:54 ` [PATCH v12 04/08] ARM: config: add defconfig for Hi3xxx Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-11 7:54 ` [PATCH v12 05/08] ARM: hi3xxx: add smp support Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-18 0:38 ` Kevin Hilman
2013-12-18 0:38 ` Kevin Hilman
2013-12-11 7:54 ` [PATCH v12 06/08] ARM: hi3xxx: add hotplug support Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-18 0:40 ` Kevin Hilman
2013-12-18 0:40 ` Kevin Hilman
2013-12-11 7:54 ` [PATCH v12 07/08] ARM: dts: enable clock binding on Hi3620 Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-18 0:20 ` Kevin Hilman [this message]
2013-12-18 0:20 ` Kevin Hilman
2013-12-11 7:54 ` [PATCH v12 08/08] ARM: hi3xxx: select GPIO in Kconfig Haojian Zhuang
2013-12-11 7:54 ` Haojian Zhuang
2013-12-18 0:44 ` Kevin Hilman
2013-12-18 0:44 ` Kevin Hilman
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=878uvjnh6w.fsf@linaro.org \
--to=khilman@linaro.org \
--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 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.