From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Horman <horms@verge.net.au>
Date: Wed, 06 Nov 2013 07:18:06 +0000
Subject: Re: [PATCH 03/10] ARM: shmobile: r8a7790: Reference clocks
Message-Id: <20131106071800.GB17384@verge.net.au>
List-Id: <linux-sh.vger.kernel.org>
References: <1383059082-26315-4-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
In-Reply-To: <1383059082-26315-4-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: linux-sh@vger.kernel.org

On Tue, Oct 29, 2013 at 04:04:35PM +0100, Laurent Pinchart wrote:
> Reference clocks using a "clocks" property in all nodes corresponding to
> devices that require a clock.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
>  arch/arm/boot/dts/r8a7790-lager-reference.dts |  2 +-
>  arch/arm/boot/dts/r8a7790-lager.dts           |  2 +-
>  arch/arm/boot/dts/r8a7790.dtsi                | 12 ++++++++++++
>  3 files changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/r8a7790-lager-reference.dts b/arch/arm/boot/dts/r8a7790-lager-reference.dts
> index 75730f5..68711ce 100644
> --- a/arch/arm/boot/dts/r8a7790-lager-reference.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager-reference.dts
> @@ -9,7 +9,7 @@
>   */
>  
>  /dts-v1/;
> -/include/ "r8a7790.dtsi"
> +#include "r8a7790.dtsi"
>  #include <dt-bindings/gpio/gpio.h>
>  
>  / {
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
> index 65ffd87..310fa63 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -9,7 +9,7 @@
>   */
>  
>  /dts-v1/;
> -/include/ "r8a7790.dtsi"
> +#include "r8a7790.dtsi"
>  #include <dt-bindings/gpio/gpio.h>
>  
>  / {
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index 423500d..94c8373 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -8,6 +8,8 @@
>   * kind, whether express or implied.
>   */
>  
> +#include <dt-bindings/clock/r8a7790-clock.h>
> +
>  / {
>  	compatible = "renesas,r8a7790";
>  	interrupt-parent = <&gic>;

I believe that the hunks above need to be either present before or merged
into the previous patch to avoid the following build errors.

#make
...
  DTC     arch/arm/boot/dts/r8a7790-lager.dtb
  DTC     arch/arm/boot/dts/r8a7790-lager-reference.dtb
Error: arch/arm/boot/dts/r8a7790.dtsi:349.30-31 syntax error
Error: arch/arm/boot/dts/r8a7790.dtsi:349.30-31 syntax error
FATAL ERROR: Unable to parse input tree
FATAL ERROR: Unable to parse input tree

# dtc -v
Version: DTC 1.3.0