Re: [PATCH] xen/arm: Update silicon-errata.txt with the Neoverse AT erratum

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Wed, 26 Aug 2020, Bertrand Marquis wrote:
> > On 26/08/2020 08:46, Bertrand Marquis wrote:
> >>> On 25 Aug 2020, at 18:42, Julien Grall <julien@xxxxxxx> wrote:
> >>>
> >>> From: Julien Grall <jgrall@xxxxxxxxxx>
> >>>
> >>> Commit 858c0be8c2fa "xen/arm: Enable CPU Erratum 1165522 for Neoverse"
> >>> added a new erratum but forgot to update silicon-errata.txt.
> >>>
> >>> Update the file accordingly to keep track of errata workaround in Xen.
> >> Oh i should have done that as part of my patch.
> >> Nice catch.
> >>>
> >>> Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> >>> Cc: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> >>> ---
> >>> docs/misc/arm/silicon-errata.txt | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/docs/misc/arm/silicon-errata.txt 
> >>> b/docs/misc/arm/silicon-errata.txt
> >>> index 11e5a9dcec2c..ee070e723a5f 100644
> >>> --- a/docs/misc/arm/silicon-errata.txt
> >>> +++ b/docs/misc/arm/silicon-errata.txt
> >>> @@ -51,4 +51,5 @@ stable hypervisors.
> >>> | ARM            | Cortex-A57      | #1319537        | N/A                
> >>>      |
> >>> | ARM            | Cortex-A72      | #1319367        | N/A                
> >>>      |
> >>> | ARM            | Cortex-A76      | #1165522        | N/A                
> >>>      |
> >>> +| ARM            | Neoverse        | #1165522        | N/A
> >> Should be Neoverse-N1 here (E1 for example is not impacted by this errata)
> >
> > Ah, right. Would you be happy if I do the change on commit?
> 
> Sure then you have my:
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>

Acked-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>