[lkml]   [2019]   [Mar]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 2/2] sifive: edac: Add EDAC driver for Sifive l2 Cache Controller
On Tue, 12 Mar 2019, Borislav Petkov wrote:

> Please no EDAC drivers for a single functional unit with RAS
> capabilities. Rather, a sifive_edac or riscv_edac driver which covers
> the whole platform or even architecture and contains support for all the
> RAS functionality there. See altera_edac, for example.

Looking at the Synopsys, Highbank, PowerPC 4xx, and TI EDAC drivers, all
of those are clearly for IP block error management, rather than platform
error management. Has the upstream guidance changed since those drivers
were merged?

The core issue for us is that we don't have a generalized "ECC management"
IP block. And I would just as soon not fake one in the DT data, since the
general DT guidance is that the data in DT is meant to describe the actual

Would it make more sense to put this driver outside of drivers/edac?

- Paul

 \ /
  Last update: 2019-03-25 01:17    [W:0.072 / U:0.872 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site