Hi Jagan, Am Mittwoch, 20. November 2019, 12:39:22 CET schrieb Jagan Teki: > Carrier board often referred as baseboard. For making > complete SBC, the associated SOM will mount on top of > this carrier board. > > Radxa has a carrier board which supports on board > peripherals, ports like USB-2.0, USB-3.0, HDMI, MIPI DSI/CSI, > eDP, Ethernet, PCIe, USB-C, 40-Pin GPIO header and etc. > > Currently this carrier board can be used together with > VMARC RK3399Por SOM for making Rock PI N10 SBC. > > So add this carrier board dtsi as a separate file in > ARM directory, so-that the same can reuse it in both > arm32 and arm64 variants of Rockchip SOMs. Do you really think someone will create an arm32 soc using that carrier board? Similarly so far I don't think we haven't even seen a lot of reuse of existing carrier boards at all, other than their initial combination. So maybe just having the content of your rockchip-radxa-carrierboard.dtsi in rockchip/rk3399pro-rock-pi-n10.dts from patch 5 might be a better start - at least until there is any further usage - if at all? Also rockchip-radxa-carrierboard might even be overly generic as there may be multiple carrierboards from Radxa later on. Heiko > Signed-off-by: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > --- > .../boot/dts/rockchip-radxa-carrierboard.dtsi | 81 +++++++++++++++++++ > 1 file changed, 81 insertions(+) > create mode 100644 arch/arm/boot/dts/rockchip-radxa-carrierboard.dtsi > > diff --git a/arch/arm/boot/dts/rockchip-radxa-carrierboard.dtsi b/arch/arm/boot/dts/rockchip-radxa-carrierboard.dtsi > new file mode 100644 > index 000000000000..df3712aedf8a > --- /dev/null > +++ b/arch/arm/boot/dts/rockchip-radxa-carrierboard.dtsi > @@ -0,0 +1,81 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright (c) 2019 Fuzhou Rockchip Electronics Co., Ltd > + * Copyright (c) 2019 Radxa Limited > + * Copyright (c) 2019 Amarula Solutions(India) > + */ > + > +#include <dt-bindings/pwm/pwm.h> > + > +/ { > + chosen { > + stdout-path = "serial2:1500000n8"; > + }; > +}; > + > +&gmac { > + status = "okay"; > +}; > + > +&i2c1 { > + status = "okay"; > + i2c-scl-rising-time-ns = <140>; > + i2c-scl-falling-time-ns = <30>; > +}; > + > +&i2c2 { > + status = "okay"; > + clock-frequency = <400000>; > + > + hym8563: hym8563@51 { > + compatible = "haoyu,hym8563"; > + reg = <0x51>; > + #clock-cells = <0>; > + clock-frequency = <32768>; > + clock-output-names = "hym8563"; > + pinctrl-names = "default"; > + pinctrl-0 = <&hym8563_int>; > + interrupt-parent = <&gpio4>; > + interrupts = <30 IRQ_TYPE_LEVEL_LOW>; > + }; > +}; > + > +&pwm0 { > + status = "okay"; > +}; > + > +&pwm2 { > + status = "okay"; > +}; > + > +&sdmmc { > + bus-width = <4>; > + cap-mmc-highspeed; > + cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA7 GPIO_ACTIVE_LOW>; > + disable-wp; > + vqmmc-supply = <&vccio_sd>; > + max-frequency = <150000000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&sdmmc_clk &sdmmc_cmd &sdmmc_cd &sdmmc_bus4>; > + status = "okay"; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart0_xfer &uart0_cts>; > + status = "okay"; > +}; > + > +&uart2 { > + status = "okay"; > +}; > + > +&pinctrl { > + hym8563 { > + hym8563_int: hym8563-int { > + rockchip,pins = > + <4 RK_PD6 0 &pcfg_pull_up>; > + }; > + }; > +}; > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-rockchip