src/sys/dev/bhnd/siba, branch main FreeBSD source tree bhnd_bus_*_resource: Remove redundant type and rid arguments 2025-12-09T20:03:42+00:00 John Baldwin jhb@FreeBSD.org 2025-12-09T20:03:42+00:00 ca942ab4b2b665e5375cc72afc4930d3b33b14f9 Remove type and rid arguments from bhnd_bus_(activate|deactivate|release)_resource. This should have been done earlier to match the changes made to bus_release_resource, etc. While fixing up the callers, remove rid members from softc structures since the only time a value is needed is as a constant input to bhnd_bus_alloc_resource*. Reviewed by: imp Differential Revision: https://reviews.freebsd.org/D53410 
Remove type and rid arguments from
bhnd_bus_(activate|deactivate|release)_resource.  This should have
been done earlier to match the changes made to bus_release_resource,
etc.

While fixing up the callers, remove rid members from softc structures
since the only time a value is needed is as a constant input to
bhnd_bus_alloc_resource*.

Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D53410
bhnd_bus_alloc_resource*: Pass rid by value 2025-12-09T20:03:25+00:00 John Baldwin jhb@FreeBSD.org 2025-12-09T20:03:25+00:00 1033312267d66496c661feefe6c14a86d7e4b95e Reviewed by: imp Differential Revision: https://reviews.freebsd.org/D53409 
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D53409
bhnd bcma/siba: Remove duplicate call to device_delete_children 2025-01-02T18:24:13+00:00 John Baldwin jhb@FreeBSD.org 2025-01-02T18:24:13+00:00 20f932af2973afc7624ac40a843b85b48a3947c7 bcma/siba_add_children already call device_delete_children if an error occurs. Differential Revision: https://reviews.freebsd.org/D47968 
bcma/siba_add_children already call device_delete_children if an error
occurs.

Differential Revision:	https://reviews.freebsd.org/D47968
siba: Use device_delete_children in siba_add_children 2025-01-02T18:23:54+00:00 John Baldwin jhb@FreeBSD.org 2025-01-02T18:23:54+00:00 919f911ea4523276b7863e32e89943f270d3a76b Differential Revision: https://reviews.freebsd.org/D47967 
Differential Revision:	https://reviews.freebsd.org/D47967
newbus: replace -1 in BUS_ADD_CHILD(...-1) with DEVICE_UNIT_ANY 2024-07-25T04:23:02+00:00 Warner Losh imp@FreeBSD.org 2024-07-25T04:23:02+00:00 a05a680469a7ac77b195021fed74e3aa58152dd7 Sponsored by: Netflix 
Sponsored by:		Netflix
bhnd: Correct the softc size in the siba_bhndb_driver definition 2023-12-05T18:47:03+00:00 Mark Johnston markj@FreeBSD.org 2023-12-05T18:47:03+00:00 4c3aa00c0a0093c78f42d138bb9eef9b1a7cbb39 struct siba_bhndb_softc embeds struct siba_softc and adds an extra field, "quirks". In practice, this bug was harmless since "quirks" is unconditionally initialized during driver attach and would have lived in the redzone of the softc allocation, but KASAN catches the out-of-bounds access. PR: 275515 Reported by: Frank Hilgendorf <frank.hilgendorf@posteo.de> MFC after: 1 week 
struct siba_bhndb_softc embeds struct siba_softc and adds an extra
field, "quirks".  In practice, this bug was harmless since "quirks" is
unconditionally initialized during driver attach and would have lived in
the redzone of the softc allocation, but KASAN catches the out-of-bounds
access.

PR:		275515
Reported by:	Frank Hilgendorf <frank.hilgendorf@posteo.de>
MFC after:	1 week
sys: Automated cleanup of cdefs and other formatting 2023-11-27T05:24:00+00:00 Warner Losh imp@FreeBSD.org 2023-11-24T20:12:57+00:00 fdafd315ad0d0f28a11b9fb4476a9ab059c62b92 Apply the following automated changes to try to eliminate no-longer-needed sys/cdefs.h includes as well as now-empty blank lines in a row. Remove /^#if.*\n#endif.*\n#include\s+<sys/cdefs.h>.*\n/ Remove /\n+#include\s+<sys/cdefs.h>.*\n+#if.*\n#endif.*\n+/ Remove /\n+#if.*\n#endif.*\n+/ Remove /^#if.*\n#endif.*\n/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/types.h>/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/param.h>/ Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/capsicum.h>/ Sponsored by: Netflix 
Apply the following automated changes to try to eliminate
no-longer-needed sys/cdefs.h includes as well as now-empty
blank lines in a row.

Remove /^#if.*\n#endif.*\n#include\s+<sys/cdefs.h>.*\n/
Remove /\n+#include\s+<sys/cdefs.h>.*\n+#if.*\n#endif.*\n+/
Remove /\n+#if.*\n#endif.*\n+/
Remove /^#if.*\n#endif.*\n/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/types.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/param.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/capsicum.h>/

Sponsored by:		Netflix
sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16T17:54:36+00:00 Warner Losh imp@FreeBSD.org 2023-08-16T17:54:36+00:00 685dc743dc3b5645e34836464128e1c0558b404b Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/ 
Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/
sys: Remove $FreeBSD$: one-line .h pattern 2023-08-16T17:54:18+00:00 Warner Losh imp@FreeBSD.org 2023-08-16T17:54:18+00:00 2ff63af9b88c7413b7d71715b5532625752a248e Remove /^\s*\*+\s*\$FreeBSD\$.*$\n/ 
Remove /^\s*\*+\s*\$FreeBSD\$.*$\n/
spdx: The BSD-2-Clause-FreeBSD identifier is obsolete, drop -FreeBSD 2023-05-12T16:44:03+00:00 Warner Losh imp@FreeBSD.org 2023-05-10T15:40:58+00:00 4d846d260e2b9a3d4d0a701462568268cbfe7a5b The SPDX folks have obsoleted the BSD-2-Clause-FreeBSD identifier. Catch up to that fact and revert to their recommended match of BSD-2-Clause. Discussed with: pfg MFC After: 3 days Sponsored by: Netflix 
The SPDX folks have obsoleted the BSD-2-Clause-FreeBSD identifier. Catch
up to that fact and revert to their recommended match of BSD-2-Clause.

Discussed with:		pfg
MFC After:		3 days
Sponsored by:		Netflix