From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793973642; b=jZ3HyzwpIBg5xJhl5g5B+87GJI6U6Sp1Sbqar0CzLzXBSmtkhr6hNHPeZpACJdpO+txD sbPMBD0jx7qoeAcr44Q04npu+hXuHJ5Ry6jw1AKVvRiuerM3k3BBLH+2lv/NWU0079jn2 b2qqZMRT5fE7YpoN09hUN3jbi/MWVxyBKDrSNZAsGJxEymrV27WM8j0suAHBPEAcKWpkZ ALxMLq6kYSe6hfiUP3YM+RboUUSYNyRcn3gfsM5PK+O0BUquExyEuR0m0E++k43edSeLo JFGEbR0hpWNwLUbgbWOxp5M58l8dERqsntFohdFw0v/JBQDl4hbYl05+GTKNcT+VuGA== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793973642; h=Received:DKIM-Signature:Date:From:To:Cc:Subject:Message-ID: References:MIME-Version:Content-Type:Content-Disposition:In-Reply-To; bh=wRVjIXy9TGBWTWpR1Rc+WZB3zQiqe4/4/J3dFtbs9zY=; b=RyGwowEvrgJJtevbOC+5f7Bd6W69DeR4SuaY0kam4C3Sdpgo4wyIzNcrx8UEQdJS51FZ +1zWW2eZxBY+LJsvxRZDygoiNAFtbSjWDfBLoykD8fu4XLk0/9W/w2gMJZ3UX5KLK6YkQ /Wf0bU+qxy9/w7GAVrszW2Uwj3bI2/rr+O/3s3qbymY/Q64mO1PeuxbAhGzclIdt0WgWE zJkJBhVQYduWxe0tVWIhWiSJNyEKqvQXIroE77Cyu0UMTilbE1KUcDOv2ISsTaFRj+wfw olaSxlJ9x5pR/8M63UrTwZwIjabIxSbe6jHYFlI7yIrFBG+5ECNwtGkRktCRiSdv4rQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=R1420DWV; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E7F4B606D81D; Fri, 6 Nov 2026 14:00:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=R1420DWV Received: by smtp.kernel.org (Postfix) with ESMTPSA id F16201F0089F; Fri, 6 Nov 2026 14:00:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1793973641; bh=wRVjIXy9TGBWTWpR1Rc+WZB3zQiqe4/4/J3dFtbs9zY=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=R1420DWVifmxmagNdoRxzZEgs/x2BPuX22tsR4ydJOJE6YHtmhXAJzuBQcwEb5i6z vyT3GFCozyBOdeA98m3xe70KhUAzyuc8GUyWYUlyDntqf355llVRiXcdbDF4laC0q8 S8ah/xRCakntFQnqXwWIm0TpQT4wGxwa+jHDm8RMFqGgI+k2XixOfWoDGHvObjzsj1 aOSze9h0zcQQvykGTKj5nCC4tUPzgCBOKrFZlJyxldUxzaJhykzDNXAS0pmT011gfv fypKx+hvGJkRIPrTVkstJE3TB3VuxCFFhIsf+1y2qxhX4dYseD8P9E0of6o4HdZKIF 5wyrZYbqBtbiQ== Date: Fri, 6 Nov 2026 15:00:39 +0100 From: Thierry Reding To: Svyatoslav Ryhel Cc: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jonathan Hunter , Neil Armstrong , Jessica Zhang , Mikko Perttunen , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 06/10] drm/tegra: Make panel parsing by tegra_output_probe() optional Message-ID: References: <20261105140510.165165-1-clamor95@gmail.com> <20261105140510.165165-7-clamor95@gmail.com> X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="n7bv3uwiurpn2o3g" Content-Disposition: inline In-Reply-To: <20261105140510.165165-7-clamor95@gmail.com> --n7bv3uwiurpn2o3g Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v6 06/10] drm/tegra: Make panel parsing by tegra_output_probe() optional MIME-Version: 1.0 On Thu, Nov 05, 2026 at 04:05:06PM +0200, Svyatoslav Ryhel wrote: > The tegra_output_probe() performs panel parsing unconditionally, even for > devices that do not require it (such as HDMI) or that handle panel parsing > on their own (like DSI). Make panel parsing configurable. >=20 > Signed-off-by: Svyatoslav Ryhel > --- > drivers/gpu/drm/tegra/drm.h | 2 +- > drivers/gpu/drm/tegra/dsi.c | 2 +- > drivers/gpu/drm/tegra/hdmi.c | 2 +- > drivers/gpu/drm/tegra/output.c | 52 ++++++++++++++++++---------------- > drivers/gpu/drm/tegra/rgb.c | 2 +- > drivers/gpu/drm/tegra/sor.c | 2 +- > 6 files changed, 32 insertions(+), 30 deletions(-) Does it break anything to try and parse this even when it's not needed? In all cases we error out early, so there shouldn't be a whole lot of extra overhead either. Adding this extra parameter just adds extra complexity for seemingly no benefit. Thierry --n7bv3uwiurpn2o3g Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmrt3YcACgkQ3SOs138+ s6E8exAAiH/mHZ9cuL6ulrpqerIySRaZiLUbRbSebdUEExACTAtEOTvyEZ56IQq6 cR/IkBzw3iJ9JwIqPZzGa06Vn0PxjKspBQZvo07AaB7TlFEHMG6YHG0TVg3ZNWDs lRvkYey20lYjg8TlxyTsQYcbp4W0u61hauzvEXSLlLvKRNcJPeW6imxriMpdwlVB BAgE3r06ECHjirSDRS+ezIe6jtnB5ptx3lzPK1LZRoTXk2zAX0R8ccSKF/zWK6cp frsXwwOrobyjPdqx19tIAQmvGkyO60BYshMxX+zpoZnNkoaIoErgnZcqXZFoDw95 I2VGZdJMEQmSH1At8ArHttea380qBSLTuPUs7D7U1xnlmSL7gk/3NwYa+/79B7ns 66llAQNwxNSqiukjVBbKe8mTIXKWmTwWmcg6aAL+CrvajG13qbJW9mhQ5mX/Xh86 y6I7PhQCuWiJ7iexFW+/4v6SF6T74d3aqa66EHPbu/dZmxQ71BZEn2Mu2e8hWK0e scMT6kYp557q63t68kUt8+9AvphY1seWA9EuDe36DxrmcuDSJh+dFO+53XbVGum9 CuYXpEd0O8NRShfAYRbW6cWgekWhgq58ywU/dGxwNOgz2F7JBrMTa9919dXsVe7t te9UHAL/6gzYA6n1+3FYpoeefVIBL3TZar28SuKEY5mMxv2z98o= =Z25q -----END PGP SIGNATURE----- --n7bv3uwiurpn2o3g--