Skip to content

Commit

Permalink
edac, highbank: Improve and unify naming
Browse files Browse the repository at this point in the history
Assinging correct names of the 'hb_mc_edac' and 'hb_l2_edac' edac
modules for module, controller and device. Reported values for
Highbank in dmesg are now:

 EDAC MC0: Giving out device to module hb_mc_edac controller
 calxeda,hb-ddr-ctrl: DEV fff00000.memory-controller (INTERRUPT)

 EDAC DEVICE0: Giving out device to module hb_l2_edac controller
 calxeda,hb-sregs-l2-ecc: DEV fff3c200.sregs (INTERRUPT)

Signed-off-by: Robert Richter <robert.richter@linaro.org>
Acked-by: Rob Herring <rob.herring@calxeda.com>
Signed-off-by: Robert Richter <rric@kernel.org>
  • Loading branch information
Robert Richter authored and Robert Richter committed Nov 4, 2013
1 parent 0ec8579 commit 41ec0e8
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 9 deletions.
17 changes: 10 additions & 7 deletions drivers/edac/highbank_l2_edac.c
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,15 @@ static irqreturn_t highbank_l2_err_handler(int irq, void *dev_id)
return IRQ_HANDLED;
}

static const struct of_device_id hb_l2_err_of_match[] = {
{ .compatible = "calxeda,hb-sregs-l2-ecc", },
{},
};
MODULE_DEVICE_TABLE(of, hb_l2_err_of_match);

static int highbank_l2_err_probe(struct platform_device *pdev)
{
const struct of_device_id *id;
struct edac_device_ctl_info *dci;
struct hb_l2_drvdata *drvdata;
struct resource *r;
Expand Down Expand Up @@ -90,7 +97,9 @@ static int highbank_l2_err_probe(struct platform_device *pdev)
goto err;
}

dci->mod_name = dev_name(&pdev->dev);
id = of_match_device(hb_l2_err_of_match, &pdev->dev);
dci->mod_name = pdev->dev.driver->name;
dci->ctl_name = id ? id->compatible : "unknown";
dci->dev_name = dev_name(&pdev->dev);

if (edac_device_add_device(dci))
Expand Down Expand Up @@ -129,12 +138,6 @@ static int highbank_l2_err_remove(struct platform_device *pdev)
return 0;
}

static const struct of_device_id hb_l2_err_of_match[] = {
{ .compatible = "calxeda,hb-sregs-l2-ecc", },
{},
};
MODULE_DEVICE_TABLE(of, hb_l2_err_of_match);

static struct platform_driver highbank_l2_edac_driver = {
.probe = highbank_l2_err_probe,
.remove = highbank_l2_err_remove,
Expand Down
5 changes: 3 additions & 2 deletions drivers/edac/highbank_mc_edac.c
Original file line number Diff line number Diff line change
Expand Up @@ -233,9 +233,10 @@ static int highbank_mc_probe(struct platform_device *pdev)
mci->mtype_cap = MEM_FLAG_DDR3;
mci->edac_ctl_cap = EDAC_FLAG_NONE | EDAC_FLAG_SECDED;
mci->edac_cap = EDAC_FLAG_SECDED;
mci->mod_name = dev_name(&pdev->dev);
mci->mod_name = pdev->dev.driver->name;
mci->mod_ver = "1";
mci->ctl_name = dev_name(&pdev->dev);
mci->ctl_name = id->compatible;
mci->dev_name = dev_name(&pdev->dev);
mci->scrub_mode = SCRUB_SW_SRC;

/* Only a single 4GB DIMM is supported */
Expand Down

0 comments on commit 41ec0e8

Please sign in to comment.