Thread (16 messages) 16 messages, 4 authors, 2017-08-14
STALE3215d REVIEWED: 3 (3M)
Revisions (9)
  1. rfc [diff vs current]
  2. v1 [diff vs current]
  3. v2 current
  4. v4 [diff vs current]
  5. v5 [diff vs current]
  6. v6 [diff vs current]
  7. v7 [diff vs current]
  8. v8 [diff vs current]
  9. v9 [diff vs current]

[PATCH v2 0/9] EDAC drivers for Armada XP L2 and DDR

From: Chris.Packham@alliedtelesis.co.nz (Chris Packham)
Date: 2017-08-04 01:42:30

On 03/08/17 00:39, Jan Luebbe wrote:
This series adds drivers for the L2 cache and DDR RAM ECC functionality as
found on the MV78230/MV78x60 SoCs. I've tested these changes with the MV78460
(on a custom board with a DDR3 ECC DIMM).

Also contained in this series are an additional debugfs wrapper and devm_
helpers for edac_mc_/edac_device_ allocation and registration, which make error
handing and cleanup simpler.

Compared to the previous v1 series, the following changes have been made:
- Add the aurora-l2 register defines earlier in the series (suggested by
   Russell King and Gregory CLEMENT )
- Changed the DT vendor prefix from "arm" to "marvell" for the ecc-enable/disable
   properties on the aurora-l2 (suggested by Russell King)
- Fix some warnings reported by checkpatch

Compared to the original RFC series, the following changes have been made:
- Integrated Chris' patches for parity and ECC configuration via DT
- Merged the DDR RAM and L2 cache drivers (as requested by Borislav, analogous
   to fsl_ddr_edac.c and mpc85xx_edac.c)
- Added myself to MAINTAINERS (requested by Borislav)
- L2 cache: Track the msg size and use snprintf (review comment by Chris)
- L2 cache: Split error injection from the check function (review comment by
   Chris)
- DDR RAM: Allow 16 bit width in addition to 32 and 64 bit (review comment by
   Chris)
Thanks for that. I'll make my life easier when I add support for the 
narrower bus-width.
- Use of_match_ptr() (review comments by Chris)
- Minor checkpatch cleanups

Chris Packham (2):
   ARM: l2x0: support parity-enable/disable on aurora
   ARM: l2x0: add marvell,ecc-enable property for aurora

Jan Luebbe (7):
   ARM: l2c: move cache-aurora-l2.h to asm/hardware
   ARM: aurora-l2: add prefix to MAX_RANGE_SIZE
   ARM: aurora-l2: add defines for parity and ECC registers
   EDAC: Add missing debugfs_create_x32 wrapper
   EDAC: Add devres helpers for
     edac_mc_alloc/edac_mc_add_mc(_with_groups)
   EDAC: Add devres helpers for
     edac_device_alloc_ctl_info/edac_device_add_device
   EDAC: Add driver for the Marvell Armada XP SDRAM and L2 cache ECC
	
Reviewed-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help