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 *,