Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=336c3c2ec7e24bdf01c8f0c311ac7081b1f73d72
Commit:     336c3c2ec7e24bdf01c8f0c311ac7081b1f73d72
Parent:     6624b35d6250632497c5943faa2ee2a91344d91c
Author:     Kumar Gala <[EMAIL PROTECTED]>
AuthorDate: Sat Feb 17 09:10:44 2007 -0600
Committer:  Kumar Gala <[EMAIL PROTECTED]>
CommitDate: Sat Feb 17 09:10:44 2007 -0600

    [POWERPC] 83xx: Cleaning up machine probing and board initcalls
    
    Cleaned up the probing functionality to be more consistent across all 83xx
    boards and added machine_is() protection around board initcalls to ensure 
they
    only do something if we are actually running on that board.
    
    Additionally, removed some dead code on mpc832x_mds.
    
    Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
 arch/powerpc/platforms/83xx/mpc8313_rdb.c |   11 ++---------
 arch/powerpc/platforms/83xx/mpc832x_mds.c |   23 ++++++++---------------
 arch/powerpc/platforms/83xx/mpc834x_itx.c |    7 +++----
 arch/powerpc/platforms/83xx/mpc834x_mds.c |   10 ++++++----
 arch/powerpc/platforms/83xx/mpc8360e_pb.c |   17 ++++++++---------
 5 files changed, 27 insertions(+), 41 deletions(-)

diff --git a/arch/powerpc/platforms/83xx/mpc8313_rdb.c 
b/arch/powerpc/platforms/83xx/mpc8313_rdb.c
index c3b98c3..32e9e94 100644
--- a/arch/powerpc/platforms/83xx/mpc8313_rdb.c
+++ b/arch/powerpc/platforms/83xx/mpc8313_rdb.c
@@ -74,16 +74,9 @@ void __init mpc8313_rdb_init_IRQ(void)
  */
 static int __init mpc8313_rdb_probe(void)
 {
-       char *model = of_get_flat_dt_prop(of_get_flat_dt_root(),
-                                         "model", NULL);
-       if (model == NULL)
-               return 0;
-       if (strcmp(model, "MPC8313ERDB"))
-               return 0;
+        unsigned long root = of_get_flat_dt_root();
 
-       DBG("MPC8313 RDB found\n");
-
-       return 1;
+        return of_flat_dt_is_compatible(root, "MPC8313ERDB");
 }
 
 define_machine(mpc8313_rdb) {
diff --git a/arch/powerpc/platforms/83xx/mpc832x_mds.c 
b/arch/powerpc/platforms/83xx/mpc832x_mds.c
index 3ecb55f..c6bfe72 100644
--- a/arch/powerpc/platforms/83xx/mpc832x_mds.c
+++ b/arch/powerpc/platforms/83xx/mpc832x_mds.c
@@ -57,11 +57,6 @@ unsigned long isa_mem_base = 0;
 
 static u8 *bcsr_regs = NULL;
 
-u8 *get_bcsr(void)
-{
-       return bcsr_regs;
-}
-
 /* ************************************************************************
  *
  * Setup the architecture
@@ -140,6 +135,9 @@ static int __init mpc832x_declare_of_platform_devices(void)
 {
        struct device_node *np;
 
+       if (!machine_is(mpc832x_mds))
+               return 0;
+
        for (np = NULL; (np = of_find_compatible_node(np, "network",
                                        "ucc_geth")) != NULL;) {
                int ucc_num;
@@ -189,6 +187,9 @@ static int __init mpc832x_rtc_hookup(void)
 {
        struct timespec tv;
 
+       if (!machine_is(mpc832x_mds))
+               return 0;
+
        ppc_md.get_rtc_time = ds1374_get_rtc_time;
        ppc_md.set_rtc_time = ds1374_set_rtc_time;
 
@@ -207,17 +208,9 @@ late_initcall(mpc832x_rtc_hookup);
  */
 static int __init mpc832x_sys_probe(void)
 {
-       char *model = of_get_flat_dt_prop(of_get_flat_dt_root(),
-                                         "model", NULL);
-
-       if (model == NULL)
-               return 0;
-       if (strcmp(model, "MPC8323EMDS"))
-               return 0;
-
-       DBG("%s found\n", model);
+        unsigned long root = of_get_flat_dt_root();
 
-       return 1;
+        return of_flat_dt_is_compatible(root, "MPC832xMDS");
 }
 
 define_machine(mpc832x_mds) {
diff --git a/arch/powerpc/platforms/83xx/mpc834x_itx.c 
b/arch/powerpc/platforms/83xx/mpc834x_itx.c
index 443a317..a8f66fb 100644
--- a/arch/powerpc/platforms/83xx/mpc834x_itx.c
+++ b/arch/powerpc/platforms/83xx/mpc834x_itx.c
@@ -100,10 +100,9 @@ static void __init mpc834x_itx_init_IRQ(void)
  */
 static int __init mpc834x_itx_probe(void)
 {
-       /* We always match for now, eventually we should look at the flat
-          dev tree to ensure this is the board we are suppose to run on
-       */
-       return 1;
+        unsigned long root = of_get_flat_dt_root();
+
+        return of_flat_dt_is_compatible(root, "MPC834xMITX");
 }
 
 define_machine(mpc834x_itx) {
diff --git a/arch/powerpc/platforms/83xx/mpc834x_mds.c 
b/arch/powerpc/platforms/83xx/mpc834x_mds.c
index d2736da..9fd9adf 100644
--- a/arch/powerpc/platforms/83xx/mpc834x_mds.c
+++ b/arch/powerpc/platforms/83xx/mpc834x_mds.c
@@ -176,6 +176,9 @@ static int __init mpc834x_rtc_hookup(void)
 {
        struct timespec tv;
 
+       if (!machine_is(mpc834x_mds))
+               return 0;
+
        ppc_md.get_rtc_time = ds1374_get_rtc_time;
        ppc_md.set_rtc_time = ds1374_set_rtc_time;
 
@@ -194,10 +197,9 @@ late_initcall(mpc834x_rtc_hookup);
  */
 static int __init mpc834x_mds_probe(void)
 {
-       /* We always match for now, eventually we should look at the flat
-          dev tree to ensure this is the board we are suppose to run on
-       */
-       return 1;
+        unsigned long root = of_get_flat_dt_root();
+
+        return of_flat_dt_is_compatible(root, "MPC834xMDS");
 }
 
 define_machine(mpc834x_mds) {
diff --git a/arch/powerpc/platforms/83xx/mpc8360e_pb.c 
b/arch/powerpc/platforms/83xx/mpc8360e_pb.c
index ccce2f9..76fcb5b 100644
--- a/arch/powerpc/platforms/83xx/mpc8360e_pb.c
+++ b/arch/powerpc/platforms/83xx/mpc8360e_pb.c
@@ -145,6 +145,9 @@ static int __init mpc8360_declare_of_platform_devices(void)
 {
        struct device_node *np;
 
+       if (!machine_is(mpc8360_sys))
+               return 0;
+
        for (np = NULL; (np = of_find_compatible_node(np, "network",
                                        "ucc_geth")) != NULL;) {
                int ucc_num;
@@ -194,6 +197,9 @@ static int __init mpc8360_rtc_hookup(void)
 {
        struct timespec tv;
 
+       if (!machine_is(mpc8360_sys))
+               return 0;
+
        ppc_md.get_rtc_time = ds1374_get_rtc_time;
        ppc_md.set_rtc_time = ds1374_set_rtc_time;
 
@@ -212,16 +218,9 @@ late_initcall(mpc8360_rtc_hookup);
  */
 static int __init mpc8360_sys_probe(void)
 {
-       char *model = of_get_flat_dt_prop(of_get_flat_dt_root(),
-                                         "model", NULL);
-       if (model == NULL)
-               return 0;
-       if (strcmp(model, "MPC8360EPB"))
-               return 0;
-
-       DBG("MPC8360EMDS-PB found\n");
+        unsigned long root = of_get_flat_dt_root();
 
-       return 1;
+        return of_flat_dt_is_compatible(root, "MPC836xMDS");
 }
 
 define_machine(mpc8360_sys) {
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to