Re: [PATCH v2] arm64: dts: mt8183-kukui: add scp node
On 09/09/2020 10:14, Pi-Hsun Shih wrote: Add scp node to mt8183-kukui Fixes: 0d5e41709f76 ("arm64: dts: mt8183: add scp node") Signed-off-by: Pi-Hsun Shih Applied to v5.9-next/dts64 I squashed this fix with the first patch to maintain bisectability. Thanks! --- Change since v1: * Add Fixes tag. --- .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 30 +++ 1 file changed, 30 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi index f0a070535b34..85f7c33ba446 100644 --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi @@ -90,6 +90,18 @@ pp3300_alw: regulator6 { regulator-max-microvolt = <330>; }; + reserved_memory: reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + scp_mem_reserved: scp_mem_region { + compatible = "shared-dma-pool"; + reg = <0 0x5000 0 0x290>; + no-map; + }; + }; + max98357a: codec0 { compatible = "maxim,max98357a"; sdmode-gpios = <&pio 175 0>; @@ -524,6 +536,13 @@ pins_clk { }; }; + scp_pins: scp { + pins_scp_uart { + pinmux = , +; + }; + }; + spi0_pins: spi0 { pins_spi{ pinmux = , @@ -651,6 +670,17 @@ pins_wifi_wakeup { }; }; +&scp { + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&scp_pins>; + + cros_ec { + compatible = "google,cros-ec-rpmsg"; + mtk,rpmsg-name = "cros-ec-rpmsg"; + }; +}; + &soc_data { status = "okay"; };
Re: [PATCH v2] arm64: dts: mt8183-kukui: add scp node
On Wed, Sep 9, 2020 at 4:58 PM Matthias Brugger wrote: > > > > On 09/09/2020 10:14, Pi-Hsun Shih wrote: > > Add scp node to mt8183-kukui > > > > Fixes: 0d5e41709f76 ("arm64: dts: mt8183: add scp node") > > Signed-off-by: Pi-Hsun Shih > > --- > > > > Change since v1: > > * Add Fixes tag. > > > > --- > > .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 30 +++ > > 1 file changed, 30 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi > > b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi > > index f0a070535b34..85f7c33ba446 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi > > +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi > > @@ -90,6 +90,18 @@ pp3300_alw: regulator6 { > > regulator-max-microvolt = <330>; > > }; > > > > + reserved_memory: reserved-memory { > > + #address-cells = <2>; > > + #size-cells = <2>; > > + ranges; > > + > > + scp_mem_reserved: scp_mem_region { > > + compatible = "shared-dma-pool"; > > + reg = <0 0x5000 0 0x290>; > > + no-map; > > + }; > > + }; > > + > > Do we expect other boards to have a different memory reservation? I can see > that > EVB and Kukui uses the same. If not, we should add the node in mt8183.dtsi > instead. > > Regards, > Matthias All our current boards use the same memory reservation, but it's still possible for future boards based on mt8183 to use different SCP memory size, so I feel this should still be left out of mt8183.dtsi. > > > max98357a: codec0 { > > compatible = "maxim,max98357a"; > > sdmode-gpios = <&pio 175 0>; > > @@ -524,6 +536,13 @@ pins_clk { > > }; > > }; > > > > + scp_pins: scp { > > + pins_scp_uart { > > + pinmux = , > > + ; > > + }; > > + }; > > + > > spi0_pins: spi0 { > > pins_spi{ > > pinmux = , > > @@ -651,6 +670,17 @@ pins_wifi_wakeup { > > }; > > }; > > > > +&scp { > > + status = "okay"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&scp_pins>; > > + > > + cros_ec { > > + compatible = "google,cros-ec-rpmsg"; > > + mtk,rpmsg-name = "cros-ec-rpmsg"; > > + }; > > +}; > > + > > &soc_data { > > status = "okay"; > > }; > >
Re: [PATCH v2] arm64: dts: mt8183-kukui: add scp node
On 09/09/2020 10:14, Pi-Hsun Shih wrote: Add scp node to mt8183-kukui Fixes: 0d5e41709f76 ("arm64: dts: mt8183: add scp node") Signed-off-by: Pi-Hsun Shih --- Change since v1: * Add Fixes tag. --- .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 30 +++ 1 file changed, 30 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi index f0a070535b34..85f7c33ba446 100644 --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi @@ -90,6 +90,18 @@ pp3300_alw: regulator6 { regulator-max-microvolt = <330>; }; + reserved_memory: reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + scp_mem_reserved: scp_mem_region { + compatible = "shared-dma-pool"; + reg = <0 0x5000 0 0x290>; + no-map; + }; + }; + Do we expect other boards to have a different memory reservation? I can see that EVB and Kukui uses the same. If not, we should add the node in mt8183.dtsi instead. Regards, Matthias max98357a: codec0 { compatible = "maxim,max98357a"; sdmode-gpios = <&pio 175 0>; @@ -524,6 +536,13 @@ pins_clk { }; }; + scp_pins: scp { + pins_scp_uart { + pinmux = , +; + }; + }; + spi0_pins: spi0 { pins_spi{ pinmux = , @@ -651,6 +670,17 @@ pins_wifi_wakeup { }; }; +&scp { + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&scp_pins>; + + cros_ec { + compatible = "google,cros-ec-rpmsg"; + mtk,rpmsg-name = "cros-ec-rpmsg"; + }; +}; + &soc_data { status = "okay"; };
[PATCH v2] arm64: dts: mt8183-kukui: add scp node
Add scp node to mt8183-kukui Fixes: 0d5e41709f76 ("arm64: dts: mt8183: add scp node") Signed-off-by: Pi-Hsun Shih --- Change since v1: * Add Fixes tag. --- .../arm64/boot/dts/mediatek/mt8183-kukui.dtsi | 30 +++ 1 file changed, 30 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi index f0a070535b34..85f7c33ba446 100644 --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui.dtsi @@ -90,6 +90,18 @@ pp3300_alw: regulator6 { regulator-max-microvolt = <330>; }; + reserved_memory: reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + scp_mem_reserved: scp_mem_region { + compatible = "shared-dma-pool"; + reg = <0 0x5000 0 0x290>; + no-map; + }; + }; + max98357a: codec0 { compatible = "maxim,max98357a"; sdmode-gpios = <&pio 175 0>; @@ -524,6 +536,13 @@ pins_clk { }; }; + scp_pins: scp { + pins_scp_uart { + pinmux = , +; + }; + }; + spi0_pins: spi0 { pins_spi{ pinmux = , @@ -651,6 +670,17 @@ pins_wifi_wakeup { }; }; +&scp { + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&scp_pins>; + + cros_ec { + compatible = "google,cros-ec-rpmsg"; + mtk,rpmsg-name = "cros-ec-rpmsg"; + }; +}; + &soc_data { status = "okay"; }; -- 2.28.0.526.ge36021eeef-goog