platforms/powerpc: Check for a powerpc platform in the probe function

Currently, we're always assuming a powerpc platform, as the powerpc
probe() function always returns true.

This change adds a check for some bits we need to work on a powerpc
platform.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

diff --git a/discover/platform-powerpc.c b/discover/platform-powerpc.c
index e2a86318f84803905c0bccc07519499410646a2b..776176956a8c9df777948728a9fcc15660998e2e 100644 (file)
--- a/discover/platform-powerpc.c
+++ b/discover/platform-powerpc.c
@@ -4,6 +4,7 @@
 #include <limits.h>
 #include <sys/types.h>
 #include <sys/wait.h>
+#include <sys/stat.h>
 
 #include <talloc/talloc.h>
 #include <list/list.h>
@@ -535,6 +536,20 @@ static int save_config(struct platform *p, struct config *config)
 static bool probe(struct platform *p, void *ctx)
 {
        struct platform_powerpc *platform;
+       struct stat statbuf;
+       int rc;
+
+       /* we need a device tree and a working nvram binary */
+       rc = stat("/proc/device-tree", &statbuf);
+       if (rc)
+               return false;
+
+       if (!S_ISDIR(statbuf.st_mode))
+               return false;
+
+       rc = process_run_simple(ctx, "nvram", "--print-config", NULL);
+       if (!WIFEXITED(rc) || WEXITSTATUS(rc) != 0)
+               return false;
 
        platform = talloc(ctx, struct platform_powerpc);
        list_init(&platform->params);