From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jagannadh.teki@gmail.com>
Date: Wed, 7 Jun 2017 13:30:13 +0530
Subject: [U-Boot] [PATCH v3 04/10] mips: bmips: add bcm63xx-spi driver
 support for BCM6338
In-Reply-To: <1496483865-25782-5-git-send-email-noltari@gmail.com>
References: <1495135788-9152-1-git-send-email-noltari@gmail.com>
 <1496483865-25782-1-git-send-email-noltari@gmail.com>
 <1496483865-25782-5-git-send-email-noltari@gmail.com>
Message-ID: <CAD6G_RQFLi-PSpkuNTTX47MMsHOKJJYfc1YrDaxSPc-dqP5R-A@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

On Sat, Jun 3, 2017 at 3:27 PM, Álvaro Fernández Rojas
<noltari@gmail.com> wrote:
> This driver manages the SPI controller present on this SoC.
>
> Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
> ---
>  v3: rename BCM6338 SPI driver to BCM6348
>  v2: add spi alias
>
>  arch/mips/dts/brcm,bcm6338.dtsi | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>
> diff --git a/arch/mips/dts/brcm,bcm6338.dtsi b/arch/mips/dts/brcm,bcm6338.dtsi
> index eb51a43..0cab44c 100644
> --- a/arch/mips/dts/brcm,bcm6338.dtsi
> +++ b/arch/mips/dts/brcm,bcm6338.dtsi
> @@ -12,6 +12,10 @@
>  / {
>         compatible = "brcm,bcm6338";
>
> +       aliases {
> +               spi0 = &spi;
> +       };
> +
>         cpus {
>                 reg = <0xfffe0000 0x4>;
>                 #address-cells = <1>;
> @@ -109,6 +113,19 @@
>                         status = "disabled";
>                 };
>
> +               spi: spi at fffe0c00 {
> +                       compatible = "brcm,bcm6348-spi";
> +                       reg = <0xfffe0c00 0xc0>;
> +                       #address-cells = <1>;
> +                       #size-cells = <0>;
> +                       clocks = <&periph_clk BCM6338_CLK_SPI>;
> +                       resets = <&periph_rst BCM6338_RST_SPI>;
> +                       spi-max-frequency = <20000000>;
> +                       num-cs = <4>;
> +

space?

> +                       status = "disabled";
> +               };

As per dts(or with new node) additions/update we generally rely with
Linux for maximum possible extent So can you try to add this in Linux
first so-that we can have proper sync.

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.