<div dir="ltr"><div>Hi Joel,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 2 Jul 2019 at 08:47, Joel Stanley <<a href="mailto:joel@jms.id.au">joel@jms.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, 1 Jul 2019 at 08:25, Samuel Jiang (江騏先)<br>
<<a href="mailto:Samuel.Jiang@quantatw.com" target="_blank">Samuel.Jiang@quantatw.com</a>> wrote:<br>
><br>
> Hi Joel,<br>
><br>
> Do you intend submit these patches to the mainline kernel for review?<br>
> I would like to see them submitted there first.<br>
><br>
> No, the patches still could not submit to the mainline kernel. We need Nuvoton teams help fill up the related source code in the mainline.<br>
<br></blockquote><div><br></div><div>We had submitted only basic NPCM7xx dtsi files to the Linux vanilla, We are planning to submit the NPCM7xx dtsi files soon but still I am not sure</div><div>if it will include all the nodes we use because some of our drivers are still in upstream process so things could change. </div><div><br></div><div>meanwhile could you please upstream Quanta to OpenBMC Linux tree?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Why can't you submit them?<br>
<br>
Patches are held in the openbmc tree only if they are on their way to<br>
be included in an upstream release.<br>
<br>
><br>
> When you do that, you don't need the "dev-5.17" part in your patch<br>
> subjects. You also can include the "tested" information in the cover<br>
> letter, but not in the patches themselves.<br>
><br>
> Based on the above reason, I prefer to keep “dev-5.17” part first and will update 1/6 patches information.<br>
<br>
Do you mean 5.1?<br>
<br>
> After Nuvoton team members back, I will discuss with them the mainline situation.<br>
><br>
> Thanks,<br>
><br>
> Samuel Jiang<br>
> On Jul 1, 2019, 2:57 PM +0800, Joel Stanley <<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>>, wrote:<br>
><br>
> Hi Samuel,<br>
><br>
> On Mon, 1 Jul 2019 at 06:02, Samuel Jiang <<a href="mailto:chyishian.jiang@gmail.com" target="_blank">chyishian.jiang@gmail.com</a>> wrote:<br>
><br>
><br>
> From: Samuel Jiang <<a href="mailto:Samuel.Jiang@quantatw.com" target="_blank">Samuel.Jiang@quantatw.com</a>><br>
><br>
> Initial Nuvoton RunBMC Module which use NPCM750 SoC.<br>
><br>
> Including features:<br>
> 1. image partitions<br>
> 2. lpc and kcs<br>
> 3. usb<br>
> 4. serial port<br>
> 5. spi<br>
> 6. fiu<br>
> 7. watchdog<br>
><br>
><br>
> Do you intend submit these patches to the mainline kernel for review?<br>
> I would like to see them submitted there first.<br>
><br>
> When you do that, you don't need the "dev-5.17" part in your patch<br>
> subjects. You also can include the "tested" information in the cover<br>
> letter, but not in the patches themselves.<br>
><br>
> Please test against 5.2.<br>
><br>
> Once these patches have had review we can include them in the openbmc tree.<br>
><br>
> Cheers,<br>
><br>
> Joel<br>
><br>
><br>
> Testeed:<br>
> Build Qunata runbmc-nuvoton image and load on RunBMC-nuvoton module<br>
><br>
> Signed-off-by: Samuel Jiang <<a href="mailto:Samuel.Jiang@quantatw.com" target="_blank">Samuel.Jiang@quantatw.com</a>><br>
> ---<br>
> arch/arm/boot/dts/nuvoton-npcm750-runbmc.dts | 292 +++++++++++++++++++<br>
> 1 file changed, 292 insertions(+)<br>
> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750-runbmc.dts<br>
><br>
> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-runbmc.dts b/arch/arm/boot/dts/nuvoton-npcm750-runbmc.dts<br>
> new file mode 100644<br>
> index 000000000000..eec815d2a638<br>
> --- /dev/null<br>
> +++ b/arch/arm/boot/dts/nuvoton-npcm750-runbmc.dts<br>
> @@ -0,0 +1,292 @@<br>
> +// SPDX-License-Identifier: GPL-2.0<br>
> +// Copyright (c) 2019 Nuvoton Technology <a href="mailto:kwliu@nuvoton.com" target="_blank">kwliu@nuvoton.com</a><br>
> +// Copyright (c) 2019 Quanta Computer Inc. <a href="mailto:Samuel.Jiang@quantatw.com" target="_blank">Samuel.Jiang@quantatw.com</a><br>
> +<br>
> +/dts-v1/;<br>
> +#include "nuvoton-npcm750.dtsi"<br>
> +<br>
> +/ {<br>
> + model = "Nuvoton npcm750 RunBMC Module";<br>
> + compatible = "nuvoton,npcm750";<br>
> +<br>
> + aliases {<br>
> + ethernet0 = &emc0;<br>
> + ethernet1 = &gmac0;<br>
> + serial0 = &serial0;<br>
> + serial1 = &serial1;<br>
> + serial2 = &serial2;<br>
> + serial3 = &serial3;<br>
> + udc0 = &udc0;<br>
> + udc1 = &udc1;<br>
> + udc2 = &udc2;<br>
> + udc3 = &udc3;<br>
> + udc4 = &udc4;<br>
> + udc5 = &udc5;<br>
> + udc6 = &udc6;<br>
> + udc7 = &udc7;<br>
> + udc8 = &udc8;<br>
> + udc9 = &udc9;<br>
> + emmc0 = &sdhci0;<br>
> + spi0 = &spi0;<br>
> + spi1 = &spi1;<br>
> + fiu0 = &fiu0;<br>
> + fiu1 = &fiu3;<br>
> + };<br>
> +<br>
> + chosen {<br>
> + stdout-path = &serial3;<br>
> + };<br>
> +<br>
> + memory {<br>
> + reg = <0 0x40000000>;<br>
> + };<br>
> +<br>
> + ahb {<br>
> + gmac0: eth@f0802000 {<br>
> + phy-mode = "rgmii-id";<br>
> + snps,eee-force-disable;<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + emc0: eth@f0825000 {<br>
> + phy-mode = "rmii";<br>
> + use-ncsi;<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + ehci1: usb@f0806000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + ohci1: ohci@f0807000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc0:udc@f0830000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc1:udc@f0831000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc2:udc@f0832000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc3:udc@f0833000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc4:udc@f0834000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc5:udc@f0835000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc6:udc@f0836000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc7:udc@f0837000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc8:udc@f0838000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + udc9:udc@f0839000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + aes:aes@f0858000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + sha:sha@f085a000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + fiu0: fiu@fb000000 {<br>
> + pinctrl-names = "default";<br>
> + pinctrl-0 = <&spi0cs1_pins>;<br>
> + status = "okay";<br>
> + spi-nor@0 {<br>
> + compatible = "jedec,spi-nor";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + reg = <0>;<br>
> + spi-rx-bus-width = <2>;<br>
> + partitions@80000000 {<br>
> + compatible = "fixed-partitions";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + bmc@0{<br>
> + label = "bmc";<br>
> + reg = <0x000000 0x4000000>;<br>
> + };<br>
> + u-boot@0 {<br>
> + label = "u-boot";<br>
> + read-only;<br>
> + reg = <0x0000000 0x80000>;<br>
> + };<br>
> + u-boot-env@100000 {<br>
> + label = "u-boot-env";<br>
> + reg = <0x00100000 0x40000>;<br>
> + };<br>
> + kernel@200000 {<br>
> + label = "kernel";<br>
> + reg = <0x0200000 0x600000>;<br>
> + };<br>
> + rofs@800000 {<br>
> + label = "rofs";<br>
> + reg = <0x0800000 0x1500000>;<br>
> + };<br>
> + rwfs@1c00000 {<br>
> + label = "rwfs";<br>
> + reg = <0x1c00000 0x300000>;<br>
> + };<br>
> + };<br>
> + };<br>
> + spi-nor@1 {<br>
> + compatible = "jedec,spi-nor";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + reg = <1>;<br>
> + npcm,fiu-rx-bus-width = <2>;<br>
> + partitions@88000000 {<br>
> + compatible = "fixed-partitions";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + spare1@0 {<br>
> + label = "spi0-cs1-spare1";<br>
> + reg = <0x0 0x800000>;<br>
> + };<br>
> + spare2@800000 {<br>
> + label = "spi0-cs1-spare2";<br>
> + reg = <0x800000 0x0>;<br>
> + };<br>
> + };<br>
> + };<br>
> + };<br>
> +<br>
> + fiu3: fiu@c0000000 {<br>
> + pinctrl-0 = <&spi3_pins>;<br>
> + status = "okay";<br>
> + spi-nor@0 {<br>
> + compatible = "jedec,spi-nor";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + reg = <0>;<br>
> + spi-rx-bus-width = <2>;<br>
> + partitions@A0000000 {<br>
> + compatible = "fixed-partitions";<br>
> + #address-cells = <1>;<br>
> + #size-cells = <1>;<br>
> + system1@0 {<br>
> + label = "spi3-system1";<br>
> + reg = <0x0 0x800000>;<br>
> + };<br>
> + system2@800000 {<br>
> + label = "spi3-system2";<br>
> + reg = <0x800000 0x0>;<br>
> + };<br>
> + };<br>
> + };<br>
> + };<br>
> +<br>
> + sdhci0: sdhci@f0842000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + pcimbox: pcimbox@f0848000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + vcd: vcd@f0810000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + ece: ece@f0820000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + apb {<br>
> +<br>
> + watchdog1: watchdog@901C {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + rng: rng@b000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + serial0: serial@1000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + serial1: serial@2000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + serial2: serial@3000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + serial3: serial@4000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + adc: adc@c000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + otp:otp@189000 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + lpc_kcs: lpc_kcs@7000 {<br>
> + kcs1: kcs1@0 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + kcs2: kcs2@0 {<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + kcs3: kcs3@0 {<br>
> + status = "okay";<br>
> + };<br>
> + };<br>
> +<br>
> + lpc_host: lpc_host@7000 {<br>
> + lpc_bpc: lpc_bpc@40 {<br>
> + monitor-ports = <0x80>;<br>
> + status = "okay";<br>
> + };<br>
> + };<br>
> +<br>
> + spi0: spi@200000 {<br>
> + cs-gpios = <&gpio6 11 GPIO_ACTIVE_LOW>;<br>
> + status = "okay";<br>
> + };<br>
> +<br>
> + spi1: spi@201000 {<br>
> + status = "okay";<br>
> + };<br>
> + };<br>
> + };<br>
> +};<br>
> +<br>
> +&gcr {<br>
> + serial_port_mux: mux-controller {<br>
> + compatible = "mmio-mux";<br>
> + #mux-control-cells = <1>;<br>
> +<br>
> + mux-reg-masks = <0x38 0x07>;<br>
> + idle-states = <6>;<br>
> + };<br>
> +};<br>
> --<br>
> 2.20.1<br>
><br></blockquote><div><br></div><div>Thanks a lot!</div><div><br></div><div>Tomer </div></div></div>