mbox series

[RESEND,PATCHv3,0/3] Update Stratix10 EDAC Bindings

Message ID 1554388597-28019-1-git-send-email-thor.thayer@linux.intel.com
Headers show
Series Update Stratix10 EDAC Bindings | expand

Message

Thor Thayer April 4, 2019, 2:36 p.m. UTC
From: Thor Thayer <thor.thayer@linux.intel.com>

This patch series makes the Stratix10 EDAC Bindings
specific to the Stratix10 ARM64 architecture.

Instead of using the Arria10 (ARM32) EDAC bindings for
Stratix10 (ARM64), create Stratix10 specific EDAC bindings
to capture architecture differences between ARM32 and ARM64.
This requires fixing the previous Stratix10 bindings.
Also add the peripheral bindings for the Stratix10.

V2: Remove Stratix10 compatible check.

V3: Remove patches not related to EDAC bindings so that
    this patchset is targeted toward the bindings.
    Improve the Stratix10 specific binding description by
    adding in descriptions of ARM64 differences.

Resend: Add Reviewed-by and Acked-by from bindings and 
    socfpga device tree maintainers. I'd like this to go
    through the EDAC tree since future patches will have
    a dependency on these bindings and device tree.

Thor Thayer (3):
  Documentation: dt: edac: Fix Stratix10 IRQ bindings
  Documentation: dt: edac: Add Stratix10 Peripheral bindings
  arm64: dts: stratix10: Use new Stratix10 EDAC bindings

 .../devicetree/bindings/edac/socfpga-eccmgr.txt    | 135 +++++++++++++++++++--
 arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi  |  25 ++--
 2 files changed, 138 insertions(+), 22 deletions(-)

Comments

Thor Thayer April 9, 2019, 7:11 p.m. UTC | #1
Hi Boris,

On 4/4/19 9:36 AM, thor.thayer@linux.intel.com wrote:
> From: Thor Thayer <thor.thayer@linux.intel.com>
> 
> This patch series makes the Stratix10 EDAC Bindings
> specific to the Stratix10 ARM64 architecture.
> 
> Instead of using the Arria10 (ARM32) EDAC bindings for
> Stratix10 (ARM64), create Stratix10 specific EDAC bindings
> to capture architecture differences between ARM32 and ARM64.
> This requires fixing the previous Stratix10 bindings.
> Also add the peripheral bindings for the Stratix10.
> 
> V2: Remove Stratix10 compatible check.
> 
> V3: Remove patches not related to EDAC bindings so that
>      this patchset is targeted toward the bindings.
>      Improve the Stratix10 specific binding description by
>      adding in descriptions of ARM64 differences.
> 
> Resend: Add Reviewed-by and Acked-by from bindings and
>      socfpga device tree maintainers. I'd like this to go
>      through the EDAC tree since future patches will have
>      a dependency on these bindings and device tree.
> 
> Thor Thayer (3):
>    Documentation: dt: edac: Fix Stratix10 IRQ bindings
>    Documentation: dt: edac: Add Stratix10 Peripheral bindings
>    arm64: dts: stratix10: Use new Stratix10 EDAC bindings
> 
>   .../devicetree/bindings/edac/socfpga-eccmgr.txt    | 135 +++++++++++++++++++--
>   arch/arm64/boot/dts/altera/socfpga_stratix10.dtsi  |  25 ++--
>   2 files changed, 138 insertions(+), 22 deletions(-)
> 

If possible, I'd like this series to go through the EDAC tree since 
future patches will follow this format.

Regards,

Thor
Borislav Petkov April 9, 2019, 8:13 p.m. UTC | #2
On Tue, Apr 09, 2019 at 02:11:58PM -0500, Thor Thayer wrote:
> If possible, I'd like this series to go through the EDAC tree since future
> patches will follow this format.

You know how that works - I need ACKs.
Thor Thayer April 9, 2019, 10:32 p.m. UTC | #3
Hi Boris,

On 4/9/19 3:13 PM, Borislav Petkov wrote:
> On Tue, Apr 09, 2019 at 02:11:58PM -0500, Thor Thayer wrote:
>> If possible, I'd like this series to go through the EDAC tree since future
>> patches will follow this format.
> 
> You know how that works - I need ACKs.
> 

I have ACKs on patches 1 & 3. Patch 2 has a Reviewed-by from Rob Herring 
which was sufficient in the past.

Here are the links to the original ACKs and Reviewed-by.

Patch 1: https://patchwork.codeaurora.org/patch/737155/
Patch 2: https://patchwork.codeaurora.org/patch/757205/
Patch 3: https://patchwork.codeaurora.org/patch/757207/
Borislav Petkov April 10, 2019, 7:04 p.m. UTC | #4
On Tue, Apr 09, 2019 at 05:32:07PM -0500, Thor Thayer wrote:
> I have ACKs on patches 1 & 3. Patch 2 has a Reviewed-by from Rob Herring
> which was sufficient in the past.

Sorry about that - I missed those because I looked only at the diffstat
and decided this series is not for me :)

Anyway, all three applied now.

Thx.