| Message ID | 20181122001142.19187-1-chris.packham@alliedtelesis.co.nz (mailing list archive) |
|---|---|
| Headers | show |
| Series | EDAC drivers for Armada XP L2 and DDR | expand |
Hi All, On 22/11/18 1:12 PM, Chris Packham wrote: > The current plan is for these to go in via the ARM tree once appropriate > Reviews/Acks have been given > > http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/525561.html Is there anything I need to be doing to move this series along? It seems to have dropped off the radar. > > This series adds drivers for the L2 cache and DDR RAM ECC functionality as > found on the MV78230/MV78x60 SoCs. Jan has tested these changes with the > MV78460 (on a custom board with a DDR3 ECC DIMM), Chris has tested these > changes with 88F6820 and 98dx3236 (both a custom boards with fixed DDR3 + ECC). > > Also contained in this series is an additional debugfs wrapper. > > Compared to the previous v6 series I've dropped the marvell,ecc-disable > property. > > Compared to the previous v5 series I've split the dt-binding documentation into > its own patch and updated armada_xp_edac.c to use a SPDX license. > > Compared to the previous v4 series I've added my s-o-b to some of Jan's > patches and rebased against v4.19.0. > > Compared to the previous v3 series, the following changes have been made: > - Use shorter names for the AURORA ECC and parity registers > - Numerous formatting changes to edac/armada_xp.c (as requested by Boris) > - Added support for Armada-38x and 98dx3236 SoCs > > 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 (4): > ARM: l2x0: support parity-enable/disable on aurora > dt-bindings: ARM: document marvell,ecc-enable binding > ARM: l2x0: add marvell,ecc-enable property for aurora > EDAC: armada_xp: Add support for more SoCs > > 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 > > .../devicetree/bindings/arm/l2c2x0.txt | 1 + > MAINTAINERS | 6 + > .../asm/hardware}/cache-aurora-l2.h | 50 +- > arch/arm/mm/cache-l2x0.c | 18 +- > drivers/edac/Kconfig | 7 + > drivers/edac/Makefile | 1 + > drivers/edac/armada_xp_edac.c | 635 ++++++++++++++++++ > drivers/edac/debugfs.c | 11 + > drivers/edac/edac_module.h | 5 + > 9 files changed, 730 insertions(+), 4 deletions(-) > rename arch/arm/{mm => include/asm/hardware}/cache-aurora-l2.h (50%) > create mode 100644 drivers/edac/armada_xp_edac.c >
Hi Borislav, Russell, This seems to have fallen off the radar. Is there anything stopping this series from being merged? On 22/11/18 1:12 PM, Chris Packham wrote: > The current plan is for these to go in via the ARM tree once appropriate > Reviews/Acks have been given > > http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/525561.html > > This series adds drivers for the L2 cache and DDR RAM ECC functionality as > found on the MV78230/MV78x60 SoCs. Jan has tested these changes with the > MV78460 (on a custom board with a DDR3 ECC DIMM), Chris has tested these > changes with 88F6820 and 98dx3236 (both a custom boards with fixed DDR3 + ECC). > > Also contained in this series is an additional debugfs wrapper. > > Compared to the previous v6 series I've dropped the marvell,ecc-disable > property. > > Compared to the previous v5 series I've split the dt-binding documentation into > its own patch and updated armada_xp_edac.c to use a SPDX license. > > Compared to the previous v4 series I've added my s-o-b to some of Jan's > patches and rebased against v4.19.0. > > Compared to the previous v3 series, the following changes have been made: > - Use shorter names for the AURORA ECC and parity registers > - Numerous formatting changes to edac/armada_xp.c (as requested by Boris) > - Added support for Armada-38x and 98dx3236 SoCs > > 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 (4): > ARM: l2x0: support parity-enable/disable on aurora > dt-bindings: ARM: document marvell,ecc-enable binding > ARM: l2x0: add marvell,ecc-enable property for aurora > EDAC: armada_xp: Add support for more SoCs > > 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 > > .../devicetree/bindings/arm/l2c2x0.txt | 1 + > MAINTAINERS | 6 + > .../asm/hardware}/cache-aurora-l2.h | 50 +- > arch/arm/mm/cache-l2x0.c | 18 +- > drivers/edac/Kconfig | 7 + > drivers/edac/Makefile | 1 + > drivers/edac/armada_xp_edac.c | 635 ++++++++++++++++++ > drivers/edac/debugfs.c | 11 + > drivers/edac/edac_module.h | 5 + > 9 files changed, 730 insertions(+), 4 deletions(-) > rename arch/arm/{mm => include/asm/hardware}/cache-aurora-l2.h (50%) > create mode 100644 drivers/edac/armada_xp_edac.c >
On Wed, Feb 27, 2019 at 08:05:45PM +0000, Chris Packham wrote: > Hi Borislav, Russell, > > This seems to have fallen off the radar. Is there anything stopping this > series from being merged? I was told this is not going through my tree and so I gave Reviewed-by's for the EDAC bits.
The current plan is for these to go in via the ARM tree once appropriate Reviews/Acks have been given http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/525561.html This series adds drivers for the L2 cache and DDR RAM ECC functionality as found on the MV78230/MV78x60 SoCs. Jan has tested these changes with the MV78460 (on a custom board with a DDR3 ECC DIMM), Chris has tested these changes with 88F6820 and 98dx3236 (both a custom boards with fixed DDR3 + ECC). Also contained in this series is an additional debugfs wrapper. Compared to the previous v6 series I've dropped the marvell,ecc-disable property. Compared to the previous v5 series I've split the dt-binding documentation into its own patch and updated armada_xp_edac.c to use a SPDX license. Compared to the previous v4 series I've added my s-o-b to some of Jan's patches and rebased against v4.19.0. Compared to the previous v3 series, the following changes have been made: - Use shorter names for the AURORA ECC and parity registers - Numerous formatting changes to edac/armada_xp.c (as requested by Boris) - Added support for Armada-38x and 98dx3236 SoCs 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 (4): ARM: l2x0: support parity-enable/disable on aurora dt-bindings: ARM: document marvell,ecc-enable binding ARM: l2x0: add marvell,ecc-enable property for aurora EDAC: armada_xp: Add support for more SoCs 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 .../devicetree/bindings/arm/l2c2x0.txt | 1 + MAINTAINERS | 6 + .../asm/hardware}/cache-aurora-l2.h | 50 +- arch/arm/mm/cache-l2x0.c | 18 +- drivers/edac/Kconfig | 7 + drivers/edac/Makefile | 1 + drivers/edac/armada_xp_edac.c | 635 ++++++++++++++++++ drivers/edac/debugfs.c | 11 + drivers/edac/edac_module.h | 5 + 9 files changed, 730 insertions(+), 4 deletions(-) rename arch/arm/{mm => include/asm/hardware}/cache-aurora-l2.h (50%) create mode 100644 drivers/edac/armada_xp_edac.c