On Mon, Apr 15, 2019 at 05:10:43PM +0530, Yash Shah wrote: > This initial ver of EDAC driver supports: > - ECC event monitoring and reporting through the EDAC framework for SiFive > L2 cache controller. > > This driver registers for notifier events from the L2 cache controller > driver (arch/riscv/mm/sifive_l2_cache.c) for L2 ECC events > > Signed-off-by: Yash Shah <yash.shah@xxxxxxxxxx> > --- > arch/riscv/Kconfig | 1 + > drivers/edac/Kconfig | 6 +++ > drivers/edac/Makefile | 1 + > drivers/edac/sifive_edac.c | 121 +++++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 129 insertions(+) > create mode 100644 drivers/edac/sifive_edac.c Please add a MAINTAINERS entry like the other EDAC drivers do so that you can get CCed on patches/bug reports. The rest looks ok to me. I'd still like for James to have a look too, when he gets to it. Thx. -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.