Messages in this thread | | | Date | Tue, 5 Jan 2016 22:09:24 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCHv7] EDAC, altera: Add Altera L2 Cache and OCRAM EDAC Support |
| |
On Tue, Jan 05, 2016 at 02:37:48PM -0600, Thor Thayer wrote: > The CONFIG_EDAC_ALTERA_MC is a little confusing because it refers to the > Memory Controller (SDRAM) and uses that in the menu string(Altera SDRAM > Memory Controller EDAC). > > Would it be confusing to rename this CONFIG_EDAC_ALTERA, update the SDRAM > code to check the ECC Enable bit instead of this config option and update > the string in the menu?
Sounds good to me.
> I'll keep this as one file and implement your suggested changes and > resubmit.
Thanks.
-- Regards/Gruss, Boris.
ECO tip #101: Trim your mails when you reply.
| |