Re: [PATCH v2 13/13] ARM: dts: ipq8074: Enable few peripherals for hk01 board

[Abhishek Sahu <absahu@codeaurora.org>]

On 2018-03-16 15:08, Sricharan R wrote:
> Signed-off-by: Sricharan R <sricharan@codeaurora.org>
> ---
>  arch/arm64/boot/dts/qcom/ipq8074-hk01.dts | 103 
> ++++++++++++++++++++++++++++++
>  1 file changed, 103 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/ipq8074-hk01.dts
> b/arch/arm64/boot/dts/qcom/ipq8074-hk01.dts
> index 6a838b5..81dff867 100644
> --- a/arch/arm64/boot/dts/qcom/ipq8074-hk01.dts
> +++ b/arch/arm64/boot/dts/qcom/ipq8074-hk01.dts
> @@ -21,6 +21,7 @@
> 
>  	aliases {
>  		serial0 = &blsp1_uart5;
> +		serial1 = &serial_blsp2;
>  	};
> 
>  	chosen {
> @@ -41,6 +42,47 @@
>  					bias-disable;
>  				};
>  			};
> +
> +			 i2c_0_pins: i2c_0_pinmux {
> +				  mux {
> +					   pins = "gpio42", "gpio43";
> +					   function = "blsp1_i2c";
> +					   drive-strength = <8>;
> +					   bias-disable;
> +				  };
> +			 };
> +
> +			 spi_0_pins: spi_0_pins {
> +				  mux {
> +					   pins = "gpio38", "gpio39", "gpio40", "gpio41";
> +					   function = "blsp0_spi";
> +					   drive-strength = <8>;
> +					   bias-disable;
> +				  };
> +			 };
> +
> +			 hsuart_pins: hsuart_pins {
> +				  mux {
> +					   pins = "gpio46", "gpio47", "gpio48", "gpio49";
> +					   function = "blsp2_uart";
> +					   drive-strength = <8>;
> +					   bias-disable;
> +				  };
> +			 };
> +
> +			 qpic_pins: qpic_pins {
> +				mux {
> +					   pins = "gpio1", "gpio3", "gpio4",
> +						  "gpio5", "gpio6", "gpio7",
> +						  "gpio8", "gpio10", "gpio11",
> +						  "gpio12", "gpio13", "gpio14",
> +						  "gpio15", "gpio16", "gpio17";
> +					   function = "qpic";
> +					   drive-strength = <8>;
> +					   bias-disable;
> +				  };
> +			};
> +
>  		};
> 
>  		serial@78b3000 {
> @@ -48,5 +90,66 @@
>  			pinctrl-names = "default";
>  			status = "ok";
>  		};
> +
> +		spi@78b5000 {
> +			 pinctrl-0 = <&spi_0_pins>;
> +			 pinctrl-names = "default";
> +			 status = "ok";
> +
> +			 m25p80@0 {
> +				  #address-cells = <1>;
> +				  #size-cells = <1>;
> +				  compatible = "n25q128a11", "jedec,spi-nor";
> +				  reg = <0>;
> +				  spi-max-frequency = <50000000>;
> +			 };
> +		};
> +
> +		serial@78B1000 {

  s/78B1000/78b1000

  With that,
  Reviewed-by: Abhishek Sahu <absahu@codeaurora.org>

> +			 pinctrl-0 = <&hsuart_pins>;
> +			 pinctrl-names = "default";
> +			 status = "ok";
> +		};
> +
> +		i2c_0@78b6000 {
> +			 pinctrl-0 = <&i2c_0_pins>;
> +			 pinctrl-names = "default";
> +			 status = "ok";
> +		};
> +
> +		dma@7984000 {
> +			 status = "ok";
> +		};
> +
> +		nand@79b0000 {
> +			pinctrl-0 = <&qpic_pins>;
> +			pinctrl-names = "default";
> +			status = "ok";
> +
> +			nand@0 {
> +				reg = <0>;
> +				nand-ecc-strength = <4>;
> +				nand-ecc-step-size = <512>;
> +				nand-bus-width = <8>;
> +			};
> +		};
> +
> +		phy@86000 {
> +			status = "ok";
> +		};
> +
> +		phy@8e000 {
> +			status = "ok";
> +		};
> +
> +		pci@20000000 {
> +			status = "ok";
> +			perst-gpio = <&tlmm 58 0x1>;
> +		};
> +
> +		pci@10000000 {
> +			status = "ok";
> +			perst-gpio = <&tlmm 61 0x1>;
> +		};
>  	};
>  };