Thread (6 messages) 6 messages, 3 authors, 2025-02-27

Re: Missing '#size-cells' in /ht@0,f2000000/pci@8/mac-io@7/i2s@0 on G5/G4 (kernel 6.13-rc3)

From: Erhard Furtner <hidden>
Date: 2025-02-27 14:54:50

On Thu, 27 Feb 2025 09:00:06 +0530
Madhavan Srinivasan [off-list ref] wrote:
On 2/27/25 3:44 AM, Erhard Furtner wrote:
quoted
On Mon, 23 Dec 2024 23:58:26 +1100
Michael Ellerman [off-list ref] wrote:
  
quoted
Erhard Furtner [off-list ref] writes:  
quoted
Greetings!

I am aware there is commit cf89c9434af122f28a3552e6f9cc5158c33ce50a
(already merged in v6.13-rc3) which fixes some of the "Missing
'#size-cells' errors" but there is more:

------------[ cut here ]------------
hub 3-0:1.0: USB hub found
Missing '#size-cells' in /ht@0,f2000000/pci@8/mac-io@7/i2s@0    
Something like below might fix it.

cheers

diff --git a/arch/powerpc/kernel/prom_init.c b/arch/powerpc/kernel/prom_init.c
index 8e776ba39497..462ee36ef891 100644
--- a/arch/powerpc/kernel/prom_init.c
+++ b/arch/powerpc/kernel/prom_init.c
@@ -2898,11 +2898,11 @@ static void __init fixup_device_tree_pmac(void)
 	char type[8];
 	phandle node;
 
-	// Some pmacs are missing #size-cells on escc nodes
+	// Some pmacs are missing #size-cells on escc & i2s nodes
 	for (node = 0; prom_next_node(&node); ) {
 		type[0] = '\0';
 		prom_getprop(node, "device_type", type, sizeof(type));
-		if (prom_strcmp(type, "escc"))
+		if (prom_strcmp(type, "escc") && prom_strcmp(type, "i2s"))
 			continue;
 
 		if (prom_getproplen(node, "#size-cells") != PROM_ERROR)
 
The fix works on my PowerMac G4 DP. And there is also https://patchwork.ozlabs.org/project/linuxppc-dev/patch/875xmizl6a.fsf@igel.home/ which seems to be the same patch.
  
Sorry, missed it. I will add the above patch to -next.

Maddy
No problem, that can happen. Thanks!

It'll have to go in for v6.13 too. v6.12 was not affected.

Regards,
Erhard
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help