lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <52e66b1d-ad00-52a0-17ae-06e898f3b13e@arm.com>
Date:   Thu, 16 Mar 2017 16:15:49 +0000
From:   Andre Przywara <andre.przywara@....com>
To:     cl@...k-chips.com, heiko@...ech.de
Cc:     mark.rutland@....com, wsa@...-dreams.de, linux-iio@...r.kernel.org,
        catalin.marinas@....com, shawn.lin@...k-chips.com,
        will.deacon@....com, kever.yang@...k-chips.com,
        dianders@...omium.org, yamada.masahiro@...ionext.com,
        tony.xie@...k-chips.com, linux-i2c@...r.kernel.org,
        pmeerw@...erw.net, lars@...afoo.de, zhengxing@...k-chips.com,
        khilman@...libre.com, linux-rockchip@...ts.infradead.org,
        jay.xu@...k-chips.com, wxt@...k-chips.com, huangtao@...k-chips.com,
        devicetree@...r.kernel.org, zhangqing@...k-chips.com,
        paweljarosz3691@...il.com, arnd@...db.de, yhx@...k-chips.com,
        knaack.h@....de, robh+dt@...nel.org, matthias.bgg@...il.com,
        rocky.hao@...k-chips.com, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org, david.wu@...k-chips.com,
        fabio.estevam@....com, andy.yan@...k-chips.com,
        akpm@...ux-foundation.org, shawnguo@...nel.org, afaerber@...e.de,
        jic23@...nel.org
Subject: Re: [PATCH v1 5/7] arm64: dts: rockchip: add core dtsi file for
 RK3328 SoCs

Hi Chen,

thanks for posting this.

And great to see those compatible strings used so nicely!

On 16/03/17 01:44, cl@...k-chips.com wrote:
> From: Chen Liang <cl@...k-chips.com>
> 
> This patch adds core dtsi file for Rockchip RK3328 SoCs.
> 
> Signed-off-by: Chen Liang <cl@...k-chips.com>
> ---
>  arch/arm64/boot/dts/rockchip/rk3328.dtsi | 1362 ++++++++++++++++++++++++++++++
>  1 file changed, 1362 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/rockchip/rk3328.dtsi
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3328.dtsi b/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> new file mode 100644
> index 0000000..ff53af9
> --- /dev/null
> +++ b/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> @@ -0,0 +1,1362 @@
> +/*
> + * Copyright (c) 2017 Fuzhou Rockchip Electronics Co., Ltd
> + *
> + * This file is dual-licensed: you can use it either under the terms
> + * of the GPL or the X11 license, at your option. Note that this dual
> + * licensing only applies to this file, and not this project as a
> + * whole.
> + *
> + *  a) This library is free software; you can redistribute it and/or
> + *     modify it under the terms of the GNU General Public License as
> + *     published by the Free Software Foundation; either version 2 of the
> + *     License, or (at your option) any later version.
> + *
> + *     This library is distributed in the hope that it will be useful,
> + *     but WITHOUT ANY WARRANTY; without even the implied warranty of
> + *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + *     GNU General Public License for more details.
> + *
> + * Or, alternatively,
> + *
> + *  b) Permission is hereby granted, free of charge, to any person
> + *     obtaining a copy of this software and associated documentation
> + *     files (the "Software"), to deal in the Software without
> + *     restriction, including without limitation the rights to use,
> + *     copy, modify, merge, publish, distribute, sublicense, and/or
> + *     sell copies of the Software, and to permit persons to whom the
> + *     Software is furnished to do so, subject to the following
> + *     conditions:
> + *
> + *     The above copyright notice and this permission notice shall be
> + *     included in all copies or substantial portions of the Software.
> + *
> + *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
> + *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> + *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> + *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> + *     OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#include <dt-bindings/clock/rk3328-cru.h>
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +#include <dt-bindings/pinctrl/rockchip.h>
> +#include <dt-bindings/soc/rockchip,boot-mode.h>
> +#include <dt-bindings/power/rk3328-power.h>
> +
> +/ {
> +	compatible = "rockchip,rk3328";
> +
> +	interrupt-parent = <&gic>;
> +	#address-cells = <2>;
> +	#size-cells = <2>;
> +
> +	aliases {
> +		serial0 = &uart0;
> +		serial1 = &uart1;
> +		serial2 = &uart2;
> +		i2c0 = &i2c0;
> +		i2c1 = &i2c1;
> +		i2c2 = &i2c2;
> +		i2c3 = &i2c3;
> +	};
> +
> +	cpus {
> +		#address-cells = <2>;
> +		#size-cells = <0>;
> +
> +		cpu0: cpu@0 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a53", "arm,armv8";
> +			reg = <0x0 0x0>;
> +			enable-method = "psci";
> +			clocks = <&cru ARMCLK>;

Can you add an "next-level-cache" property here (and in every other CPU
node)? This avoids a warning when the kernel boots:
...
Unable to detect cache hierarchy from DT for CPU <x>
...
(And of course the rather simple L2 cache entry as well).
See amlogic/meson-gx.dtsi for an example.

> +		};
> +		cpu1: cpu@1 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a53", "arm,armv8";
> +			reg = <0x0 0x1>;
> +			enable-method = "psci";
> +		};
> +		cpu2: cpu@2 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a53", "arm,armv8";
> +			reg = <0x0 0x2>;
> +			enable-method = "psci";
> +		};
> +		cpu3: cpu@3 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a53", "arm,armv8";
> +			reg = <0x0 0x3>;
> +			enable-method = "psci";
> +		};
> +	};
> +
> +	arm-pmu {
> +		compatible = "arm,cortex-a53-pmu";
> +		interrupts = <GIC_SPI 100 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>;
> +		interrupt-affinity = <&cpu0>, <&cpu1>, <&cpu2>, <&cpu3>;
> +	};
> +
> +	psci {
> +		compatible = "arm,psci-1.0";

I wonder if it would be more flexible to have an additional PSCI 0.2
compatible string as a fallback in here, for supporting OSes which don't
know about PSCI 1.0?

> +		method = "smc";
> +	};
> +
> +	timer {
> +		compatible = "arm,armv8-timer";
> +		interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
> +			     <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
> +			     <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
> +			     <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>;
> +	};
> +
> +	xin24m: xin24m {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <24000000>;
> +		clock-output-names = "xin24m";
> +	};
> +
> +	grf: syscon@...00000 {
> +		compatible = "rockchip,rk3328-grf", "syscon", "simple-mfd";
> +		reg = <0x0 0xff100000 0x0 0x1000>;
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +
> +		power: power-controller {
> +			compatible = "rockchip,rk3328-power-controller";
> +			#power-domain-cells = <1>;
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			status = "disabled";
> +
> +			pd_hevc@...328_PD_HEVC {
> +				reg = <RK3328_PD_HEVC>;
> +			};
> +			pd_video@...328_PD_VIDEO {
> +				reg = <RK3328_PD_VIDEO>;
> +			};
> +			pd_vpu@...328_PD_VPU {
> +				reg = <RK3328_PD_VPU>;
> +			};
> +		};
> +
> +		reboot-mode {
> +			compatible = "syscon-reboot-mode";
> +			offset = <0x5c8>;
> +			mode-normal = <BOOT_NORMAL>;
> +			mode-recovery = <BOOT_RECOVERY>;
> +			mode-bootloader = <BOOT_FASTBOOT>;
> +			mode-loader = <BOOT_BL_DOWNLOAD>;
> +		};
> +
> +	};
> +
> +	uart0: serial@...10000 {
> +		compatible = "rockchip,rk3328-uart", "snps,dw-apb-uart";
> +		reg = <0x0 0xff110000 0x0 0x100>;
> +		interrupts = <GIC_SPI 55 IRQ_TYPE_LEVEL_HIGH>;
> +		clocks = <&cru SCLK_UART0>, <&cru PCLK_UART0>;
> +		clock-names = "baudclk", "apb_pclk";
> +		reg-shift = <2>;
> +		reg-io-width = <4>;
> +		dmas = <&dmac 2>, <&dmac 3>;
> +		#dma-cells = <2>;
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&uart0_xfer &uart0_cts &uart0_rts>;
> +		status = "disabled";
> +	};
> +
> +	uart1: serial@...20000 {
> +		compatible = "rockchip,rk3328-uart", "snps,dw-apb-uart";
> +		reg = <0x0 0xff120000 0x0 0x100>;
> +		interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> +		clocks = <&cru SCLK_UART1>, <&cru PCLK_UART1>;
> +		clock-names = "sclk_uart", "pclk_uart";
> +		reg-shift = <2>;
> +		reg-io-width = <4>;
> +		dmas = <&dmac 4>, <&dmac 5>;
> +		#dma-cells = <2>;
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&uart1_xfer &uart1_cts &uart1_rts>;
> +		status = "disabled";
> +	};
> +
> +	uart2: serial@...30000 {
> +		compatible = "rockchip,rk3328-uart", "snps,dw-apb-uart";
> +		reg = <0x0 0xff130000 0x0 0x100>;
> +		interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
> +		clocks = <&cru SCLK_UART2>, <&cru PCLK_UART2>;
> +		clock-names = "baudclk", "apb_pclk";
> +		reg-shift = <2>;
> +		reg-io-width = <4>;
> +		dmas = <&dmac 6>, <&dmac 7>;
> +		#dma-cells = <2>;
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&uart2m1_xfer>;
> +		status = "disabled";
> +	};
> +
> +	i2c0: i2c@...50000 {
> +		compatible = "rockchip,rk3328-i2c", "rockchip,rk3399-i2c";
> +		reg = <0x0 0xff150000 0x0 0x1000>;
> +		interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&cru SCLK_I2C0>, <&cru PCLK_I2C0>;
> +		clock-names = "i2c", "pclk";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&i2c0_xfer>;
> +		status = "disabled";
> +	};
> +
> +	i2c1: i2c@...60000 {
> +		compatible = "rockchip,rk3328-i2c", "rockchip,rk3399-i2c";
> +		reg = <0x0 0xff160000 0x0 0x1000>;
> +		interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&cru SCLK_I2C1>, <&cru PCLK_I2C1>;
> +		clock-names = "i2c", "pclk";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&i2c1_xfer>;
> +		status = "disabled";
> +	};
> +
> +	i2c2: i2c@...70000 {
> +		compatible = "rockchip,rk3328-i2c", "rockchip,rk3399-i2c";
> +		reg = <0x0 0xff170000 0x0 0x1000>;
> +		interrupts = <GIC_SPI 38 IRQ_TYPE_LEVEL_HIGH>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&cru SCLK_I2C2>, <&cru PCLK_I2C2>;
> +		clock-names = "i2c", "pclk";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&i2c2_xfer>;
> +		status = "disabled";
> +	};
> +
> +	i2c3: i2c@...80000 {
> +		compatible = "rockchip,rk3328-i2c", "rockchip,rk3399-i2c";
> +		reg = <0x0 0xff180000 0x0 0x1000>;
> +		interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&cru SCLK_I2C3>, <&cru PCLK_I2C3>;
> +		clock-names = "i2c", "pclk";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&i2c3_xfer>;
> +		status = "disabled";
> +	};
> +
> +	spi0: spi@...90000 {
> +		compatible = "rockchip,rk3328-spi", "rockchip,rk3066-spi";
> +		reg = <0x0 0xff190000 0x0 0x1000>;
> +		interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>;
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		clocks = <&cru SCLK_SPI>, <&cru PCLK_SPI>;
> +		clock-names = "spiclk", "apb_pclk";
> +		dmas = <&dmac 8>, <&dmac 9>;
> +		#dma-cells = <2>;
> +		dma-names = "tx", "rx";
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&spi0m2_clk &spi0m2_tx &spi0m2_rx &spi0m2_cs0>;
> +		status = "disabled";
> +	};
> +
> +	wdt: watchdog@...a0000 {
> +		compatible = "snps,dw-wdt";
> +		reg = <0x0 0xff1a0000 0x0 0x100>;
> +		interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>;
> +		status = "disabled";
> +	};
> +
> +	amba {
> +		compatible = "simple-bus";
> +		#address-cells = <2>;
> +		#size-cells = <2>;
> +		ranges;
> +
> +		dmac: dmac@...f0000 {
> +			compatible = "arm,pl330", "arm,primecell";
> +			reg = <0x0 0xff1f0000 0x0 0x4000>;
> +			interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>,
> +				     <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&cru ACLK_DMAC>;
> +			clock-names = "apb_pclk";
> +			#dma-cells = <1>;
> +		};
> +	};
> +
> +	saradc: saradc@...80000 {
> +		compatible = "rockchip,rk3328-saradc", "rockchip,rk3399-saradc";
> +		reg = <0x0 0xff280000 0x0 0x100>;
> +		interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_HIGH>;
> +		#io-channel-cells = <1>;
> +		clocks = <&cru SCLK_SARADC>, <&cru PCLK_SARADC>;
> +		clock-names = "saradc", "apb_pclk";
> +		resets = <&cru SRST_SARADC_P>;
> +		reset-names = "saradc-apb";
> +		status = "disabled";
> +	};
> +
> +	cru: clock-controller@...40000 {
> +		compatible = "rockchip,rk3328-cru", "rockchip,cru", "syscon";
> +		reg = <0x0 0xff440000 0x0 0x1000>;
> +		rockchip,grf = <&grf>;
> +		#clock-cells = <1>;
> +		#reset-cells = <1>;
> +		assigned-clocks =
> +			<&cru DCLK_LCDC>, <&cru SCLK_PDM>,
> +			<&cru SCLK_RTC32K>, <&cru SCLK_UART0>,
> +			<&cru SCLK_UART1>, <&cru SCLK_UART2>,
> +			<&cru ACLK_BUS_PRE>, <&cru ACLK_PERI_PRE>,
> +			<&cru ACLK_VIO_PRE>, <&cru ACLK_RGA_PRE>,
> +			<&cru ACLK_VOP_PRE>, <&cru ACLK_RKVDEC_PRE>,
> +			<&cru ACLK_RKVENC>, <&cru ACLK_VPU_PRE>,
> +			<&cru SCLK_VDEC_CABAC>, <&cru SCLK_VDEC_CORE>,
> +			<&cru SCLK_VENC_CORE>, <&cru SCLK_VENC_DSP>,
> +			<&cru SCLK_SDIO>, <&cru SCLK_TSP>,
> +			<&cru SCLK_WIFI>, <&cru ARMCLK>,
> +			<&cru PLL_GPLL>, <&cru PLL_CPLL>,
> +			<&cru ACLK_BUS_PRE>, <&cru HCLK_BUS_PRE>,
> +			<&cru PCLK_BUS_PRE>, <&cru ACLK_PERI_PRE>,
> +			<&cru HCLK_PERI>, <&cru PCLK_PERI>,
> +			<&cru ACLK_VIO_PRE>, <&cru HCLK_VIO_PRE>,
> +			<&cru ACLK_RGA_PRE>, <&cru SCLK_RGA>,
> +			<&cru ACLK_VOP_PRE>, <&cru ACLK_RKVDEC_PRE>,
> +			<&cru ACLK_RKVENC>, <&cru ACLK_VPU_PRE>,
> +			<&cru SCLK_VDEC_CABAC>, <&cru SCLK_VDEC_CORE>,
> +			<&cru SCLK_VENC_CORE>, <&cru SCLK_VENC_DSP>,
> +			<&cru SCLK_EFUSE>, <&cru PCLK_DDR>,
> +			<&cru ACLK_GMAC>, <&cru PCLK_GMAC>,
> +			<&cru SCLK_RTC32K>, <&cru SCLK_USB3OTG_SUSPEND>;
> +		assigned-clock-parents =
> +			<&cru HDMIPHY>, <&cru PLL_APLL>,
> +			<&cru PLL_GPLL>, <&xin24m>,
> +			<&xin24m>, <&xin24m>;
> +		assigned-clock-rates =
> +			<0>, <61440000>,
> +			<0>, <24000000>,
> +			<24000000>, <24000000>,
> +			<15000000>, <15000000>,
> +			<100000000>, <100000000>,
> +			<100000000>, <100000000>,
> +			<50000000>, <100000000>,
> +			<100000000>, <100000000>,
> +			<50000000>, <50000000>,
> +			<50000000>, <50000000>,
> +			<24000000>, <600000000>,
> +			<491520000>, <1200000000>,
> +			<150000000>, <75000000>,
> +			<75000000>, <150000000>,
> +			<75000000>, <75000000>,
> +			<300000000>, <100000000>,
> +			<300000000>, <200000000>,
> +			<400000000>, <500000000>,
> +			<200000000>, <300000000>,
> +			<300000000>, <250000000>,
> +			<200000000>, <100000000>,
> +			<24000000>, <100000000>,
> +			<150000000>, <50000000>,
> +			<32768>, <32768>;
> +	};
> +
> +	gmac2io: eth@...40000 {
> +		compatible = "rockchip,rk3328-gmac";
> +		reg = <0x0 0xff540000 0x0 0x10000>;
> +		rockchip,grf = <&grf>;
> +		interrupts = <GIC_SPI 24 IRQ_TYPE_LEVEL_HIGH>;
> +		interrupt-names = "macirq";
> +		clocks = <&cru SCLK_MAC2IO>, <&cru SCLK_MAC2IO_RX>,
> +			 <&cru SCLK_MAC2IO_TX>, <&cru SCLK_MAC2IO_REF>,
> +			 <&cru SCLK_MAC2IO_REFOUT>, <&cru ACLK_MAC2IO>,
> +			 <&cru PCLK_MAC2IO>;
> +		clock-names = "stmmaceth", "mac_clk_rx",
> +			      "mac_clk_tx", "clk_mac_ref",
> +			      "clk_mac_refout", "aclk_mac",
> +			      "pclk_mac";
> +		resets = <&cru SRST_GMAC2IO_A>;
> +		reset-names = "stmmaceth";
> +		status = "disabled";
> +	};
> +
> +	gic: interrupt-controller@...11000 {
> +		compatible = "arm,gic-400";
> +		#interrupt-cells = <3>;
> +		#address-cells = <0>;
> +		interrupt-controller;
> +		reg = <0x0 0xff811000 0 0x1000>,
> +		      <0x0 0xff812000 0 0x2000>,
> +		      <0x0 0xff814000 0 0x2000>,
> +		      <0x0 0xff816000 0 0x2000>;
> +		interrupts = <GIC_PPI 9
> +		      (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>;
> +	};
> +
> +	pinctrl: pinctrl {
> +		compatible = "rockchip,rk3328-pinctrl";
> +		rockchip,grf = <&grf>;
> +		#address-cells = <2>;
> +		#size-cells = <2>;
> +		ranges;
> +
> +		gpio0: gpio0@...10000 {
> +			compatible = "rockchip,gpio-bank";
> +			reg = <0x0 0xff210000 0x0 0x100>;
> +			interrupts = <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&cru PCLK_GPIO0>;
> +
> +			gpio-controller;
> +			#gpio-cells = <2>;
> +
> +			interrupt-controller;
> +			#interrupt-cells = <2>;
> +		};
> +
> +		gpio1: gpio1@...20000 {
> +			compatible = "rockchip,gpio-bank";
> +			reg = <0x0 0xff220000 0x0 0x100>;
> +			interrupts = <GIC_SPI 52 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&cru PCLK_GPIO1>;
> +
> +			gpio-controller;
> +			#gpio-cells = <2>;
> +
> +			interrupt-controller;
> +			#interrupt-cells = <2>;
> +		};
> +
> +		gpio2: gpio2@...30000 {
> +			compatible = "rockchip,gpio-bank";
> +			reg = <0x0 0xff230000 0x0 0x100>;
> +			interrupts = <GIC_SPI 53 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&cru PCLK_GPIO2>;
> +
> +			gpio-controller;
> +			#gpio-cells = <2>;
> +
> +			interrupt-controller;
> +			#interrupt-cells = <2>;
> +		};
> +
> +		gpio3: gpio3@...40000 {
> +			compatible = "rockchip,gpio-bank";
> +			reg = <0x0 0xff240000 0x0 0x100>;
> +			interrupts = <GIC_SPI 54 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&cru PCLK_GPIO3>;
> +
> +			gpio-controller;
> +			#gpio-cells = <2>;
> +
> +			interrupt-controller;
> +			#interrupt-cells = <2>;
> +		};
> +
> +		pcfg_pull_up: pcfg-pull-up {
> +			bias-pull-up;
> +		};
> +
> +		pcfg_pull_down: pcfg-pull-down {
> +			bias-pull-down;
> +		};
> +
> +		pcfg_pull_none: pcfg-pull-none {
> +			bias-disable;
> +		};
> +
> +		pcfg_pull_none_2ma: pcfg-pull-none-2ma {
> +			bias-disable;
> +			drive-strength = <2>;
> +		};
> +
> +		pcfg_pull_up_2ma: pcfg-pull-up-2ma {
> +			bias-pull-up;
> +			drive-strength = <2>;
> +		};
> +
> +		pcfg_pull_up_4ma: pcfg-pull-up-4ma {
> +			bias-pull-up;
> +			drive-strength = <4>;
> +		};
> +
> +		pcfg_pull_none_4ma: pcfg-pull-none-4ma {
> +			bias-disable;
> +			drive-strength = <4>;
> +		};
> +
> +		pcfg_pull_down_4ma: pcfg-pull-down-4ma {
> +			bias-pull-down;
> +			drive-strength = <4>;
> +		};
> +
> +		pcfg_pull_none_8ma: pcfg-pull-none-8ma {
> +			bias-disable;
> +			drive-strength = <8>;
> +		};
> +
> +		pcfg_pull_up_8ma: pcfg-pull-up-8ma {
> +			bias-pull-up;
> +			drive-strength = <8>;
> +		};
> +
> +		pcfg_pull_none_12ma: pcfg-pull-none-12ma {
> +			bias-disable;
> +			drive-strength = <12>;
> +		};
> +
> +		pcfg_pull_up_12ma: pcfg-pull-up-12ma {
> +			bias-pull-up;
> +			drive-strength = <12>;
> +		};
> +
> +		pcfg_output_high: pcfg-output-high {
> +			output-high;
> +		};
> +
> +		pcfg_output_low: pcfg-output-low {
> +			output-low;
> +		};
> +
> +		pcfg_input_high: pcfg-input-high {
> +			bias-pull-up;
> +			input-enable;
> +		};
> +
> +		pcfg_input: pcfg-input {
> +			input-enable;
> +		};
> +
> +		i2c0 {
> +			i2c0_xfer: i2c0-xfer {
> +				rockchip,pins =
> +					<2 RK_PD0 RK_FUNC_1 &pcfg_pull_none>,

So as Heiko hinted already in that other email, can we just use "1" here
instead of the rather pointless RK_FUNC_1?
At the end of the day the DT should be the source of information, so
trying to hide this with some redundant definition is not helping, IMHO.

Cheers,
Andre.


> +					<2 RK_PD1 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		i2c1 {
> +			i2c1_xfer: i2c1-xfer {
> +				rockchip,pins =
> +					<2 RK_PA4 RK_FUNC_2 &pcfg_pull_none>,
> +					<2 RK_PA5 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		i2c2 {
> +			i2c2_xfer: i2c2-xfer {
> +				rockchip,pins =
> +					<2 RK_PB5 RK_FUNC_1 &pcfg_pull_none>,
> +					<2 RK_PB6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		i2c3 {
> +			i2c3_xfer: i2c3-xfer {
> +				rockchip,pins =
> +					<0 RK_PA5 RK_FUNC_2 &pcfg_pull_none>,
> +					<0 RK_PA6 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +			i2c3_gpio: i2c3-gpio {
> +				rockchip,pins =
> +					<0 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>,
> +					<0 RK_PA6 RK_FUNC_GPIO &pcfg_pull_none>;
> +			};
> +		};
> +
> +		hdmi_i2c {
> +			hdmii2c_xfer: hdmii2c-xfer {
> +				rockchip,pins =
> +					<0 RK_PA5 RK_FUNC_1 &pcfg_pull_none>,
> +					<0 RK_PA6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		tsadc {
> +			otp_gpio: otp-gpio {
> +				rockchip,pins =
> +					<2 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
> +			};
> +
> +			otp_out: otp-out {
> +				rockchip,pins =
> +					<2 RK_PB5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		uart0 {
> +			uart0_xfer: uart0-xfer {
> +				rockchip,pins =
> +					<1 RK_PB1 RK_FUNC_1 &pcfg_pull_up>,
> +					<1 RK_PB0 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			uart0_cts: uart0-cts {
> +				rockchip,pins =
> +					<1 RK_PB3 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			uart0_rts: uart0-rts {
> +				rockchip,pins =
> +					<1 RK_PB2 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			uart0_rts_gpio: uart0-rts-gpio {
> +				rockchip,pins =
> +					<1 RK_PB2 RK_FUNC_GPIO &pcfg_pull_none>;
> +			};
> +		};
> +
> +		uart1 {
> +			uart1_xfer: uart1-xfer {
> +				rockchip,pins =
> +					<3 RK_PA4 RK_FUNC_4 &pcfg_pull_up>,
> +					<3 RK_PA6 RK_FUNC_4 &pcfg_pull_none>;
> +			};
> +
> +			uart1_cts: uart1-cts {
> +				rockchip,pins =
> +					<3 RK_PA7 RK_FUNC_4 &pcfg_pull_none>;
> +			};
> +
> +			uart1_rts: uart1-rts {
> +				rockchip,pins =
> +					<3 RK_PA5 RK_FUNC_4 &pcfg_pull_none>;
> +			};
> +
> +			uart1_rts_gpio: uart1-rts-gpio {
> +				rockchip,pins =
> +					<3 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>;
> +			};
> +		};
> +
> +		uart2-0 {
> +			uart2m0_xfer: uart2m0-xfer {
> +				rockchip,pins =
> +					<1 RK_PA0 RK_FUNC_2 &pcfg_pull_up>,
> +					<1 RK_PA1 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		uart2-1 {
> +			uart2m1_xfer: uart2m1-xfer {
> +				rockchip,pins =
> +					<2 RK_PA0 RK_FUNC_1 &pcfg_pull_up>,
> +					<2 RK_PA1 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		spi0-0 {
> +			spi0m0_clk: spi0m0-clk {
> +				rockchip,pins =
> +					<2 RK_PB0 RK_FUNC_1 &pcfg_pull_up>;
> +			};
> +
> +			spi0m0_cs0: spi0m0-cs0 {
> +				rockchip,pins =
> +					<2 RK_PB3 RK_FUNC_1 &pcfg_pull_up>;
> +			};
> +
> +			spi0m0_tx: spi0m0-tx {
> +				rockchip,pins =
> +					<2 RK_PB1 RK_FUNC_1 &pcfg_pull_up>;
> +			};
> +
> +			spi0m0_rx: spi0m0-rx {
> +				rockchip,pins =
> +					<2 RK_PB2 RK_FUNC_1 &pcfg_pull_up>;
> +			};
> +
> +			spi0m0_cs1: spi0m0-cs1 {
> +				rockchip,pins =
> +					<2 RK_PB4 RK_FUNC_1 &pcfg_pull_up>;
> +			};
> +		};
> +
> +		spi0-1 {
> +			spi0m1_clk: spi0m1-clk {
> +				rockchip,pins =
> +					<3 RK_PC7 RK_FUNC_2 &pcfg_pull_up>;
> +			};
> +
> +			spi0m1_cs0: spi0m1-cs0 {
> +				rockchip,pins =
> +					<3 RK_PD2 RK_FUNC_2 &pcfg_pull_up>;
> +			};
> +
> +			spi0m1_tx: spi0m1-tx {
> +				rockchip,pins =
> +					<3 RK_PD1 RK_FUNC_2 &pcfg_pull_up>;
> +			};
> +
> +			spi0m1_rx: spi0m1-rx {
> +				rockchip,pins =
> +					<3 RK_PD0 RK_FUNC_2 &pcfg_pull_up>;
> +			};
> +
> +			spi0m1_cs1: spi0m1-cs1 {
> +				rockchip,pins =
> +					<3 RK_PD3 RK_FUNC_2 &pcfg_pull_up>;
> +			};
> +		};
> +
> +		spi0-2 {
> +			spi0m2_clk: spi0m2-clk {
> +				rockchip,pins =
> +					<3 RK_PA0 RK_FUNC_4 &pcfg_pull_up>;
> +			};
> +
> +			spi0m2_cs0: spi0m2-cs0 {
> +				rockchip,pins =
> +					<3 RK_PB0 RK_FUNC_3 &pcfg_pull_up>;
> +			};
> +
> +			spi0m2_tx: spi0m2-tx {
> +				rockchip,pins =
> +					<3 RK_PA1 RK_FUNC_4 &pcfg_pull_up>;
> +			};
> +
> +			spi0m2_rx: spi0m2-rx {
> +				rockchip,pins =
> +					<3 RK_PA2 RK_FUNC_4 &pcfg_pull_up>;
> +			};
> +		};
> +
> +		i2s1 {
> +			i2s1_mclk: i2s1-mclk {
> +				rockchip,pins =
> +					<2 RK_PB7 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sclk: i2s1-sclk {
> +				rockchip,pins =
> +					<2 RK_PC2 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_lrckrx: i2s1-lrckrx {
> +				rockchip,pins =
> +					<2 RK_PC0 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_lrcktx: i2s1-lrcktx {
> +				rockchip,pins =
> +					<2 RK_PC1 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sdi: i2s1-sdi {
> +				rockchip,pins =
> +					<2 RK_PC3 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sdo: i2s1-sdo {
> +				rockchip,pins =
> +					<2 RK_PC7 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sdio1: i2s1-sdio1 {
> +				rockchip,pins =
> +					<2 RK_PC4 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sdio2: i2s1-sdio2 {
> +				rockchip,pins =
> +					<2 RK_PC5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sdio3: i2s1-sdio3 {
> +				rockchip,pins =
> +					<2 RK_PC6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s1_sleep: i2s1-sleep {
> +				rockchip,pins =
> +					<2 RK_PB7 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC0 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC1 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC2 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC3 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC4 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC5 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC6 RK_FUNC_GPIO &pcfg_input_high>,
> +					<2 RK_PC7 RK_FUNC_GPIO &pcfg_input_high>;
> +			};
> +		};
> +
> +		i2s2-0 {
> +			i2s2m0_mclk: i2s2m0-mclk {
> +				rockchip,pins =
> +					<1 RK_PC5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_sclk: i2s2m0-sclk {
> +				rockchip,pins =
> +					<1 RK_PC6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_lrckrx: i2s2m0-lrckrx {
> +				rockchip,pins =
> +					<1 RK_PD2 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_lrcktx: i2s2m0-lrcktx {
> +				rockchip,pins =
> +					<1 RK_PC7 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_sdi: i2s2m0-sdi {
> +				rockchip,pins =
> +					<1 RK_PD0 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_sdo: i2s2m0-sdo {
> +				rockchip,pins =
> +					<1 RK_PD1 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m0_sleep: i2s2m0-sleep {
> +				rockchip,pins =
> +					<1 RK_PC5 RK_FUNC_GPIO &pcfg_input_high>,
> +					<1 RK_PC6 RK_FUNC_GPIO &pcfg_input_high>,
> +					<1 RK_PD2 RK_FUNC_GPIO &pcfg_input_high>,
> +					<1 RK_PC7 RK_FUNC_GPIO &pcfg_input_high>,
> +					<1 RK_PD0 RK_FUNC_GPIO &pcfg_input_high>,
> +					<1 RK_PD1 RK_FUNC_GPIO &pcfg_input_high>;
> +			};
> +		};
> +
> +		i2s2-1 {
> +			i2s2m1_mclk: i2s2m1-mclk {
> +				rockchip,pins =
> +					<1 RK_PC5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_sclk: i2s2m1-sclk {
> +				rockchip,pins =
> +					<3 RK_PA0 RK_FUNC_6 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_lrckrx: i2sm1-lrckrx {
> +				rockchip,pins =
> +					<3 RK_PB0 RK_FUNC_6 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_lrcktx: i2s2m1-lrcktx {
> +				rockchip,pins =
> +					<3 RK_PB0 RK_FUNC_4 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_sdi: i2s2m1-sdi {
> +				rockchip,pins =
> +					<3 RK_PA2 RK_FUNC_6 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_sdo: i2s2m1-sdo {
> +				rockchip,pins =
> +					<3 RK_PA1 RK_FUNC_6 &pcfg_pull_none>;
> +			};
> +
> +			i2s2m1_sleep: i2s2m1-sleep {
> +				rockchip,pins =
> +					<1 RK_PC5 RK_FUNC_GPIO &pcfg_input_high>,
> +					<3 RK_PA0 RK_FUNC_GPIO &pcfg_input_high>,
> +					<3 RK_PB0 RK_FUNC_GPIO &pcfg_input_high>,
> +					<3 RK_PA2 RK_FUNC_GPIO &pcfg_input_high>,
> +					<3 RK_PA1 RK_FUNC_GPIO &pcfg_input_high>;
> +			};
> +		};
> +
> +		spdif-0 {
> +			spdifm0_tx: spdifm0-tx {
> +				rockchip,pins =
> +					<0 RK_PD3 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		spdif-1 {
> +			spdifm1_tx: spdifm1-tx {
> +				rockchip,pins =
> +					<2 RK_PC1 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		spdif-2 {
> +			spdifm2_tx: spdifm2-tx {
> +				rockchip,pins =
> +					<0 RK_PA2 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		sdmmc0-0 {
> +			sdmmc0m0_pwren: sdmmc0m0-pwren {
> +				rockchip,pins =
> +					<2 RK_PA7 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0m0_gpio: sdmmc0m0-gpio {
> +				rockchip,pins =
> +					<2 RK_PA7 RK_FUNC_GPIO &pcfg_pull_up_4ma>;
> +			};
> +		};
> +
> +		sdmmc0-1 {
> +			sdmmc0m1_pwren: sdmmc0m1-pwren {
> +				rockchip,pins =
> +					<0 RK_PD6 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0m1_gpio: sdmmc0m1-gpio {
> +				rockchip,pins =
> +					<0 RK_PD6 RK_FUNC_GPIO &pcfg_pull_up_4ma>;
> +			};
> +		};
> +
> +		sdmmc0 {
> +			sdmmc0_clk: sdmmc0-clk {
> +				rockchip,pins =
> +					<1 RK_PA6 RK_FUNC_1 &pcfg_pull_none_4ma>;
> +			};
> +
> +			sdmmc0_cmd: sdmmc0-cmd {
> +				rockchip,pins =
> +					<1 RK_PA4 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0_dectn: sdmmc0-dectn {
> +				rockchip,pins =
> +					<1 RK_PA5 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0_wrprt: sdmmc0-wrprt {
> +				rockchip,pins =
> +					<1 RK_PA7 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0_bus1: sdmmc0-bus1 {
> +				rockchip,pins =
> +					<1 RK_PA0 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0_bus4: sdmmc0-bus4 {
> +				rockchip,pins =
> +					<1 RK_PA0 RK_FUNC_1 &pcfg_pull_up_4ma>,
> +					<1 RK_PA1 RK_FUNC_1 &pcfg_pull_up_4ma>,
> +					<1 RK_PA2 RK_FUNC_1 &pcfg_pull_up_4ma>,
> +					<1 RK_PA3 RK_FUNC_1 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0_gpio: sdmmc0-gpio {
> +				rockchip,pins =
> +					<1 RK_PA6 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA4 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA5 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA7 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA3 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA2 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_up_4ma>;
> +			};
> +		};
> +
> +		sdmmc0ext {
> +			sdmmc0ext_clk: sdmmc0ext-clk {
> +				rockchip,pins =
> +					<3 RK_PA2 RK_FUNC_3 &pcfg_pull_none_4ma>;
> +			};
> +
> +			sdmmc0ext_cmd: sdmmc0ext-cmd {
> +				rockchip,pins =
> +					<3 RK_PA0 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0ext_wrprt: sdmmc0ext-wrprt {
> +				rockchip,pins =
> +					<3 RK_PA3 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0ext_dectn: sdmmc0ext-dectn {
> +				rockchip,pins =
> +					<3 RK_PA1 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0ext_bus1: sdmmc0ext-bus1 {
> +				rockchip,pins =
> +					<3 RK_PA4 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0ext_bus4: sdmmc0ext-bus4 {
> +				rockchip,pins =
> +					<3 RK_PA4 RK_FUNC_3 &pcfg_pull_up_4ma>,
> +					<3 RK_PA5 RK_FUNC_3 &pcfg_pull_up_4ma>,
> +					<3 RK_PA6 RK_FUNC_3 &pcfg_pull_up_4ma>,
> +					<3 RK_PA7 RK_FUNC_3 &pcfg_pull_up_4ma>;
> +			};
> +
> +			sdmmc0ext_gpio: sdmmc0ext-gpio {
> +				rockchip,pins =
> +					<3 RK_PA0 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA1 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA2 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA3 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA4 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA5 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA6 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<3 RK_PA7 RK_FUNC_GPIO &pcfg_pull_up_4ma>;
> +			};
> +		};
> +
> +		sdmmc1 {
> +			sdmmc1_clk: sdmmc1-clk {
> +				rockchip,pins =
> +					<1 RK_PB4 RK_FUNC_1 &pcfg_pull_none_8ma>;
> +			};
> +
> +			sdmmc1_cmd: sdmmc1-cmd {
> +				rockchip,pins =
> +					<1 RK_PB5 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_pwren: sdmmc1-pwren {
> +				rockchip,pins =
> +					<1 RK_PC2 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_wrprt: sdmmc1-wrprt {
> +				rockchip,pins =
> +					<1 RK_PC4 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_dectn: sdmmc1-dectn {
> +				rockchip,pins =
> +					<1 RK_PC3 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_bus1: sdmmc1-bus1 {
> +				rockchip,pins =
> +					<1 RK_PB6 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_bus4: sdmmc1-bus4 {
> +				rockchip,pins =
> +					<1 RK_PB4 RK_FUNC_1 &pcfg_pull_up_8ma>,
> +					<1 RK_PB5 RK_FUNC_1 &pcfg_pull_up_8ma>,
> +					<1 RK_PC0 RK_FUNC_1 &pcfg_pull_up_8ma>,
> +					<1 RK_PC1 RK_FUNC_1 &pcfg_pull_up_8ma>;
> +			};
> +
> +			sdmmc1_gpio: sdmmc1-gpio {
> +				rockchip,pins =
> +					<1 RK_PB4 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PB5 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PB6 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PB7 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PC0 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PC1 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PC2 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PC3 RK_FUNC_GPIO &pcfg_pull_up_4ma>,
> +					<1 RK_PC4 RK_FUNC_GPIO &pcfg_pull_up_4ma>;
> +			};
> +		};
> +
> +		emmc {
> +			emmc_clk: emmc-clk {
> +				rockchip,pins =
> +					<3 RK_PC5 RK_FUNC_2 &pcfg_pull_none_12ma>;
> +			};
> +
> +			emmc_cmd: emmc-cmd {
> +				rockchip,pins =
> +					<3 RK_PC3 RK_FUNC_2 &pcfg_pull_up_12ma>;
> +			};
> +
> +			emmc_pwren: emmc-pwren {
> +				rockchip,pins =
> +					<3 RK_PC6 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +
> +			emmc_rstnout: emmc-rstnout {
> +				rockchip,pins =
> +					<3 RK_PC4 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +
> +			emmc_bus1: emmc-bus1 {
> +				rockchip,pins =
> +					<0 RK_PA7 RK_FUNC_2 &pcfg_pull_up_12ma>;
> +			};
> +
> +			emmc_bus4: emmc-bus4 {
> +				rockchip,pins =
> +					<0 RK_PA7 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD4 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD5 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD6 RK_FUNC_2 &pcfg_pull_up_12ma>;
> +			};
> +
> +			emmc_bus8: emmc-bus8 {
> +				rockchip,pins =
> +					<0 RK_PA7 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD4 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD5 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD6 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<2 RK_PD7 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<3 RK_PC0 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<3 RK_PC1 RK_FUNC_2 &pcfg_pull_up_12ma>,
> +					<3 RK_PC2 RK_FUNC_2 &pcfg_pull_up_12ma>;
> +			};
> +		};
> +
> +		pwm0 {
> +			pwm0_pin: pwm0-pin {
> +				rockchip,pins =
> +					<2 RK_PA4 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		pwm1 {
> +			pwm1_pin: pwm1-pin {
> +				rockchip,pins =
> +					<2 RK_PA5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		pwm2 {
> +			pwm2_pin: pwm2-pin {
> +				rockchip,pins =
> +					<2 RK_PA6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		pwmir {
> +			pwmir_pin: pwmir-pin {
> +				rockchip,pins =
> +					<2 RK_PA2 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		gmac-1 {
> +			rgmiim1_pins: rgmiim1-pins {
> +				rockchip,pins =
> +					/* mac_txclk */
> +					<1 RK_PB4 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_rxclk */
> +					<1 RK_PB5 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_mdio */
> +					<1 RK_PC3 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_txen */
> +					<1 RK_PD1 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_clk */
> +					<1 RK_PC5 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxdv */
> +					<1 RK_PC6 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_mdc */
> +					<1 RK_PC7 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxd1 */
> +					<1 RK_PB2 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxd0 */
> +					<1 RK_PB3 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_txd1 */
> +					<1 RK_PB0 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_txd0 */
> +					<1 RK_PB1 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_rxd3 */
> +					<1 RK_PB6 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxd2 */
> +					<1 RK_PB7 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_txd3 */
> +					<1 RK_PC0 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_txd2 */
> +					<1 RK_PC1 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +
> +					/* mac_txclk */
> +					<0 RK_PB0 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txen */
> +					<0 RK_PB4 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_clk */
> +					<0 RK_PD0 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd1 */
> +					<0 RK_PC0 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd0 */
> +					<0 RK_PC1 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd3 */
> +					<0 RK_PC7 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd2 */
> +					<0 RK_PC6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			rmiim1_pins: rmiim1-pins {
> +				rockchip,pins =
> +					/* mac_mdio */
> +					<1 RK_PC3 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_txen */
> +					<1 RK_PD1 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_clk */
> +					<1 RK_PC5 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxer */
> +					<1 RK_PD0 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxdv */
> +					<1 RK_PC6 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_mdc */
> +					<1 RK_PC7 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxd1 */
> +					<1 RK_PB2 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_rxd0 */
> +					<1 RK_PB3 RK_FUNC_2 &pcfg_pull_none_2ma>,
> +					/* mac_txd1 */
> +					<1 RK_PB0 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +					/* mac_txd0 */
> +					<1 RK_PB1 RK_FUNC_2 &pcfg_pull_none_12ma>,
> +
> +					/* mac_mdio */
> +					<0 RK_PB3 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txen */
> +					<0 RK_PB4 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_clk */
> +					<0 RK_PD0 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_mdc */
> +					<0 RK_PC3 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd1 */
> +					<0 RK_PC0 RK_FUNC_1 &pcfg_pull_none>,
> +					/* mac_txd0 */
> +					<0 RK_PC1 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		gmac2phy {
> +			fephyled_speed100: fephyled-speed100 {
> +				rockchip,pins =
> +					<0 RK_PD7 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_speed10: fephyled-speed10 {
> +				rockchip,pins =
> +					<0 RK_PD6 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_duplex: fephyled-duplex {
> +				rockchip,pins =
> +					<0 RK_PD6 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_rxm0: fephyled-rxm0 {
> +				rockchip,pins =
> +					<0 RK_PD5 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_txm0: fephyled-txm0 {
> +				rockchip,pins =
> +					<0 RK_PD5 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_linkm0: fephyled-linkm0 {
> +				rockchip,pins =
> +					<0 RK_PD4 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_rxm1: fephyled-rxm1 {
> +				rockchip,pins =
> +					<2 RK_PD1 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_txm1: fephyled-txm1 {
> +				rockchip,pins =
> +					<2 RK_PD1 RK_FUNC_3 &pcfg_pull_none>;
> +			};
> +
> +			fephyled_linkm1: fephyled-linkm1 {
> +				rockchip,pins =
> +					<2 RK_PD0 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		tsadc_pin {
> +			tsadc_int: tsadc-int {
> +				rockchip,pins =
> +					<2 RK_PB5 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +			tsadc_gpio: tsadc-gpio {
> +				rockchip,pins =
> +					<2 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
> +			};
> +		};
> +
> +		hdmi_pin {
> +			hdmi_cec: hdmi-cec {
> +				rockchip,pins =
> +					<0 RK_PA3 RK_FUNC_1 &pcfg_pull_none>;
> +			};
> +
> +			hdmi_hpd: hdmi-hpd {
> +				rockchip,pins =
> +					<0 RK_PA4 RK_FUNC_1 &pcfg_pull_down>;
> +			};
> +		};
> +
> +		cif-0 {
> +			dvp_d2d9_m0:dvp-d2d9-m0 {
> +				rockchip,pins =
> +					/* cif_d0 */
> +					<3 RK_PA4 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d1 */
> +					<3 RK_PA5 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d2 */
> +					<3 RK_PA6 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d3 */
> +					<3 RK_PA7 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d4 */
> +					<3 RK_PB0 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d5m0 */
> +					<3 RK_PB1 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d6m0 */
> +					<3 RK_PB2 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d7m0 */
> +					<3 RK_PB3 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_href */
> +					<3 RK_PA1 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_vsync */
> +					<3 RK_PA0 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_clkoutm0 */
> +					<3 RK_PA3 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_clkin */
> +					<3 RK_PA2 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +
> +		cif-1 {
> +			dvp_d2d9_m1:dvp-d2d9-m1 {
> +				rockchip,pins =
> +					/* cif_d0 */
> +					<3 RK_PA4 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d1 */
> +					<3 RK_PA5 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d2 */
> +					<3 RK_PA6 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d3 */
> +					<3 RK_PA7 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d4 */
> +					<3 RK_PB0 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_d5m1 */
> +					<2 RK_PC0 RK_FUNC_4 &pcfg_pull_none>,
> +					/* cif_d6m1 */
> +					<2 RK_PC1 RK_FUNC_4 &pcfg_pull_none>,
> +					/* cif_d7m1 */
> +					<2 RK_PC2 RK_FUNC_4 &pcfg_pull_none>,
> +					/* cif_href */
> +					<3 RK_PA1 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_vsync */
> +					<3 RK_PA0 RK_FUNC_2 &pcfg_pull_none>,
> +					/* cif_clkoutm1 */
> +					<2 RK_PB7 RK_FUNC_4 &pcfg_pull_none>,
> +					/* cif_clkin */
> +					<3 RK_PA2 RK_FUNC_2 &pcfg_pull_none>;
> +			};
> +		};
> +	};
> +};
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ