From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ryder Lee <ryder.lee@mediatek.com>
Date: Tue, 2 Oct 2018 21:34:28 +0800
Subject: [U-Boot] [PATCH 04/20] arm: dts: MediaTek: add MT7629 reference
 board support
In-Reply-To: <40d4a090-a192-3617-adb6-44336f3667e6@gmail.com>
References: <cover.1538460580.git.ryder.lee@mediatek.com>
 <18b18d60f006d1da99b9442cac2424963d37958c.1538460580.git.ryder.lee@mediatek.com>
 <40d4a090-a192-3617-adb6-44336f3667e6@gmail.com>
Message-ID: <1538487268.18888.17.camel@mtkswgap22>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Tue, 2018-10-02 at 15:07 +0200, Matthias Brugger wrote:
> I'm very happy to see Mediatek working on mainline u-boot support!
> 
> On 02/10/2018 08:13, Ryder Lee wrote:
> > This patch adds mt7629.dtsi and reference board support.
> I think reference board support is already added in 2/20.
> AFAICS you just device tree and the includes it needs.
> 
> > 
> > Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
> > ---
> >  arch/arm/dts/Makefile                    |   3 +
> >  arch/arm/dts/mt7629-rfb.dts              |  71 ++++++++++
> >  arch/arm/dts/mt7629.dtsi                 | 232 +++++++++++++++++++++++++++++++
> >  include/configs/mt7629.h                 |   2 +
> >  include/dt-bindings/clock/mt7629-clk.h   | 217 +++++++++++++++++++++++++++++
> >  include/dt-bindings/power/mt7629-power.h |  13 ++
> >  6 files changed, 538 insertions(+)
> >  create mode 100644 arch/arm/dts/mt7629-rfb.dts
> >  create mode 100644 arch/arm/dts/mt7629.dtsi
> >  create mode 100644 include/dt-bindings/clock/mt7629-clk.h
> >  create mode 100644 include/dt-bindings/power/mt7629-power.h
> > 
> [...]
> > +
> > +	watchdog: watchdog at 10212000 {
> > +		compatible = "mediatek,wdt";
> 
> Will this be passed to the kernel later?

No, I didn't pass dtb in my tests. I just let them independent.

> If so, you should add a upstream-like compatible (at least).
> BTW none of the mt7629 compatible is upstream in the kernel. Not sure if that's
> an issue for u-boot, TBH.

I'm not sure either. If we decide to pass the U-boot dtb to kernel I
will modify the string in v1. A related question is that we rely on the
fallback compatible "mediatek,mt6577-*" for timer, wdt, and other common
IPs, but obviously, we didn't have MT6577 support in U-boot. That's why
I pick a generic name for timer and WDT.

Also, I directly use "ns16500a" as compatible for uart in U-boot.

Moreover, I think/guess the DTS is hard to share with kernel as there
are many properties don't support in U-boot stage.

Ryder

> > +		reg = <0x10212000 0x600>;
> > +		interrupts = <GIC_SPI 128 IRQ_TYPE_EDGE_FALLING>;
> > +		#reset-cells = <1>;
> > +		status = "disabled";
> > +	};
> > +
> > +	wdt-reboot {
> > +		compatible = "wdt-reboot";
> > +		wdt = <&watchdog>;
> > +	};
> > +
> 
> Regards,
> Matthias