linux-kernel - Re: [PATCH 2/2] arm64: dts: ti: k3-j721e-common-proc-board: add DP to j7 evm

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: <00538a2f-fcbf-321a-1ce0-645913836e73@ideasonboard.com>
Date:   Fri, 11 Mar 2022 10:44:52 +0200
From:   Tomi Valkeinen <tomi.valkeinen@...asonboard.com>
To:     Rahul T R <r-ravikumar@...com>, nm@...com
Cc:     vigneshr@...com, kristo@...nel.org, robh+dt@...nel.org,
        krzysztof.kozlowski@...onical.com,
        linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, laurent.pinchart@...asonboard.com
Subject: Re: [PATCH 2/2] arm64: dts: ti: k3-j721e-common-proc-board: add DP to
 j7 evm

On 22/02/2022 18:32, Rahul T R wrote:
> From: Tomi Valkeinen <tomi.valkeinen@...com>
> 
> Add the endpoint nodes to describe connection from
> DSS => MHDP => DisplayPort connector.
> Also add the required pinmux nodes for hotplug.
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@...com>
> Signed-off-by: Rahul T R <r-ravikumar@...com>
> ---
>   .../dts/ti/k3-j721e-common-proc-board.dts     | 66 +++++++++++++++++--
>   1 file changed, 62 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> index 2d7596911b27..fe20c193f299 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> @@ -148,6 +148,28 @@
>   		pinctrl-0 = <&main_mcan2_gpio_pins_default>;
>   		standby-gpios = <&main_gpio0 127 GPIO_ACTIVE_HIGH>;
>   	};
> +
> +	dp_pwr_3v3: fixedregulator-dp-prw {
> +		compatible = "regulator-fixed";
> +		regulator-name = "dp-pwr";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		gpio = <&exp4 0 0>;	/* P0 - DP0_PWR_SW_EN */
> +		enable-active-high;
> +	};
> +
> +	dp0: connector {
> +		compatible = "dp-connector";
> +		label = "DP0";
> +		type = "full-size";
> +		dp-pwr-supply = <&dp_pwr_3v3>;
> +
> +		port {
> +			dp_connector_in: endpoint {
> +				remote-endpoint = <&dp0_out>;
> +			};
> +		};
> +	};
>   };
>   
>   &main_pmx0 {
> @@ -190,6 +212,12 @@
>   		>;
>   	};
>   
> +	dp0_pins_default: dp0-pins-default {
> +		pinctrl-single,pins = <
> +			J721E_IOPAD(0x1c4, PIN_INPUT, 5) /* SPI0_CS1.DP0_HPD */
> +		>;
> +	};
> +
>   	main_i2c1_exp4_pins_default: main-i2c1-exp4-pins-default {
>   		pinctrl-single,pins = <
>   			J721E_IOPAD(0x230, PIN_INPUT, 7) /* (U2) ECAP0_IN_APWM_OUT.GPIO1_11 */
> @@ -660,6 +688,40 @@
>   				 <&k3_clks 152 18>;	/* PLL23_HSDIV0 */
>   };
>   
> +&dss_ports {
> +	port@0 {
> +		reg = <0>;
> +
> +		dpi0_out: endpoint {
> +			remote-endpoint = <&dp0_in>;
> +		};
> +	};
> +};
> +
> +&mhdp {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&dp0_pins_default>;
> +};
> +
> +&dp0_ports {
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +
> +	port@0 {
> +		reg = <0>;
> +		dp0_in: endpoint {
> +			remote-endpoint = <&dpi0_out>;
> +		};
> +	};
> +
> +	port@4 {
> +		reg = <4>;
> +		dp0_out: endpoint {
> +			remote-endpoint = <&dp_connector_in>;
> +		};
> +	};
> +};
> +
>   &mcasp0 {
>   	status = "disabled";
>   };
> @@ -845,10 +907,6 @@
>   	status = "disabled";
>   };
>   
> -&dss {
> -	status = "disabled";
> -};
> -
>   &icssg0_mdio {
>   	status = "disabled";
>   };

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>

  Tomi