| Message ID | 1551121006-4657-1-git-send-email-thor.thayer@linux.intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Independent SOCFPGA SDRAM EDAC config | expand |
On Mon, Feb 25, 2019 at 12:56:44PM -0600, thor.thayer@linux.intel.com wrote: > From: Thor Thayer <thor.thayer@linux.intel.com> > > Most users want EDAC support so make it the default. > > SOCFPGA SDRAM EDAC reporting was enabled by the parent EDAC > config (CONFIG_ALTERA_EDAC) since initial customers always > wanted SDRAM EDAC enabled. > There are cases where the SDRAM needs to be disabled while > the other block EDACs remain enabled. > This patch set 1) splits out the SDRAM EDAC into a separate > config and 2) enables all the EDAC blocks by default for > 32 bit SOCFPGA. > > V2 Changes > Rebase socfpga_defconfig against arm-soc/arm/defconfig > > Thor Thayer (2): > EDAC, altera: Add separate SDRAM EDAC config > ARM: socfpga_defconfig: enable EDAC by default Am I supposed to take the second patch too or is it going through Dinh's tree?
Hi Boris, On 2/25/19 2:42 PM, Borislav Petkov wrote: > On Mon, Feb 25, 2019 at 12:56:44PM -0600, thor.thayer@linux.intel.com wrote: >> From: Thor Thayer <thor.thayer@linux.intel.com> >> >> Most users want EDAC support so make it the default. >> >> SOCFPGA SDRAM EDAC reporting was enabled by the parent EDAC >> config (CONFIG_ALTERA_EDAC) since initial customers always >> wanted SDRAM EDAC enabled. >> There are cases where the SDRAM needs to be disabled while >> the other block EDACs remain enabled. >> This patch set 1) splits out the SDRAM EDAC into a separate >> config and 2) enables all the EDAC blocks by default for >> 32 bit SOCFPGA. >> >> V2 Changes >> Rebase socfpga_defconfig against arm-soc/arm/defconfig >> >> Thor Thayer (2): >> EDAC, altera: Add separate SDRAM EDAC config >> ARM: socfpga_defconfig: enable EDAC by default > > Am I supposed to take the second patch too or is it going through Dinh's > tree? > If you don't mind, I'll take the defconfig patch to avoid any conflicts. Thanks, Dinh
On Tue, Feb 26, 2019 at 08:59:43AM -0600, Dinh Nguyen wrote:
> If you don't mind, I'll take the defconfig patch to avoid any conflicts.
Ok, thanks. Applied the first one.
From: Thor Thayer <thor.thayer@linux.intel.com> Most users want EDAC support so make it the default. SOCFPGA SDRAM EDAC reporting was enabled by the parent EDAC config (CONFIG_ALTERA_EDAC) since initial customers always wanted SDRAM EDAC enabled. There are cases where the SDRAM needs to be disabled while the other block EDACs remain enabled. This patch set 1) splits out the SDRAM EDAC into a separate config and 2) enables all the EDAC blocks by default for 32 bit SOCFPGA. V2 Changes Rebase socfpga_defconfig against arm-soc/arm/defconfig Thor Thayer (2): EDAC, altera: Add separate SDRAM EDAC config ARM: socfpga_defconfig: enable EDAC by default arch/arm/configs/socfpga_defconfig | 12 +++++++ drivers/edac/Kconfig | 14 ++++++-- drivers/edac/altera_edac.c | 67 ++++++++++++++++++++------------------ 3 files changed, 59 insertions(+), 34 deletions(-)