Commit 9f5cdeb7 authored by Douglas Anderson's avatar Douglas Anderson Committed by Bjorn Andersson
Browse files

arm64: dts: qcom: sdm845: Fix cheza qspi pin config



Cheza's SPI flash hookups (qspi) are exactly the same as trogdor's.
Apply the same solution that's described in the patch ("arm64: dts:
qcom: sc7180: Fix trogdor qspi pin config")

Signed-off-by: default avatarDouglas Anderson <dianders@chromium.org>
Acked-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20230323102605.14.I82951106ab8170f973a4c1c7d9b034655bbe2f60@changeid
parent 5f89df31
Loading
Loading
Loading
Loading
+27 −7
Original line number Diff line number Diff line
@@ -317,8 +317,9 @@ venus_mem: memory@96000000 {

&qspi {
	status = "okay";
	pinctrl-names = "default";
	pinctrl-0 = <&qspi_clk &qspi_cs0 &qspi_data01>;
	pinctrl-names = "default", "sleep";
	pinctrl-0 = <&qspi_clk>, <&qspi_cs0>, <&qspi_data0>, <&qspi_data1>;
	pinctrl-1 = <&qspi_sleep>;

	flash@0 {
		compatible = "jedec,spi-nor";
@@ -993,16 +994,19 @@ &wifi {
/* PINCTRL - additions to nodes defined in sdm845.dtsi */

&qspi_cs0 {
	bias-disable;
	bias-disable;		/* External pullup */
};

&qspi_clk {
	bias-disable;
	bias-disable;		/* Rely on Cr50 internal pulldown */
};

&qspi_data01 {
	/* High-Z when no transfers; nice to park the lines */
	bias-pull-up;
&qspi_data0 {
	bias-disable;		/* Rely on Cr50 internal pulldown */
};

&qspi_data1 {
	bias-pull-down;
};

&qup_i2c3_default {
@@ -1231,6 +1235,22 @@ pen_rst_l: pen-rst-l-state {
		output-high;
	};

	qspi_sleep: qspi-sleep-state {
		pins = "gpio90", "gpio91", "gpio92", "gpio95";

		/*
		 * When we're not actively transferring we want pins as GPIOs
		 * with output disabled so that the quad SPI IP block stops
		 * driving them. We rely on the normal pulls configured in
		 * the active state and don't redefine them here. Also note
		 * that we don't need the reverse (output-enable) in the
		 * normal mode since the "output-enable" only matters for
		 * GPIO function.
		 */
		function = "gpio";
		output-disable;
	};

	sdc2_clk: sdc2-clk-state {
		pins = "sdc2_clk";
		bias-disable;
+7 −2
Original line number Diff line number Diff line
@@ -2767,8 +2767,13 @@ qspi_cs1: qspi-cs1-state {
				function = "qspi_cs";
			};

			qspi_data01: qspi-data01-state {
				pins = "gpio91", "gpio92";
			qspi_data0: qspi-data0-state {
				pins = "gpio91";
				function = "qspi_data";
			};

			qspi_data1: qspi-data1-state {
				pins = "gpio92";
				function = "qspi_data";
			};