Commit 6936c625 authored by Kumar Gala's avatar Kumar Gala

[POWERPC] 85xx: Cleaning up machine probing

Cleaned up the probing functionality to be more consistent across all 85xx
boards and actually check to see if we should be running on a given board.
Signed-off-by: default avatarKumar Gala <galak@kernel.crashing.org>
parent 52094879
...@@ -74,7 +74,6 @@ static void __init mpc8568_mds_setup_arch(void) ...@@ -74,7 +74,6 @@ static void __init mpc8568_mds_setup_arch(void)
struct device_node *np; struct device_node *np;
static u8 *bcsr_regs = NULL; static u8 *bcsr_regs = NULL;
if (ppc_md.progress) if (ppc_md.progress)
ppc_md.progress("mpc8568_mds_setup_arch()", 0); ppc_md.progress("mpc8568_mds_setup_arch()", 0);
...@@ -206,7 +205,6 @@ static void __init mpc8568_mds_pic_init(void) ...@@ -206,7 +205,6 @@ static void __init mpc8568_mds_pic_init(void)
mpic_init(mpic); mpic_init(mpic);
#ifdef CONFIG_QUICC_ENGINE #ifdef CONFIG_QUICC_ENGINE
np = of_find_node_by_type(NULL, "qeic"); np = of_find_node_by_type(NULL, "qeic");
if (!np) if (!np)
...@@ -217,24 +215,15 @@ static void __init mpc8568_mds_pic_init(void) ...@@ -217,24 +215,15 @@ static void __init mpc8568_mds_pic_init(void)
#endif /* CONFIG_QUICC_ENGINE */ #endif /* CONFIG_QUICC_ENGINE */
} }
static int __init mpc8568_mds_probe(void) static int __init mpc8568_mds_probe(void)
{ {
char *model = of_get_flat_dt_prop(of_get_flat_dt_root(), unsigned long root = of_get_flat_dt_root();
"model", NULL);
if (model == NULL)
return 0;
if (strcmp(model, "MPC8568EMDS"))
return 0;
DBG("MPC8568EMDS found\n");
return 1; return of_flat_dt_is_compatible(root, "MPC85xxMDS");
} }
define_machine(mpc8568_mds) { define_machine(mpc8568_mds) {
.name = "MPC8568E MDS", .name = "MPC85xx MDS",
.probe = mpc8568_mds_probe, .probe = mpc8568_mds_probe,
.setup_arch = mpc8568_mds_setup_arch, .setup_arch = mpc8568_mds_setup_arch,
.init_IRQ = mpc8568_mds_pic_init, .init_IRQ = mpc8568_mds_pic_init,
......
...@@ -272,10 +272,9 @@ static void mpc85xx_ads_show_cpuinfo(struct seq_file *m) ...@@ -272,10 +272,9 @@ static void mpc85xx_ads_show_cpuinfo(struct seq_file *m)
*/ */
static int __init mpc85xx_ads_probe(void) static int __init mpc85xx_ads_probe(void)
{ {
/* We always match for now, eventually we should look at the flat unsigned long root = of_get_flat_dt_root();
dev tree to ensure this is the board we are suppose to run on
*/ return of_flat_dt_is_compatible(root, "MPC85xxADS");
return 1;
} }
define_machine(mpc85xx_ads) { define_machine(mpc85xx_ads) {
......
...@@ -291,11 +291,9 @@ static void mpc85xx_cds_show_cpuinfo(struct seq_file *m) ...@@ -291,11 +291,9 @@ static void mpc85xx_cds_show_cpuinfo(struct seq_file *m)
*/ */
static int __init mpc85xx_cds_probe(void) static int __init mpc85xx_cds_probe(void)
{ {
/* We always match for now, eventually we should look at unsigned long root = of_get_flat_dt_root();
* the flat dev tree to ensure this is the board we are
* supposed to run on return of_flat_dt_is_compatible(root, "MPC85xxCDS");
*/
return 1;
} }
define_machine(mpc85xx_cds) { define_machine(mpc85xx_cds) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment