From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH] tests: modetest: Accept connector names in addition to connector IDs Date: Tue, 21 Jul 2015 11:05:59 +0200 Message-ID: <20150721090557.GD20388@ulmo.nvidia.com> References: <1437411380-10015-1-git-send-email-laurent.pinchart@ideasonboard.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1974524795==" Return-path: Received: from hqemgate14.nvidia.com (hqemgate14.nvidia.com [216.228.121.143]) by gabe.freedesktop.org (Postfix) with ESMTPS id B600E6E91B for ; Tue, 21 Jul 2015 02:06:21 -0700 (PDT) In-Reply-To: <1437411380-10015-1-git-send-email-laurent.pinchart@ideasonboard.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1974524795== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="Y7xTucakfITjPcLV" Content-Disposition: inline --Y7xTucakfITjPcLV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jul 20, 2015 at 07:56:20PM +0300, Laurent Pinchart wrote: > From: Thierry Reding >=20 > Allow connector names to be used in the specification of the -s option. > This requires storing the string passed on the command-line so that it > can later be resolved to a connector ID (after the DRM device has been > opened). >=20 > Connector names are constructed from the connector type name and > connector type ID using the same format as used internally in the > Linux kernel. >=20 > Signed-off-by: Thierry Reding > Signed-off-by: Laurent Pinchart > --- > tests/modetest/modetest.c | 94 +++++++++++++++++++++++++++++++++++++++++= ------ > 1 file changed, 82 insertions(+), 12 deletions(-) >=20 > I've carried this patch in my tree for too long and I'm now tired of reba= sing > it. It's thus time to get it in mainline. >=20 > Compared to the original version developed by Thierry, I've >=20 > - rebased the patch on top of latest master > - dropped dependencies on other WIP patches in Thierry's tree > - fixed printf alignment > - used asprintf to simplify name string allocation > - constructed the connector name using the same format as the Linux kernel Heh, I started to get all these patches into shape last week, and this will probably give me more rebase headaches. But I guess that's what I deserve for letting things slide... Thierry --Y7xTucakfITjPcLV Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVrgt1AAoJEN0jrNd/PrOhRKAQALXZUroVwnHjnoRWHgUBIIf6 yPBcVBYw3eA8BiiYc7ktycC6pJ/V3r4Dr1C9YFkYFo11BYRcvzF9Li8M7VnLTMxp DEPS8oQtH/E/EdsxdIATti8nBTm8aywwBVI0Ko16YY+J4yDAdvR7vaIGRLnKVYu6 13a7olZHT8N59q3i3oZm5/HG43666dx9BgcMA07XtrRyR6GWaJWs6MtBq4FIQjUu UnpRFNjeL44E6QuwR3wasSkzeLocjjkuI3HTr2hpDiJhZFMNvPw4YJ3IYTTy/tss KW8sT+yhtv2sYHIXfmsPSvF/43iut4pghBqlh1/Wr6COWCqk6gUq1PtykOFMab+d QzQRlU4AlmrOEwuBApT6IdBn6Vvax8QwslMk3TgsgQ72LfQ1Xl2MDatMzbv9n51C u1o4gN4AqlT4FDgmQrPBV25jSlBxOpwSUKfcKKVlAtYCwVHdcomkfIcyw4dgY03F WpsnfD1p9Thz6OBrUefJaKf8zNrb0ALzvSKuWO1H8f5KB+qsrSzI3g2F4nOrrpin 5NlOVyK6PUSr7O96cNM7CKdvBpcZBhFWffQ8LpwrptBD3vyu8IwMYFsVzwHYJH84 WTxaa8hoTmK/6yhZtI+sbaXNeczcJC6hm7yCnMXiWFLzInvcGTykx6gTunrVvv1R NSZJHEV5Sv8i/KRliiDQ =AZfT -----END PGP SIGNATURE----- --Y7xTucakfITjPcLV-- --===============1974524795== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============1974524795==--