From mboxrd@z Thu Jan  1 00:00:00 1970
From: Eric Anholt <eric@anholt.net>
Subject: Re: [PATCH 6/7] drm/drivers: drop redundant
	drm_edid_to_eld() calls
Date: Wed, 01 Nov 2017 09:54:33 -0700
Message-ID: <87y3nqndme.fsf@anholt.net>
References: <cover.1509545641.git.jani.nikula@intel.com>
 <0959ca02b983afc9e74dd9acd190ba6e25f21678.1509545641.git.jani.nikula@intel.com>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0833351361=="
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
In-Reply-To: <0959ca02b983afc9e74dd9acd190ba6e25f21678.1509545641.git.jani.nikula@intel.com>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: dri-devel@lists.freedesktop.org
Cc: Archit Taneja <architt@codeaurora.org>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, jani.nikula@intel.com, intel-gfx@lists.freedesktop.org, Russell King <linux@armlinux.org.uk>, Andrzej Hajda <a.hajda@samsung.com>, CK Hu <ck.hu@mediatek.com>, Ben Skeggs <bskeggs@redhat.com>, Philipp Zabel <p.zabel@pengutronix.de>, Alex Deucher <alexander.deucher@amd.com>, Vincent Abriou <vincent.abriou@st.com>, Christian =?utf-8?Q?K=C3=B6nig?= <christian.koenig@amd.com>, Mark Yao <mark.yao@rock-chips.com>
List-Id: dri-devel@lists.freedesktop.org

--===============0833351361==
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha512; protocol="application/pgp-signature"

--=-=-=
Content-Type: text/plain

Jani Nikula <jani.nikula@intel.com> writes:

> drm_add_edid_modes() now fills in the ELD automatically, so the calls to
> drm_edid_to_eld() are redundant. Remove them.
>
> All the other places are obvious, but nv50 has detached
> drm_edid_to_eld() from the drm_add_edid_modes() call.

Nice!  For vc4,

Acked-by: Eric Anholt <eric@anholt.net>

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

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

iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAln5/EkACgkQtdYpNtH8
nuhLWg//d0RwjKEwMnmeqeEQje6H4w7nw3aBfwARKLXX+uh0CKLeG+S7cYZx8NsI
c61UqZbCLR+q+JZgdgTZ4lDORo55kwsKHnYIXTqB87MkkVw9F+PCfxcFGe21S/dV
sr13PCYc8hHSNPjfxACB9vyiTf2ANHAId0PU/fAHmGvnmxk74jN2bYQDdKlL4s4T
YCO9l5GBYEmjA5Ll4w//0xIGAZrSXfRgBK8AkW5NaxRY9/naXHseXwGnyiMQF64+
YrMsUQ+kBYHkFPtuJOb/J7q+JZwUhZ7YEz5ly8GdSPV4r83DyqmTh9bIYQF7dczC
3tPRLYVSAVVFCTfBCLNCcwHxHLCs/S4DnlKsjVPR1n6A3XCAhy94BK0GsCp+z3SA
R01dtjdSdAhYAeMm75Lo+fkhgxHx4Y+rVjWK7VVpZEI2lixykaiVZRuLsjTtMpwd
SdIlqk94TrdmkEBuAGesRfXTKwtRIaugoD9i74KiWU9+3NfPDglTLQ+YBbScdSs7
2q1C+IZJ/n7AvVr7m0/nbpNfNwfMg4hBe1CyAtnsZ+iqJGnjkk+QhI9CkKm/P7/Z
YRjZ9dWPhWFfVlgqFOHXpExROodhsbxBAMEG2235wlpmeHwX4xUQA3E+JKxO/s9O
yiKzZON1rxYRS0TYV7tGFpyP0vIA5FNeyX4TXXxu/SZRls3GPns=
=EN/B
-----END PGP SIGNATURE-----
--=-=-=--

--===============0833351361==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline

X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4
IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz
dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==

--===============0833351361==--