Hi,
>> +static struct edac_dev_sysfs_attribute xgene_edac_l3_sysfs_attributes[] = {
>> + { .attr = {
>> + .name = "inject_ctrl",
>> + .mode = (S_IRUGO | S_IWUSR)
>> + },
>> + .show = xgene_edac_l3_inject_ctrl_show,
>> + .store = xgene_edac_l3_inject_ctrl_store },
>> +
>> + /* End of list */
>> + { .attr = {.name = NULL } }
>> +};
>
> Why are those sysfs nodes? Didn't we say that inject nodes should be in
> debugfs?
>
> So I'm going to stop looking here and wait for you to do the same
> changes to L3 and SOC as for the rest of the driver. Go through what
> just went upstream and do the same changes to that patch instead of
> blindly resending the original version.
>
> It is not a competition who gets their stuff upstream first, ok?!
Sorry about this and being a bit careless here.
>
>> +
>> + if (edac_op_state == EDAC_OPSTATE_POLL)
>> + edac_dev->edac_check = xgene_edac_l3_check;
>> +
>> + edac_dev->sysfs_attributes = xgene_edac_l3_sysfs_attributes;
>> +
>> + rc = edac_device_add_device(edac_dev);
>> + if (rc > 0) {
>> + dev_err(edac->dev, "failed edac_device_add_device()\n");
>> + rc = -ENOMEM;
>> + goto err1;
>> + }
>> +
>> + if (edac_op_state == EDAC_OPSTATE_INT)
>> + edac_dev->op_state = OP_RUNNING_INTERRUPT;
>> +
>> + list_add(&ctx->next, &edac->l3s);
>> +
>> + xgene_edac_l3_hw_init(edac_dev, 1);
>
> Shouldn't you init the hw regs *before* you add it to the list of l3s
> and *not* after?
I want to make sure that the node is added in case there is an pending
interrupt. Otherwise, it will not get cleared and will continuously
generates interrupt.
-Loc
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
