On Thu, 2016-09-22 at 18:51:29 +0530, Jagan Teki wrote:
> From: Jagan Teki <jt...@openedev.com>
> 
> Added basic dts support for MicroZed board.
> 
> - UART
> - SDHCI
> - Ethernet
> 
> Cc: Soren Brinkmann <soren.brinkm...@xilinx.com>
> Cc: Michal Simek <michal.si...@xilinx.com>
> Signed-off-by: Jagan Teki <jt...@openedev.com>
> ---
> Changes for v2:
>       - Add SDHCI
>       - Add Ethernet
> 
>  arch/arm/boot/dts/Makefile          |  1 +
>  arch/arm/boot/dts/zynq-microzed.dts | 95 
> +++++++++++++++++++++++++++++++++++++
>  2 files changed, 96 insertions(+)
>  create mode 100644 arch/arm/boot/dts/zynq-microzed.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index faacd52..4d7b858 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -862,6 +862,7 @@ dtb-$(CONFIG_ARCH_VT8500) += \
>       wm8750-apc8750.dtb \
>       wm8850-w70v2.dtb
>  dtb-$(CONFIG_ARCH_ZYNQ) += \
> +     zynq-microzed.dtb \
>       zynq-parallella.dtb \
>       zynq-zc702.dtb \
>       zynq-zc706.dtb \
> diff --git a/arch/arm/boot/dts/zynq-microzed.dts 
> b/arch/arm/boot/dts/zynq-microzed.dts
> new file mode 100644
> index 0000000..9e64496
> --- /dev/null
> +++ b/arch/arm/boot/dts/zynq-microzed.dts
> @@ -0,0 +1,95 @@
> +/*
> + * Copyright (C) 2015 Jagan Teki <jt...@openedev.com>
> + *
> + * This software is licensed under the terms of the GNU General Public
> + * License version 2, as published by the Free Software Foundation, and
> + * may be copied, distributed, and modified under those terms.
> + *
> + * This program 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.
> + */
> +/dts-v1/;
> +/include/ "zynq-7000.dtsi"
> +
> +/ {
> +     model = "Zynq MicroZED Development Board";
> +     compatible = "xlnx,zynq-microzed", "xlnx,zynq-7000";
> +
> +     aliases {
> +             ethernet0 = &gem0;
> +             serial0 = &uart1;
> +     };
> +
> +     memory {
> +             device_type = "memory";
> +             reg = <0x0 0x40000000>;
> +     };
> +
> +     chosen {
> +             bootargs = "earlycon";
> +             stdout-path = "serial0:115200n8";
> +     };
> +
> +     usb_phy0: phy0 {
> +             compatible = "usb-nop-xceiv";
> +             #phy-cells = <0>;
> +     };
> +};
> +
> +&clkc {
> +     ps-clk-frequency = <33333333>;
> +};
> +
> +&gem0 {
> +     status = "okay";
> +     phy-mode = "rgmii-id";
> +     phy-handle = <&ethernet_phy>;
> +
> +     ethernet_phy: ethernet-phy@0 {
> +             reg = <0>;
> +     };
> +};
> +
> +&sdhci0 {
> +     status = "okay";
> +};
> +
> +&uart1 {
> +     status = "okay";
> +};
> +
> +&usb0 {
> +     status = "okay";
> +     dr_mode = "host";
> +     usb-phy = <&usb_phy0>;
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_usb0_default>;
> +};
> +
> +&pinctrl0 {
> +     pinctrl_usb0_default: usb0-default {
> +             mux {
> +                     groups = "usb0_0_grp";
> +                     function = "usb0";
> +             };
> +
> +             conf {
> +                     groups = "usb0_0_grp";
> +                     slew-rate = <0>;
> +                     io-standard = <1>;
> +             };
> +
> +             conf-rx {
> +                     pins = "MIO29", "MIO31", "MIO36";
> +                     bias-high-impedance;
> +             };
> +
> +             conf-tx {
> +                     pins = "MIO28", "MIO30", "MIO32", "MIO33", "MIO34",
> +                            "MIO35", "MIO37", "MIO38", "MIO39";
> +                     bias-disable;
> +             };
> +     };
> +};

I guess it's not strictly required, but shouldn't there be pinctrl
descriptions for all devices?

        Sören

Reply via email to