Module Name:    src
Committed By:   riastradh
Date:           Sun Dec 19 12:41:54 UTC 2021

Modified Files:
        src/sys/external/bsd/drm2/dist/drm: drm_dp_helper.c
        src/sys/external/bsd/drm2/dist/drm/i915/display: intel_dp.c
        src/sys/external/bsd/drm2/dist/include/drm: drm_dp_helper.h

Log Message:
drm: add drm_dp_aux_fini; use it in i915 to fix missing mutex destroy.

Author: chuq <c...@netbsd.org>
Committer: Taylor R Campbell <riastr...@netbsd.org>


To generate a diff of this commit:
cvs rdiff -u -r1.15 -r1.16 src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c
cvs rdiff -u -r1.6 -r1.7 \
    src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c
cvs rdiff -u -r1.8 -r1.9 \
    src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c
diff -u src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c:1.15 src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c:1.16
--- src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c:1.15	Sun Dec 19 12:32:01 2021
+++ src/sys/external/bsd/drm2/dist/drm/drm_dp_helper.c	Sun Dec 19 12:41:54 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: drm_dp_helper.c,v 1.15 2021/12/19 12:32:01 riastradh Exp $	*/
+/*	$NetBSD: drm_dp_helper.c,v 1.16 2021/12/19 12:41:54 riastradh Exp $	*/
 
 /*
  * Copyright © 2009 Keith Packard
@@ -23,7 +23,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: drm_dp_helper.c,v 1.15 2021/12/19 12:32:01 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: drm_dp_helper.c,v 1.16 2021/12/19 12:41:54 riastradh Exp $");
 
 #include <linux/delay.h>
 #include <linux/errno.h>
@@ -1024,6 +1024,17 @@ void drm_dp_aux_init(struct drm_dp_aux *
 EXPORT_SYMBOL(drm_dp_aux_init);
 
 /**
+ * drm_dp_aux_fini() - undo what drm_dp_aux_init() does.
+ * @aux: DisplayPort AUX channel
+ */
+void drm_dp_aux_fini(struct drm_dp_aux *aux)
+{
+	mutex_destroy(&aux->cec.lock);
+	mutex_destroy(&aux->hw_mutex);
+}
+EXPORT_SYMBOL(drm_dp_aux_fini);
+
+/**
  * drm_dp_aux_register() - initialise and register aux channel
  * @aux: DisplayPort AUX channel
  *
@@ -1075,8 +1086,7 @@ void drm_dp_aux_unregister(struct drm_dp
 {
 	drm_dp_aux_unregister_devnode(aux);
 	i2c_del_adapter(&aux->ddc);
-	mutex_destroy(&aux->cec.lock);
-	mutex_destroy(&aux->hw_mutex);
+	drm_dp_aux_fini(aux);
 }
 EXPORT_SYMBOL(drm_dp_aux_unregister);
 

Index: src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c
diff -u src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c:1.6 src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c:1.7
--- src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c:1.6	Sun Dec 19 12:40:31 2021
+++ src/sys/external/bsd/drm2/dist/drm/i915/display/intel_dp.c	Sun Dec 19 12:41:54 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: intel_dp.c,v 1.6 2021/12/19 12:40:31 riastradh Exp $	*/
+/*	$NetBSD: intel_dp.c,v 1.7 2021/12/19 12:41:54 riastradh Exp $	*/
 
 /*
  * Copyright © 2008 Intel Corporation
@@ -28,7 +28,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: intel_dp.c,v 1.6 2021/12/19 12:40:31 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: intel_dp.c,v 1.7 2021/12/19 12:41:54 riastradh Exp $");
 
 #include <linux/export.h>
 #include <linux/i2c.h>
@@ -1730,6 +1730,7 @@ static i915_reg_t skl_aux_data_reg(struc
 static void
 intel_dp_aux_fini(struct intel_dp *intel_dp)
 {
+	drm_dp_aux_fini(&intel_dp->aux);
 	kfree(__UNCONST(intel_dp->aux.name));
 }
 

Index: src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h
diff -u src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h:1.8 src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h:1.9
--- src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h:1.8	Sun Dec 19 10:48:47 2021
+++ src/sys/external/bsd/drm2/dist/include/drm/drm_dp_helper.h	Sun Dec 19 12:41:54 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: drm_dp_helper.h,v 1.8 2021/12/19 10:48:47 riastradh Exp $	*/
+/*	$NetBSD: drm_dp_helper.h,v 1.9 2021/12/19 12:41:54 riastradh Exp $	*/
 
 /*
  * Copyright © 2008 Keith Packard
@@ -1472,6 +1472,7 @@ void drm_dp_downstream_debug(struct seq_
 
 void drm_dp_remote_aux_init(struct drm_dp_aux *aux);
 void drm_dp_aux_init(struct drm_dp_aux *aux);
+void drm_dp_aux_fini(struct drm_dp_aux *aux);
 int drm_dp_aux_register(struct drm_dp_aux *aux);
 void drm_dp_aux_unregister(struct drm_dp_aux *aux);
 

Reply via email to