All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manish Narani <MNARANI@xilinx.com>
To: Manish Narani <MNARANI@xilinx.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"mark.rutland@arm.com" <mark.rutland@arm.com>,
	Michal Simek <michals@xilinx.com>, "bp@alien8.de" <bp@alien8.de>,
	"mchehab@kernel.org" <mchehab@kernel.org>,
	"amit.kucheria@linaro.org" <amit.kucheria@linaro.org>,
	"sudeep.holla@arm.com" <sudeep.holla@arm.com>,
	"leoyang.li@nxp.com" <leoyang.li@nxp.com>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>
Subject: RE: [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver
Date: Fri, 2 Nov 2018 08:38:52 +0000	[thread overview]
Message-ID: <DM6PR02MB52271A9C7D7829CA1467B657C1CF0@DM6PR02MB5227.namprd02.prod.outlook.com> (raw)
In-Reply-To: <1540447621-22870-1-git-send-email-manish.narani@xilinx.com>

Ping!


> -----Original Message-----
> From: Manish Narani [mailto:manish.narani@xilinx.com]
> Sent: Thursday, October 25, 2018 11:37 AM
> To: robh+dt@kernel.org; mark.rutland@arm.com; Michal Simek
> <michals@xilinx.com>; bp@alien8.de; mchehab@kernel.org; Manish Narani
> <MNARANI@xilinx.com>; amit.kucheria@linaro.org; sudeep.holla@arm.com;
> leoyang.li@nxp.com
> Cc: devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; linux-edac@vger.kernel.org
> Subject: [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver
> 
> This patch series enhances the current EDAC driver to support different
> platforms. This series adds support for ZynqMP DDRC controller in synopsys
> EDAC driver. This series also adds Device tree properties and relevant binding
> documentation.
> 
> Changes in v2:
> 	- Moved checking of DDR_ECC_INTR_SUPPORT from (1/4) to (3/4) as it
> is
> 	  a feature of ZynqMP DDRC
> 	- The Binding Documentation in (2/4) is modified as per the review
> 	  comments
> 
> Changes in v3:
> 	- The commit message in (2/4) is modified (Synopsys EDAC Driver -->
> 	  ZynqMP DDRC)
> 
> Changes in v4:
> 	- Updated the commit message in (1/4)
> 	- Renamed function pointer names removing 'synps_' in (1/4)
> 	- Shortened unnecessary long lines as per the review comment on (1/4)
> 
> Changes in v5:
> 	- Updated the commit message in (2/4) and (4/4).
> 	- Removed the unnecessary check for match data in probe() in (1/4)
> 	- Some Indentation changes for better readability in (1/4) and (3/4)
> 	- Removed repeated code in (3/4)
> 	- Used 'zynq' and 'zynqmp' instead of 'synps_enh_edac' in function
> names
> 
> Changes in v6:
> 	- Splitted the patches according to functionalities
> 	- Addressed code style comments from v5 review
> 	- Moved the Error Injection to CONFIG_EDAC_DEBUG mode
> 
> Changes in v7:
> 	- Included DTS patch (6/7) which was missed in v6 patch set
> 
> Changes in v8:
> 	- patch (1/7) from v7 is split in to 3 different logically different patches
> 		1. functional changes like code cleanup
> 		2. functions renaming
> 		3. comments cleanup
> 	- Added a separate patch (4) for making always successful functions as
> void
> 	- Corrected 'Too many parentheses' review comment in patch (5)
> 	- Corrected comments as per the v7 review feedback
> 	- Made dedicated functions for IRQ setup, IRQ enable and IRQ disable
> in patch (8)
> 	- Addressed review comments in patch (10)
> 
> Changes in v9:
> 	- Added check for return value of of_device_get_match_data() function
> 	  in (1/6).
> 	- From v8 the first 5 patches are removed in this series as they are
> 	  applied on:
> 
> 	https://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git/?h=edac-for-
> 4.20-synps
> 	- Updated Kconfig to check for ARCH_ZYNQMP instead of ARM64
> 
> Change in v10:
> 	- Moved the checking for ce_cnt and ue_cnt before the readl() call
> 	- Aligned arguments on the opening brace in setup_irq()
> 
> Manish Narani (6):
>   edac: synopsys: Add error handling for NULL in probe()
>   dt: bindings: Document ZynqMP DDRC in Synopsys documentation
>   edac: synopsys: Add macro defines for ZynqMP DDRC
>   edac: synopsys: Add EDAC ECC support for ZynqMP DDRC
>   arm64: zynqmp: Add DDRC node
>   edac: synopsys: Add Error Injection support for ZynqMP DDRC
> 
>  .../bindings/memory-controllers/synopsys.txt       |  27 +-
>  arch/arm64/boot/dts/xilinx/zynqmp.dtsi             |   7 +
>  drivers/edac/Kconfig                               |   2 +-
>  drivers/edac/synopsys_edac.c                       | 911 ++++++++++++++++++++-
>  4 files changed, 918 insertions(+), 29 deletions(-)
> 
> --
> 2.1.1


WARNING: multiple messages have this Message-ID (diff)
From: MNARANI@xilinx.com (Manish Narani)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver
Date: Fri, 2 Nov 2018 08:38:52 +0000	[thread overview]
Message-ID: <DM6PR02MB52271A9C7D7829CA1467B657C1CF0@DM6PR02MB5227.namprd02.prod.outlook.com> (raw)
In-Reply-To: <1540447621-22870-1-git-send-email-manish.narani@xilinx.com>

Ping!


> -----Original Message-----
> From: Manish Narani [mailto:manish.narani at xilinx.com]
> Sent: Thursday, October 25, 2018 11:37 AM
> To: robh+dt at kernel.org; mark.rutland at arm.com; Michal Simek
> <michals@xilinx.com>; bp at alien8.de; mchehab at kernel.org; Manish Narani
> <MNARANI@xilinx.com>; amit.kucheria at linaro.org; sudeep.holla at arm.com;
> leoyang.li at nxp.com
> Cc: devicetree at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; linux-edac at vger.kernel.org
> Subject: [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver
> 
> This patch series enhances the current EDAC driver to support different
> platforms. This series adds support for ZynqMP DDRC controller in synopsys
> EDAC driver. This series also adds Device tree properties and relevant binding
> documentation.
> 
> Changes in v2:
> 	- Moved checking of DDR_ECC_INTR_SUPPORT from (1/4) to (3/4) as it
> is
> 	  a feature of ZynqMP DDRC
> 	- The Binding Documentation in (2/4) is modified as per the review
> 	  comments
> 
> Changes in v3:
> 	- The commit message in (2/4) is modified (Synopsys EDAC Driver -->
> 	  ZynqMP DDRC)
> 
> Changes in v4:
> 	- Updated the commit message in (1/4)
> 	- Renamed function pointer names removing 'synps_' in (1/4)
> 	- Shortened unnecessary long lines as per the review comment on (1/4)
> 
> Changes in v5:
> 	- Updated the commit message in (2/4) and (4/4).
> 	- Removed the unnecessary check for match data in probe() in (1/4)
> 	- Some Indentation changes for better readability in (1/4) and (3/4)
> 	- Removed repeated code in (3/4)
> 	- Used 'zynq' and 'zynqmp' instead of 'synps_enh_edac' in function
> names
> 
> Changes in v6:
> 	- Splitted the patches according to functionalities
> 	- Addressed code style comments from v5 review
> 	- Moved the Error Injection to CONFIG_EDAC_DEBUG mode
> 
> Changes in v7:
> 	- Included DTS patch (6/7) which was missed in v6 patch set
> 
> Changes in v8:
> 	- patch (1/7) from v7 is split in to 3 different logically different patches
> 		1. functional changes like code cleanup
> 		2. functions renaming
> 		3. comments cleanup
> 	- Added a separate patch (4) for making always successful functions as
> void
> 	- Corrected 'Too many parentheses' review comment in patch (5)
> 	- Corrected comments as per the v7 review feedback
> 	- Made dedicated functions for IRQ setup, IRQ enable and IRQ disable
> in patch (8)
> 	- Addressed review comments in patch (10)
> 
> Changes in v9:
> 	- Added check for return value of of_device_get_match_data() function
> 	  in (1/6).
> 	- From v8 the first 5 patches are removed in this series as they are
> 	  applied on:
> 
> 	https://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git/?h=edac-for-
> 4.20-synps
> 	- Updated Kconfig to check for ARCH_ZYNQMP instead of ARM64
> 
> Change in v10:
> 	- Moved the checking for ce_cnt and ue_cnt before the readl() call
> 	- Aligned arguments on the opening brace in setup_irq()
> 
> Manish Narani (6):
>   edac: synopsys: Add error handling for NULL in probe()
>   dt: bindings: Document ZynqMP DDRC in Synopsys documentation
>   edac: synopsys: Add macro defines for ZynqMP DDRC
>   edac: synopsys: Add EDAC ECC support for ZynqMP DDRC
>   arm64: zynqmp: Add DDRC node
>   edac: synopsys: Add Error Injection support for ZynqMP DDRC
> 
>  .../bindings/memory-controllers/synopsys.txt       |  27 +-
>  arch/arm64/boot/dts/xilinx/zynqmp.dtsi             |   7 +
>  drivers/edac/Kconfig                               |   2 +-
>  drivers/edac/synopsys_edac.c                       | 911 ++++++++++++++++++++-
>  4 files changed, 918 insertions(+), 29 deletions(-)
> 
> --
> 2.1.1

  parent reply	other threads:[~2018-11-02  8:39 UTC|newest]

Thread overview: 85+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-25  6:06 [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver Manish Narani
2018-10-25  6:06 ` Manish Narani
2018-10-25  6:06 ` Manish Narani
2018-10-25  6:06 ` [PATCH v10 1/6] edac: synopsys: Add error handling for NULL in probe() Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` [v10,1/6] " Manish Narani
2018-10-25  6:06 ` [PATCH v10 2/6] dt: bindings: Document ZynqMP DDRC in Synopsys documentation Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` [v10,2/6] " Manish Narani
2018-10-25  6:06 ` [PATCH v10 3/6] edac: synopsys: Add macro defines for ZynqMP DDRC Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` [v10,3/6] " Manish Narani
2018-10-25  6:06 ` [PATCH v10 4/6] edac: synopsys: Add EDAC ECC support " Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` Manish Narani
2018-10-25  6:06   ` [v10,4/6] " Manish Narani
2018-10-25  6:07 ` [PATCH v10 5/6] arm64: zynqmp: Add DDRC node Manish Narani
2018-10-25  6:07   ` Manish Narani
2018-10-25  6:07   ` Manish Narani
2018-10-25  6:07   ` [v10,5/6] " Manish Narani
2018-11-05 12:56   ` [PATCH v10 5/6] " Borislav Petkov
2018-11-05 12:56     ` Borislav Petkov
2018-11-05 12:56     ` [v10,5/6] " Borislav Petkov
2018-11-05 13:06     ` [PATCH v10 5/6] " Michal Simek
2018-11-05 13:06       ` Michal Simek
2018-11-05 13:06       ` Michal Simek
2018-11-05 13:06       ` [v10,5/6] " Michal Simek
2018-11-05 13:20       ` [PATCH v10 5/6] " Borislav Petkov
2018-11-05 13:20         ` Borislav Petkov
2018-11-05 13:20         ` [v10,5/6] " Borislav Petkov
2018-11-05 13:32         ` [PATCH v10 5/6] " Michal Simek
2018-11-05 13:32           ` Michal Simek
2018-11-05 13:32           ` Michal Simek
2018-11-05 13:32           ` [v10,5/6] " Michal Simek
2018-11-05 13:42           ` [PATCH v10 5/6] " Borislav Petkov
2018-11-05 13:42             ` Borislav Petkov
2018-11-05 13:42             ` [v10,5/6] " Borislav Petkov
2018-11-05 13:45             ` [PATCH v10 5/6] " Michal Simek
2018-11-05 13:45               ` Michal Simek
2018-11-05 13:45               ` Michal Simek
2018-11-05 13:45               ` [v10,5/6] " Michal Simek
2018-11-05 14:51             ` [PATCH v10 5/6] " Olof Johansson
2018-11-05 14:51               ` Olof Johansson
2018-11-05 14:51               ` [v10,5/6] " Olof Johansson
2018-11-05 19:47               ` [PATCH v10 5/6] " Borislav Petkov
2018-11-05 19:47                 ` Borislav Petkov
2018-11-05 19:47                 ` [v10,5/6] " Borislav Petkov
2018-11-05 20:38                 ` [PATCH v10 5/6] " Olof Johansson
2018-11-05 20:38                   ` Olof Johansson
2018-11-05 20:38                   ` [v10,5/6] " Olof Johansson
2018-11-05 20:43                   ` [PATCH v10 5/6] " Borislav Petkov
2018-11-05 20:43                     ` Borislav Petkov
2018-11-05 20:43                     ` [v10,5/6] " Borislav Petkov
2018-11-06  6:46                     ` [PATCH v10 5/6] " Michal Simek
2018-11-06  6:46                       ` Michal Simek
2018-11-06  6:46                       ` Michal Simek
2018-11-06  6:46                       ` [v10,5/6] " Michal Simek
2018-11-06  9:22                       ` [PATCH v10 5/6] " Borislav Petkov
2018-11-06  9:22                         ` Borislav Petkov
2018-11-06  9:22                         ` [v10,5/6] " Borislav Petkov
2018-11-06 11:54   ` [PATCH v10 5/6] " Michal Simek
2018-11-06 11:54     ` Michal Simek
2018-11-06 11:54     ` Michal Simek
2018-11-06 11:54     ` [v10,5/6] " Michal Simek
2018-10-25  6:07 ` [PATCH v10 6/6] edac: synopsys: Add Error Injection support for ZynqMP DDRC Manish Narani
2018-10-25  6:07   ` Manish Narani
2018-10-25  6:07   ` Manish Narani
2018-10-25  6:07   ` [v10,6/6] " Manish Narani
2018-11-02  8:38 ` Manish Narani [this message]
2018-11-02  8:38   ` [PATCH v10 0/6] EDAC: Enhancements to Synopsys EDAC driver Manish Narani
2018-11-02  8:38   ` Manish Narani
2018-11-02  8:58   ` Borislav Petkov
2018-11-02  8:58     ` Borislav Petkov
2018-11-02  8:58     ` Borislav Petkov
2018-11-06 10:03 ` Borislav Petkov
2018-11-06 10:03   ` Borislav Petkov
2018-11-06 10:42   ` Manish Narani
2018-11-06 10:42     ` Manish Narani
2018-11-06 10:42     ` Manish Narani
2018-11-06 10:58     ` Borislav Petkov
2018-11-06 10:58       ` Borislav Petkov
2018-11-06 10:58       ` 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=DM6PR02MB52271A9C7D7829CA1467B657C1CF0@DM6PR02MB5227.namprd02.prod.outlook.com \
    --to=mnarani@xilinx.com \
    --cc=amit.kucheria@linaro.org \
    --cc=bp@alien8.de \
    --cc=devicetree@vger.kernel.org \
    --cc=leoyang.li@nxp.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-edac@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mchehab@kernel.org \
    --cc=michals@xilinx.com \
    --cc=robh+dt@kernel.org \
    --cc=sudeep.holla@arm.com \
    /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.