diff mbox

[xf86-video-intel] sna: Reprobe if kernel updates the connector mode list

Message ID 1476743484-16733-1-git-send-email-manasi.d.navare@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Navare, Manasi Oct. 17, 2016, 10:31 p.m. UTC
Output_check_status() should return a false if it detects
that the connector mode list has changed so that sna_mode_discover
can reprobe.

Fixes: eb01cc549d4d (sna: Refresh mode list if the kernel updates)

Signed-off-by: Manasi Navare <manasi.d.navare@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ville Syrjala <ville.syrjala@linux.intel.com>
---
 src/sna/sna_display.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Chris Wilson Oct. 18, 2016, 8:46 a.m. UTC | #1
On Mon, Oct 17, 2016 at 03:31:24PM -0700, Manasi Navare wrote:
> Output_check_status() should return a false if it detects
> that the connector mode list has changed so that sna_mode_discover
> can reprobe.
> 
> Fixes: eb01cc549d4d (sna: Refresh mode list if the kernel updates)
> 
> Signed-off-by: Manasi Navare <manasi.d.navare@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Ville Syrjala <ville.syrjala@linux.intel.com>

Pushed, thanks.
-Chris
diff mbox

Patch

diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c
index 15be27c..ea7e287 100644
--- a/src/sna/sna_display.c
+++ b/src/sna/sna_display.c
@@ -5191,7 +5191,7 @@  output_check_status(struct sna *sna, struct sna_output *output)
 		return true;
 
 	if (output->num_modes != compat_conn.conn.count_modes)
-		return true;
+		return false;
 
 	if (output->edid_len == 0)
 		return false;