Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 27 Jul 2023 17:11:15 +0200
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Svyatoslav Ryhel <clamor95@gmail.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Mikko Perttunen <mperttunen@nvidia.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Maxim Schwalm <maxim.schwalm@gmail.com>,
        Dmitry Osipenko <digetx@gmail.com>, devicetree@vger.kernel.org,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v2 2/2] ARM: tegra: transformers: add connector node
Message-ID: <ZMKJE1G87-jWeg2_@orome>
References: <20230618085046.10081-1-clamor95@gmail.com>
 <20230618085046.10081-3-clamor95@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="hlXONs87O1x4A5Mj"
Content-Disposition: inline
In-Reply-To: <20230618085046.10081-3-clamor95@gmail.com>
User-Agent: Mutt/2.2.10 (2023-03-25)
Precedence: bulk


--hlXONs87O1x4A5Mj
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Jun 18, 2023 at 11:50:46AM +0300, Svyatoslav Ryhel wrote:
> All ASUS Transformers have micro-HDMI connector directly available.
> After Tegra HDMI got bridge/connector support, we should use connector
> framework for proper HW description.
>=20
> Tested-by: Andreas Westman Dorcsak <hedmoo@yahoo.com> # ASUS TF T30
> Tested-by: Robert Eckelmann <longnoserob@gmail.com> # ASUS TF101 T20
> Tested-by: Svyatoslav Ryhel <clamor95@gmail.com> # ASUS TF201 T30
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> ---
>  arch/arm/boot/dts/tegra20-asus-tf101.dts      | 22 ++++++++++++++++---
>  .../dts/tegra30-asus-transformer-common.dtsi  | 21 ++++++++++++++++--
>  2 files changed, 38 insertions(+), 5 deletions(-)
>=20
> diff --git a/arch/arm/boot/dts/tegra20-asus-tf101.dts b/arch/arm/boot/dts=
/tegra20-asus-tf101.dts
> index c2a9c3fb5b33..97350f566539 100644
> --- a/arch/arm/boot/dts/tegra20-asus-tf101.dts
> +++ b/arch/arm/boot/dts/tegra20-asus-tf101.dts
> @@ -82,9 +82,11 @@ hdmi@54280000 {
>  			pll-supply =3D <&hdmi_pll_reg>;
>  			hdmi-supply =3D <&vdd_hdmi_en>;
> =20
> -			nvidia,ddc-i2c-bus =3D <&hdmi_ddc>;
> -			nvidia,hpd-gpio =3D <&gpio TEGRA_GPIO(N, 7)
> -				GPIO_ACTIVE_HIGH>;
> +			port@0 {
> +				hdmi_out: endpoint {
> +					remote-endpoint =3D <&connector_in>;
> +				};
> +			};

Does this need a bindings change? nvidia,tegra20-hdmi currently doesn't
support OF graphs, so this would probably fail to validate if we merge
it without a corresponding DT bindings update.

Thierry

--hlXONs87O1x4A5Mj
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmTCiRMACgkQ3SOs138+
s6E9qxAAlNVbjPuB+C8Cgsu4EsoY/eiPXDFu2GUEmDJ9NdiJs1gnzIfBd6o53zjW
58Ozc/9jcrMoqjO8yhCtF/5DxZL3EQnyjIat/VQI5wd8bUjJwqrE8N/TAgcOU9qR
q3EWJs51pe0QQGImdz5hadlzRHIVIFtMJsCKTEvfytYiRkM5DIAN0JFEPqIWTJ8u
08aZaATSY16sPtnvu6Kao0tZfCkY5nrrS5/81EJ4cfMxYyhR0WtEr7AQdLOfWh3l
9p/v3+wZheNbgfy+3Rq2wWVPFoZ6GZgzr+LOSbt9TsV8neyqKTE75PF2Ni3cUhGd
+KWyVNinysYAkGmsPEhwkNhbBF6re4ZtKL+9VZTMxFNMgrJev8V6wIMZIBjhHseW
g35apJYPFtnW6JEz108iexlCDOrV9Q5cLnA4eM/8nOoWd9/2oWtPhSYKQDJz1MYf
7V13e+u11thmiGysZ1SP8SpN7ZV3zyWAxBqx3/vniObofLGkG8mN9/e5kuHblYB4
S8JVwYmxATIL/gFPwng9xojBWxYyy++QvQI7enHjtRXcS+3wmYOIWYLul+r7AQ+/
18kdezTzOIEGKtqtIusmMwJN048x48I5fcU6Xea2Ov2Wzwf0LcJx5iK+MJMvt4U5
j7bk6I8DsMu3/n98qAUGr5ZvBulb6Qp9M5c6e4aS/NEDCbM3gp0=
=emKT
-----END PGP SIGNATURE-----

--hlXONs87O1x4A5Mj--