Module Name:    src
Committed By:   thorpej
Date:           Fri Jan 21 15:55:37 UTC 2022

Modified Files:
        src/sys/arch/sparc/sparc: promlib.c
        src/sys/dev/i2c: i2c.c
        src/sys/dev/pci: pci.c
        src/sys/kern: subr_device.c
        src/sys/sys: device.h

Log Message:
Replace devhandle_invalidate(), which invalidates a devhandle, with
devhandle_invalid(), which returns an invalid devhandle.


To generate a diff of this commit:
cvs rdiff -u -r1.50 -r1.51 src/sys/arch/sparc/sparc/promlib.c
cvs rdiff -u -r1.81 -r1.82 src/sys/dev/i2c/i2c.c
cvs rdiff -u -r1.163 -r1.164 src/sys/dev/pci/pci.c
cvs rdiff -u -r1.9 -r1.10 src/sys/kern/subr_device.c
cvs rdiff -u -r1.175 -r1.176 src/sys/sys/device.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/arch/sparc/sparc/promlib.c
diff -u src/sys/arch/sparc/sparc/promlib.c:1.50 src/sys/arch/sparc/sparc/promlib.c:1.51
--- src/sys/arch/sparc/sparc/promlib.c:1.50	Fri Sep 17 16:16:18 2021
+++ src/sys/arch/sparc/sparc/promlib.c	Fri Jan 21 15:55:36 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: promlib.c,v 1.50 2021/09/17 16:16:18 thorpej Exp $ */
+/*	$NetBSD: promlib.c,v 1.51 2022/01/21 15:55:36 thorpej Exp $ */
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -35,7 +35,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: promlib.c,v 1.50 2021/09/17 16:16:18 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: promlib.c,v 1.51 2022/01/21 15:55:36 thorpej Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_sparc_arch.h"
@@ -99,11 +99,7 @@ static void	opf_interpret_simple(const c
 static devhandle_t
 null_prom_to_devhandle(int node __unused)
 {
-	devhandle_t devhandle;
-
-	devhandle_invalidate(&devhandle);
-
-	return devhandle;
+	return devhandle_invalid();
 }
 
 static int

Index: src/sys/dev/i2c/i2c.c
diff -u src/sys/dev/i2c/i2c.c:1.81 src/sys/dev/i2c/i2c.c:1.82
--- src/sys/dev/i2c/i2c.c:1.81	Mon Jan 17 19:34:31 2022
+++ src/sys/dev/i2c/i2c.c	Fri Jan 21 15:55:36 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: i2c.c,v 1.81 2022/01/17 19:34:31 thorpej Exp $	*/
+/*	$NetBSD: i2c.c,v 1.82 2022/01/21 15:55:36 thorpej Exp $	*/
 
 /*
  * Copyright (c) 2003 Wasabi Systems, Inc.
@@ -53,7 +53,7 @@
 #endif /* _KERNEL_OPT */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: i2c.c,v 1.81 2022/01/17 19:34:31 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: i2c.c,v 1.82 2022/01/21 15:55:36 thorpej Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -495,7 +495,7 @@ iic_attach(device_t parent, device_t sel
 			ia.ia_cookietype = cookietype;
 			ia.ia_prop = dev;
 
-			devhandle_invalidate(&devhandle);
+			devhandle = devhandle_invalid();
 #ifdef I2C_USE_FDT
 			if (cookietype == I2C_COOKIE_OF) {
 				devhandle = devhandle_from_of((int)cookie);

Index: src/sys/dev/pci/pci.c
diff -u src/sys/dev/pci/pci.c:1.163 src/sys/dev/pci/pci.c:1.164
--- src/sys/dev/pci/pci.c:1.163	Sun Oct 10 23:28:36 2021
+++ src/sys/dev/pci/pci.c	Fri Jan 21 15:55:36 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: pci.c,v 1.163 2021/10/10 23:28:36 msaitoh Exp $	*/
+/*	$NetBSD: pci.c,v 1.164 2022/01/21 15:55:36 thorpej Exp $	*/
 
 /*
  * Copyright (c) 1995, 1996, 1997, 1998
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pci.c,v 1.163 2021/10/10 23:28:36 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pci.c,v 1.164 2022/01/21 15:55:36 thorpej Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_pci.h"
@@ -283,7 +283,7 @@ pci_bus_get_child_devhandle(struct pci_s
 		 * device was not found in the platform device tree.
 		 * Return an invalid handle.
 		 */
-		devhandle_invalidate(&args.devhandle);
+		return devhandle_invalid();
 	}
 
 	return args.devhandle;

Index: src/sys/kern/subr_device.c
diff -u src/sys/kern/subr_device.c:1.9 src/sys/kern/subr_device.c:1.10
--- src/sys/kern/subr_device.c:1.9	Wed Sep 15 17:33:08 2021
+++ src/sys/kern/subr_device.c	Fri Jan 21 15:55:36 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: subr_device.c,v 1.9 2021/09/15 17:33:08 thorpej Exp $	*/
+/*	$NetBSD: subr_device.c,v 1.10 2022/01/21 15:55:36 thorpej Exp $	*/
 
 /*
  * Copyright (c) 2006, 2021 The NetBSD Foundation, Inc.
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: subr_device.c,v 1.9 2021/09/15 17:33:08 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: subr_device.c,v 1.10 2022/01/21 15:55:36 thorpej Exp $");
 
 #include <sys/param.h>
 #include <sys/device.h>
@@ -57,11 +57,14 @@ devhandle_is_valid(devhandle_t handle)
 	return devhandle_is_valid_internal(&handle);
 }
 
-void
-devhandle_invalidate(devhandle_t * const handlep)
+devhandle_t
+devhandle_invalid(void)
 {
-	handlep->impl = NULL;
-	handlep->uintptr = 0;
+	static const devhandle_t invalid_devhandle = {
+		.impl = NULL,
+		.uintptr = 0,
+	};
+	return invalid_devhandle;
 }
 
 devhandle_type_t

Index: src/sys/sys/device.h
diff -u src/sys/sys/device.h:1.175 src/sys/sys/device.h:1.176
--- src/sys/sys/device.h:1.175	Wed Sep 15 17:33:08 2021
+++ src/sys/sys/device.h	Fri Jan 21 15:55:36 2022
@@ -1,4 +1,4 @@
-/* $NetBSD: device.h,v 1.175 2021/09/15 17:33:08 thorpej Exp $ */
+/* $NetBSD: device.h,v 1.176 2022/01/21 15:55:36 thorpej Exp $ */
 
 /*
  * Copyright (c) 2021 The NetBSD Foundation, Inc.
@@ -672,7 +672,7 @@ void		device_set_handle(device_t, devhan
 devhandle_t	device_handle(device_t);
 
 bool		devhandle_is_valid(devhandle_t);
-void		devhandle_invalidate(devhandle_t *);
+devhandle_t	devhandle_invalid(void);
 devhandle_type_t devhandle_type(devhandle_t);
 
 device_call_t	devhandle_lookup_device_call(devhandle_t, const char *,

Reply via email to