On Tue, May 05, 2015 at 10:02:22PM -0600, Loc Ho wrote: > This patch adds support for the APM X-Gene SoC memory controller EDAC driver > for DT. > > v8: > * Change ASM_EDAC_H to __ASM_EDAC_H in file edac.h > * Add WARN_ONCE in stub function atomic_scrub > * Update DTS binding documentation (with only memory controller node) > * Temporary remove L1/L2, L3, and SoC driver code and update memory driver > code accordingly What does that mean exactly? They'll get added later? It is called now xgene_edac_mc.c. Am I to expect more xgene_edac_<functional_unit>.c submissions? What happened to building everything around the shared IRQ handler? Also, I see this SOB chain in some of the patches: Signed-off-by: Feng Kan <fkan@xxxxxxx> Signed-off-by: Loc Ho <lho@xxxxxxx> and it is not clear who did what, from looking at the chain. If you both authored the patches, you can write Originally-by: Feng [ Loc: did this and that] Signed-off-by: Loc for example. See Documentation/SubmittingPatches for more info. Thanks. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- -- 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