Ilia Mirkin
2017-Sep-03 15:46 UTC
[Nouveau] v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
Commit df8dc97cd17 (drm/nouveau/kms/nv50: use drm core i2c-over-aux algorithm) switched things over to the drm algo. Unfortunately it generates address-only transactions. Prior to GF119, the hardware had no support for such things, and GF119+ the nouveau code did not handle these properly. The result of this bug is that EDID doesn't make it over for DP (and eDP) screens, which in turn wreaks all sorts of havoc. There have been lots of reports on IRC, a few on mailing lists, and also: https://bugs.freedesktop.org/show_bug.cgi?id=102528 Commit 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions) fixes the nouveau support for such transactions on GF119+ and flips things back over to the custom algo for earlier chips. Please include into the next v4.12.x release: commit 13a86519202c5d119d83640d6f781f3181205d2c Author: Ben Skeggs <bskeggs at redhat.com> Date: Wed Jul 19 16:49:59 2017 +1000 drm/nouveau/i2c/gf119-: add support for address-only transactions Since switching the I2C-over-AUX helpers, there have been regressions on some display combinations due to us not having support for "address only" transactions. This commits enables support for them for GF119 and newer. Earlier GPUs have been reverted to a custom I2C-over-AUX algorithm. Signed-off-by: Ben Skeggs <bskeggs at redhat.com>
rosenp at gmail.com
2017-Sep-03 22:56 UTC
[Nouveau] v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
Fedora bug: https://bugzilla.redhat.com/show_bug.cgi?id=1482220 Also, +1. On Sun, 2017-09-03 at 11:46 -0400, Ilia Mirkin wrote:> Commit df8dc97cd17 (drm/nouveau/kms/nv50: use drm core i2c-over-aux > algorithm) switched things over to the drm algo. Unfortunately it > generates address-only transactions. Prior to GF119, the hardware had > no support for such things, and GF119+ the nouveau code did not > handle > these properly. > > The result of this bug is that EDID doesn't make it over for DP (and > eDP) screens, which in turn wreaks all sorts of havoc. There have > been > lots of reports on IRC, a few on mailing lists, and also: > > https://bugs.freedesktop.org/show_bug.cgi?id=102528 > > Commit 13a86519202 (drm/nouveau/i2c/gf119-: add support for > address-only transactions) fixes the nouveau support for such > transactions on GF119+ and flips things back over to the custom algo > for earlier chips. Please include into the next v4.12.x release: > > commit 13a86519202c5d119d83640d6f781f3181205d2c > Author: Ben Skeggs <bskeggs at redhat.com> > Date: Wed Jul 19 16:49:59 2017 +1000 > > drm/nouveau/i2c/gf119-: add support for address-only transactions > > Since switching the I2C-over-AUX helpers, there have been > regressions on > some display combinations due to us not having support for > "address only" > transactions. > > This commits enables support for them for GF119 and newer. > > Earlier GPUs have been reverted to a custom I2C-over-AUX > algorithm. > > Signed-off-by: Ben Skeggs <bskeggs at redhat.com> > _______________________________________________ > Nouveau mailing list > Nouveau at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/nouveau
Greg KH
2017-Sep-04 11:19 UTC
[Nouveau] v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
On Sun, Sep 03, 2017 at 11:46:57AM -0400, Ilia Mirkin wrote:> Commit df8dc97cd17 (drm/nouveau/kms/nv50: use drm core i2c-over-aux > algorithm) switched things over to the drm algo. Unfortunately it > generates address-only transactions. Prior to GF119, the hardware had > no support for such things, and GF119+ the nouveau code did not handle > these properly. > > The result of this bug is that EDID doesn't make it over for DP (and > eDP) screens, which in turn wreaks all sorts of havoc. There have been > lots of reports on IRC, a few on mailing lists, and also: > > https://bugs.freedesktop.org/show_bug.cgi?id=102528 > > Commit 13a86519202 (drm/nouveau/i2c/gf119-: add support for > address-only transactions) fixes the nouveau support for such > transactions on GF119+ and flips things back over to the custom algo > for earlier chips. Please include into the next v4.12.x release:Now queued up, thanks. greg k-h
Possibly Parallel Threads
- [PATCH] disp/gf119-: avoid creating non-existent heads
- [Bug 54646] New: i2c failure on Quadro NVS 4200M (GF119 (NVD9))
- [PATCH] disp/gf119-: avoid creating non-existent heads
- [Bug 102528] New: Display messed up shortly after boot on laptop with GT218M
- [PATCH] drm/nouveau/disp/gf119: add missing drive vfunc ptr