Skip to content

Commit

Permalink
mfd: vexpress-sysreg: Don't skip initialization on probe
Browse files Browse the repository at this point in the history
The vexpress-sysreg driver does not have to be initialized
early, when the platform doesn't require this. Unfortunately
in such case it wasn't initialized correctly - master site
lookup and config bridge registration were missing. Fixed now.

Signed-off-by: Pawel Moll <pawel.moll@arm.com>
  • Loading branch information
Pawel Moll committed Jan 24, 2013
1 parent ab838bc commit 5266629
Showing 1 changed file with 20 additions and 12 deletions.
32 changes: 20 additions & 12 deletions drivers/mfd/vexpress-sysreg.c
Original file line number Diff line number Diff line change
Expand Up @@ -313,19 +313,11 @@ static void vexpress_sysreg_config_complete(unsigned long data)
}


void __init vexpress_sysreg_early_init(void __iomem *base)
void __init vexpress_sysreg_setup(struct device_node *node)
{
struct device_node *node = of_find_compatible_node(NULL, NULL,
"arm,vexpress-sysreg");

if (node)
base = of_iomap(node, 0);

if (WARN_ON(!base))
if (WARN_ON(!vexpress_sysreg_base))
return;

vexpress_sysreg_base = base;

if (readl(vexpress_sysreg_base + SYS_MISC) & SYS_MISC_MASTERSITE)
vexpress_master_site = VEXPRESS_SITE_DB2;
else
Expand All @@ -336,9 +328,23 @@ void __init vexpress_sysreg_early_init(void __iomem *base)
WARN_ON(!vexpress_sysreg_config_bridge);
}

void __init vexpress_sysreg_early_init(void __iomem *base)
{
vexpress_sysreg_base = base;
vexpress_sysreg_setup(NULL);
}

void __init vexpress_sysreg_of_early_init(void)
{
vexpress_sysreg_early_init(NULL);
struct device_node *node = of_find_compatible_node(NULL, NULL,
"arm,vexpress-sysreg");

if (node) {
vexpress_sysreg_base = of_iomap(node, 0);
vexpress_sysreg_setup(node);
} else {
pr_info("vexpress-sysreg: No Device Tree node found.");
}
}


Expand Down Expand Up @@ -426,9 +432,11 @@ static int vexpress_sysreg_probe(struct platform_device *pdev)
return -EBUSY;
}

if (!vexpress_sysreg_base)
if (!vexpress_sysreg_base) {
vexpress_sysreg_base = devm_ioremap(&pdev->dev, res->start,
resource_size(res));
vexpress_sysreg_setup(pdev->dev.of_node);
}

if (!vexpress_sysreg_base) {
dev_err(&pdev->dev, "Failed to obtain base address!\n");
Expand Down

0 comments on commit 5266629

Please sign in to comment.