Bug 166011

Summary: kudzu assumes all 'sound' devices in device tree are snd-powermac
Product: [Fedora] Fedora Reporter: David Woodhouse <dwmw2>
Component: kudzuAssignee: Bill Nottingham <notting>
Status: CLOSED RAWHIDE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.1.120-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-08-19 07:18:33 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description David Woodhouse 2005-08-15 19:09:20 UTC
Patch says it all...

--- kudzu-1.1.116.2/macio.c~	2005-04-25 23:09:53.000000000 +0100
+++ kudzu-1.1.116.2/macio.c	2005-08-15 11:13:43.000000000 +0100
@@ -153,7 +153,7 @@ struct device *macioProbe( enum deviceCl
 	// Supported
 	for (n = list->result->next; n != list->result; n = n->next)
 	{
-		if (n->path)
+		if (n->path && strstr(n->path, "mac-io"))
 			ret = 1;
 	}

Comment 1 Bill Nottingham 2005-08-19 07:18:33 UTC
Thanks, added to CVS, will be in 1.1.120-1.

Comment 2 David Woodhouse 2005-08-19 08:52:13 UTC
Thanks. Please could you do an initscripts rebuild some time to pull it in? The
kmodule utility will still be loading the wrong modules until that happens.

Actually, could initscripts use a dynamically linked libkudzu to avoid the need
for  such rebuilds?



Comment 3 Bill Nottingham 2005-08-19 21:23:57 UTC
Well, there is no dynamic libkudzu, so, not really. (It's not as practical to
have one, since it depends on other static-only libraries, such as libpci.)