From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH] drm/i915: always set FDI composite sync bit Date: Tue, 11 Oct 2011 09:40:18 -0700 Message-ID: <20111011094018.13d61b3d@jbarnes-desktop> References: <1318282132-8104-1-git-send-email-jbarnes@virtuousgeek.org> <4E944E50.1070903@redhat.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0585673244==" Return-path: Received: from oproxy6-pub.bluehost.com (oproxy6-pub.bluehost.com [67.222.54.6]) by gabe.freedesktop.org (Postfix) with SMTP id ACCCE9E951 for ; Tue, 11 Oct 2011 09:40:24 -0700 (PDT) In-Reply-To: <4E944E50.1070903@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Adam Jackson Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============0585673244== Content-Type: multipart/signed; micalg=PGP-SHA1; boundary="Sig_/BwKJO6WnDzugvzmT2KWW/7T"; protocol="application/pgp-signature" --Sig_/BwKJO6WnDzugvzmT2KWW/7T Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Tue, 11 Oct 2011 10:10:24 -0400 Adam Jackson wrote: > On 10/10/11 7:22 PM, Keith Packard wrote: > > On Mon, 10 Oct 2011 14:28:52 -0700, Jesse Barnes wrote: > > > >> It's needed for 3 pipe support as well as just regular functionality > >> (e.g. DisplayPort). > > > > Any explanation on how you get sync without this? As in, why did this > > ever work? >=20 > To a first approximation, it didn't. I was never able to get anything=20 > reliably lit up besides the boot output (and only that one; hotplug=20 > after was a disaster). >=20 > I don't have register dumps handy to compare BIOS setup with this=20 > change, but I could if needed. >=20 > It still seems entirely magical and probably wrong in some situations.=20 > And I'm thrilled to see that PPT is functionally different from CPT=20 > (seriously, stop doing that) instead of just moving bit definitions=20 > around (seriously, stop doing that). But this change is massively=20 > better than before. Ok just got confirmation that always setting the composite bit is safe on IVB per the specs we provide to board designers (i.e. they're required to make composite sync work, and fsync/lsync is optional). --=20 Jesse Barnes, Intel Open Source Technology Center --Sig_/BwKJO6WnDzugvzmT2KWW/7T Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQIcBAEBAgAGBQJOlHFyAAoJEIEoDkX4Qk9hCtYQAM+6BA1cWnzW1bUC8wz8r9JU MSDhmIQaBCAH296Q3/Cnfer8se8dyrVd+UfPpPxbpUCkGqdgpWoSaeO/ar6NWlfM TsvC9PhJpYUTyPMFgBqpxSlNeo8HxkPQjypdIM1ynVVSB7qAl9O1YUFVvMA7p7j5 8HVVD61KW+tH6qJHQT1sXR7iH+8vVirL/DvDfIeF/X65+XEFrkfzYZ9Lty1CYzGd q2vjQTyuD5IbJ3baUomIxD8+Mfpwe4Tv/t0aD/3UAsqfPsZHJ5bQLYT01DGTFx38 wdl+S9hgITL5rVMhlliO0ygy3eK9haw9Ib5U8mCboSj/xfpTiWe5Lb4lqqXPcDIF R9+kqf9jtJ1uLeafTkEnlwVGpXzkSUUIUpQdblYmEsYmAxxz+hionND0HciiagzX sSjd0pwgR2dgTV8YGxZIvWik3oLKV71MiHH60Lx+4ArI3ANAMiJxGLIuGHoPfb8X xagGIFOoZVjTn5/RNPJUhrWGiQg+5FUQzf+ORt0rID6Tr0S5BreaeIF5fyfvaOhi sPlarVLpkKeIrOokoyZmWhvQdzrEyp+kuhkBsVKGjuSGXcwf6eZxkI1hR4sq0+uk +QTcjgCm6jaODIDaSzbAsIY09VIP34UJH6Xfu7fRId7hiQxyUExp6fEEWxoZWRJO UT+vmiNlo0WSCPrMEZ7+ =XkOv -----END PGP SIGNATURE----- --Sig_/BwKJO6WnDzugvzmT2KWW/7T-- --===============0585673244== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0585673244==--