Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 4 Jan 2022 18:03:30 GMT
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 6b06f8a7927f - stable/13 - amdtemp: Revert related part of "Make CPU children" commit.
Message-ID:  <202201041803.204I3UPH025672@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by mav:

URL: https://cgit.FreeBSD.org/src/commit/?id=6b06f8a7927fff624685eb4131292b6736a08728

commit 6b06f8a7927fff624685eb4131292b6736a08728
Author:     Alexander Motin <mav@FreeBSD.org>
AuthorDate: 2021-12-01 18:00:17 +0000
Commit:     Alexander Motin <mav@FreeBSD.org>
CommitDate: 2022-01-04 17:21:49 +0000

    amdtemp: Revert related part of "Make CPU children" commit.
    
    While it still looks like previous code worked by coincidence, this
    change broke things even more instead of fixing.
    
    Reported by:    avg@
    MFC after:      1 week
    
    (cherry picked from commit 94a72c5ac4285b2940e5d2c7e53d21c84023defc)
---
 sys/dev/amdtemp/amdtemp.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/sys/dev/amdtemp/amdtemp.c b/sys/dev/amdtemp/amdtemp.c
index c1d696d070a5..21227a47c513 100644
--- a/sys/dev/amdtemp/amdtemp.c
+++ b/sys/dev/amdtemp/amdtemp.c
@@ -263,8 +263,7 @@ amdtemp_identify(driver_t *driver, device_t parent)
 		return;
 
 	if (amdtemp_match(parent, NULL)) {
-		child = device_add_child(parent, "amdtemp",
-		    device_get_unit(parent));
+		child = device_add_child(parent, "amdtemp", -1);
 		if (child == NULL)
 			device_printf(parent, "add amdtemp child failed\n");
 	}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202201041803.204I3UPH025672>