[patch] cell: correctly detect systemsim host

[patch] cell: correctly detect systemsim host

[Christian Krafft]

Systemsim uses a different compatible property in the device tree.

Signed-off-by: Christian Krafft <krafft@de.ibm.com>
---

Index: linux/arch/powerpc/platforms/cell/setup.c
===================================================================
--- linux.orig/arch/powerpc/platforms/cell/setup.c
+++ linux/arch/powerpc/platforms/cell/setup.c
@@ -222,10 +222,13 @@ static int __init cell_probe(void)
  	 * more appropriate detection logic
  	 */
  	unsigned long root = of_get_flat_dt_root();
-	if (!of_flat_dt_is_compatible(root, "IBM,CPBW-1.0"))
-		return 0;
+	if (of_flat_dt_is_compatible(root, "IBM,CPBW-1.0"))
+		return 1;

-	return 1;
+	if (of_flat_dt_is_compatible(root, "IBM,CPBW-SystemSim"))
+		return 1;
+
+	return 0;
  }

  /*

Re: [Cbe-oss-dev] [patch] cell: correctly detect systemsim host

[Arnd Bergmann]

On Thursday 13 April 2006 13:41, Christian Krafft wrote:
> Systemsim uses a different compatible property in the device tree.
> 
> Signed-off-by: Christian Krafft <krafft@de.ibm.com>

Thanks.

I'm wondering if we should have that upstream, since some
more patches are currently required to run on systemsim
anyway. For now, I'll keep it in my tree.

	Arnd <><

Re: [Cbe-oss-dev] [patch] cell: correctly detect systemsim host

[Joel Schopp]

>> Systemsim uses a different compatible property in the device tree.
>>
>> Signed-off-by: Christian Krafft <krafft@de.ibm.com>
> 
> Thanks.
> 
> I'm wondering if we should have that upstream, since some
> more patches are currently required to run on systemsim
> anyway. For now, I'll keep it in my tree.

It is clean enough to go upstream and doesn't depend on anything else, 
so I say put it upstream.  That's just one less patch to carry outside 
of mainline.  No reason to punish perfectly good patches just because 
the rest of the systemsim stuff isn't ready yet.

-Joel

Acked-by: Joel Schopp <jschopp@austin.ibm.com>