All of lore.kernel.org
 help / color / mirror / Atom feed
* v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
@ 2017-09-03 15:46 ` Ilia Mirkin
  0 siblings, 0 replies; 5+ messages in thread
From: Ilia Mirkin @ 2017-09-03 15:46 UTC (permalink / raw)
  To: stable; +Cc: Ben Skeggs, Dave Airlie, nouveau

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@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@redhat.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
@ 2017-09-03 15:46 ` Ilia Mirkin
  0 siblings, 0 replies; 5+ messages in thread
From: Ilia Mirkin @ 2017-09-03 15:46 UTC (permalink / raw)
  To: stable; +Cc: Dave Airlie, Ben Skeggs, nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

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@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@redhat.com>
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Nouveau] v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
  2017-09-03 15:46 ` Ilia Mirkin
  (?)
@ 2017-09-03 22:56 ` rosenp
  -1 siblings, 0 replies; 5+ messages in thread
From: rosenp @ 2017-09-03 22:56 UTC (permalink / raw)
  To: Ilia Mirkin, stable; +Cc: Dave Airlie, Ben Skeggs, nouveau

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@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@redhat.com>
> _______________________________________________
> Nouveau mailing list
> Nouveau@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/nouveau

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
@ 2017-09-04 11:19   ` Greg KH
  0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2017-09-04 11:19 UTC (permalink / raw)
  To: Ilia Mirkin; +Cc: stable, Ben Skeggs, Dave Airlie, nouveau

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions)
@ 2017-09-04 11:19   ` Greg KH
  0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2017-09-04 11:19 UTC (permalink / raw)
  To: Ilia Mirkin
  Cc: Dave Airlie, Ben Skeggs, stable,
	nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

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
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-09-04 11:19 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-03 15:46 v4.12 backport request: 13a86519202 (drm/nouveau/i2c/gf119-: add support for address-only transactions) Ilia Mirkin
2017-09-03 15:46 ` Ilia Mirkin
2017-09-03 22:56 ` [Nouveau] " rosenp
2017-09-04 11:19 ` Greg KH
2017-09-04 11:19   ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.