All of lore.kernel.org
 help / color / mirror / Atom feed
From: jlu@pengutronix.de (Jan Luebbe)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/7] EDAC drivers for Armada XP L2 and DDR
Date: Fri, 10 Nov 2017 10:03:01 +0100	[thread overview]
Message-ID: <20171110090308.21562-1-jlu@pengutronix.de> (raw)

Hi Boris,

finally I've found some time to address your feedback and resend this. On the
previous version, Russell said that he would merge the entire series once the
EDAC bits have been reviewed.

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 is an additional debugfs wrapper.


Compared to the previous v2 series, the following changes have been made:
- Allocate EDAC structures later during probing and drop devres support
  patches (requested by Boris)
- Make drvdata->width usage consistent according to the comment (suggested by
  Chris)

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 Boris, analogous
  to fsl_ddr_edac.c and mpc85xx_edac.c)
- Added myself to MAINTAINERS (requested by Boris)
- 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)
- 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 (5):
  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 driver for the Marvell Armada XP SDRAM and L2 cache ECC

 Documentation/devicetree/bindings/arm/l2c2x0.txt |   2 +
 MAINTAINERS                                      |   6 +
 arch/arm/include/asm/hardware/cache-aurora-l2.h  | 103 ++++
 arch/arm/mm/cache-aurora-l2.h                    |  55 --
 arch/arm/mm/cache-l2x0.c                         |  20 +-
 drivers/edac/Kconfig                             |   7 +
 drivers/edac/Makefile                            |   1 +
 drivers/edac/armada_xp_edac.c                    | 658 +++++++++++++++++++++++
 drivers/edac/debugfs.c                           |  11 +
 drivers/edac/edac_module.h                       |   5 +
 10 files changed, 810 insertions(+), 58 deletions(-)
 create mode 100644 arch/arm/include/asm/hardware/cache-aurora-l2.h
 delete mode 100644 arch/arm/mm/cache-aurora-l2.h
 create mode 100644 drivers/edac/armada_xp_edac.c

-- 
2.11.0

             reply	other threads:[~2017-11-10  9:03 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-10  9:03 Jan Luebbe [this message]
2017-11-10  9:03 [v3,1/7] ARM: l2c: move cache-aurora-l2.h to asm/hardware Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 1/7] " Jan Luebbe
2017-11-10  9:03 [v3,2/7] ARM: aurora-l2: add prefix to MAX_RANGE_SIZE Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 2/7] " Jan Luebbe
2017-11-10  9:03 [v3,3/7] ARM: aurora-l2: add defines for parity and ECC registers Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 3/7] " Jan Luebbe
2017-11-10  9:03 [v3,4/7] ARM: l2x0: support parity-enable/disable on aurora Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 4/7] " Jan Luebbe
2017-11-10  9:03 [v3,5/7] ARM: l2x0: add marvell,ecc-enable property for aurora Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 5/7] " Jan Luebbe
2017-11-10  9:03 [v3,6/7] EDAC: Add missing debugfs_create_x32 wrapper Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 6/7] " Jan Luebbe
2017-11-10  9:03 [v3,7/7] EDAC: Add driver for the Marvell Armada XP SDRAM and L2 cache ECC Jan Lübbe
2017-11-10  9:03 ` [PATCH v3 7/7] " Jan Luebbe
2017-11-12 20:15 [v3,7/7] " Chris Packham
2017-11-12 20:15 ` [PATCH v3 7/7] " Chris Packham
2017-11-16 19:31 [v3,7/7] " Borislav Petkov
2017-11-16 19:31 ` [PATCH v3 7/7] " Borislav Petkov
2017-11-16 19:32 [v3,6/7] EDAC: Add missing debugfs_create_x32 wrapper Borislav Petkov
2017-11-16 19:32 ` [PATCH v3 6/7] " Borislav Petkov
2017-11-16 21:05 [v3,7/7] EDAC: Add driver for the Marvell Armada XP SDRAM and L2 cache ECC Andrew Lunn
2017-11-16 21:05 ` [PATCH v3 7/7] " Andrew Lunn
2018-01-08 22:00 [v3,7/7] " Chris Packham
2018-01-08 22:00 ` [PATCH v3 7/7] " Chris Packham
2018-01-09  9:18 [v3,7/7] " Jan Lübbe
2018-01-09  9:18 ` [PATCH v3 7/7] " Jan Lübbe
2018-01-10 21:54 [v3,7/7] " Chris Packham
2018-01-10 21:54 ` [PATCH v3 7/7] " Chris Packham
2018-01-10 23:05 [v3,7/7] " Chris Packham
2018-01-10 23:05 ` [PATCH v3 7/7] " Chris Packham
2018-01-11 19:28 [v3,7/7] " Borislav Petkov
2018-01-11 19:28 ` [PATCH v3 7/7] " Borislav Petkov
2018-01-11 19:31 [v3,7/7] " Borislav Petkov
2018-01-11 19:31 ` [PATCH v3 7/7] " Borislav Petkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171110090308.21562-1-jlu@pengutronix.de \
    --to=jlu@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.