From mboxrd@z Thu Jan  1 00:00:00 1970
From: shawn.guo@linaro.org (Shawn Guo)
Date: Mon, 11 May 2015 22:36:45 +0800
Subject: [PATCH v2] ARM: dts: cubox-i: add support for Broadcom
 Wifi/Bluetooth devices
In-Reply-To: <CAOMZO5AYq1TBWsTUZ2Csy98qy2Ma-MSt+4uwwDUOaGnZaYUiiA@mail.gmail.com>
References: <1430932810-28756-1-git-send-email-festevam@gmail.com>
 <20150511130016.GE9347@dragon>
 <CAOMZO5Cy1uhbqSfRKw_K++VhTQW7Z29GwKNAiLgB_UrF4YG9-Q@mail.gmail.com>
 <CAOMZO5AYq1TBWsTUZ2Csy98qy2Ma-MSt+4uwwDUOaGnZaYUiiA@mail.gmail.com>
Message-ID: <20150511143644.GP9347@dragon>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Mon, May 11, 2015 at 11:15:33AM -0300, Fabio Estevam wrote:
> On Mon, May 11, 2015 at 10:46 AM, Fabio Estevam <festevam@gmail.com> wrote:
> > Hi Shawn,
> >
> > On Mon, May 11, 2015 at 10:00 AM, Shawn Guo <shawn.guo@linaro.org> wrote:
> >
> >> One of the device tree maintainers doesn't like "clocks" container.
> >> Since you're not using generic node name for child clock, the container
> >> can just be saved.
> >
> > Just to make sure I understand your suggestion: would you like the
> > clock nodes to look like this?
> 
> I think I got it. Do you mean like this?

Yes.

shawn

> 
> #include <dt-bindings/gpio/gpio.h>
> / {
>     clk_sdio: sdio-clock {
>         compatible = "gpio-gate-clock";
>         #clock-cells = <0>;
>         pinctrl-names = "default";
>         pinctrl-0 = <&pinctrl_microsom_brcm_osc>;
>         enable-gpios = <&gpio5 5 GPIO_ACTIVE_HIGH>;
>     };
> 
>     regulators {
>         compatible = "simple-bus";