Commit 2b81ec69 authored by Shawn Guo's avatar Shawn Guo Committed by Mark Brown

ASoC: fsl: check property 'compatible' for the machine name

Check /compatible rather than /model to determine the machine name.
The p1022ds older device trees get a different /model from the new
ones, while /compatible is consistent there, so checking /compatible
will save the bother of detecting older p1022ds device trees.
Signed-off-by: default avatarShawn Guo <shawn.guo@linaro.org>
Acked-by: default avatarTimur Tabi <timur@freescale.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 5b596483
...@@ -716,12 +716,12 @@ static int __devinit fsl_ssi_probe(struct platform_device *pdev) ...@@ -716,12 +716,12 @@ static int __devinit fsl_ssi_probe(struct platform_device *pdev)
} }
/* Trigger the machine driver's probe function. The platform driver /* Trigger the machine driver's probe function. The platform driver
* name of the machine driver is taken from the /model property of the * name of the machine driver is taken from /compatible property of the
* device tree. We also pass the address of the CPU DAI driver * device tree. We also pass the address of the CPU DAI driver
* structure. * structure.
*/ */
sprop = of_get_property(of_find_node_by_path("/"), "model", NULL); sprop = of_get_property(of_find_node_by_path("/"), "compatible", NULL);
/* Sometimes the model name has a "fsl," prefix, so we strip that. */ /* Sometimes the compatible name has a "fsl," prefix, so we strip it. */
p = strrchr(sprop, ','); p = strrchr(sprop, ',');
if (p) if (p)
sprop = p + 1; sprop = p + 1;
......
...@@ -546,7 +546,7 @@ static struct platform_driver mpc8610_hpcd_driver = { ...@@ -546,7 +546,7 @@ static struct platform_driver mpc8610_hpcd_driver = {
.probe = mpc8610_hpcd_probe, .probe = mpc8610_hpcd_probe,
.remove = __devexit_p(mpc8610_hpcd_remove), .remove = __devexit_p(mpc8610_hpcd_remove),
.driver = { .driver = {
/* The name must match the 'model' property in the device tree, /* The name must match 'compatible' property in the device tree,
* in lowercase letters. * in lowercase letters.
*/ */
.name = "snd-soc-mpc8610hpcd", .name = "snd-soc-mpc8610hpcd",
......
...@@ -543,6 +543,11 @@ static struct platform_driver p1022_ds_driver = { ...@@ -543,6 +543,11 @@ static struct platform_driver p1022_ds_driver = {
.probe = p1022_ds_probe, .probe = p1022_ds_probe,
.remove = __devexit_p(p1022_ds_remove), .remove = __devexit_p(p1022_ds_remove),
.driver = { .driver = {
/*
* The name must match 'compatible' property in the device tree,
* in lowercase letters.
*/
.name = "snd-soc-p1022ds",
.owner = THIS_MODULE, .owner = THIS_MODULE,
}, },
}; };
...@@ -556,33 +561,6 @@ static int __init p1022_ds_init(void) ...@@ -556,33 +561,6 @@ static int __init p1022_ds_init(void)
{ {
struct device_node *guts_np; struct device_node *guts_np;
struct resource res; struct resource res;
const char *sprop;
/*
* Check if we're actually running on a P1022DS. Older device trees
* have a model of "fsl,P1022" and newer ones use "fsl,P1022DS", so we
* need to support both. The SSI driver uses that property to link to
* the machine driver, so have to match it.
*/
sprop = of_get_property(of_find_node_by_path("/"), "model", NULL);
if (!sprop) {
pr_err("snd-soc-p1022ds: missing /model node");
return -ENODEV;
}
pr_debug("snd-soc-p1022ds: board model name is %s\n", sprop);
/*
* The name of this board, taken from the device tree. Normally, this is a*
* fixed string, but some P1022DS device trees have a /model property of
* "fsl,P1022", and others have "fsl,P1022DS".
*/
if (strcasecmp(sprop, "fsl,p1022ds") == 0)
p1022_ds_driver.driver.name = "snd-soc-p1022ds";
else if (strcasecmp(sprop, "fsl,p1022") == 0)
p1022_ds_driver.driver.name = "snd-soc-p1022";
else
return -ENODEV;
/* Get the physical address of the global utilities registers */ /* Get the physical address of the global utilities registers */
guts_np = of_find_compatible_node(NULL, NULL, "fsl,p1022-guts"); guts_np = of_find_compatible_node(NULL, NULL, "fsl,p1022-guts");
......
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