[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v2 1/3] target-arm: delete ARM_FEATURE_A64FX
From: |
address@hidden |
Subject: |
RE: [PATCH v2 1/3] target-arm: delete ARM_FEATURE_A64FX |
Date: |
Tue, 3 Aug 2021 00:23:59 +0000 |
> This feature doesn't exist in upstream QEMU, so this won't apply.
> For a v2 of a patch, the patches should be based on upstream, not on top of
> the v1
> series.
Thank you for your comment.
I understood your point.
Best regards.
> -----Original Message-----
> From: Peter Maydell <peter.maydell@linaro.org>
> Sent: Friday, July 30, 2021 7:35 PM
> To: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com>
> Cc: qemu-arm <qemu-arm@nongnu.org>; QEMU Developers
> <qemu-devel@nongnu.org>
> Subject: Re: [PATCH v2 1/3] target-arm: delete ARM_FEATURE_A64FX
>
> On Fri, 30 Jul 2021 at 04:08, Shuuichirou Ishii <ishii.shuuichir@fujitsu.com>
> wrote:
> >
> > The ARM_FEATURE_A64FX property was added, but there is no function
> > that uses this property yet, so it will be removed until a function
> > that uses it is added.
> >
> > Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com>
> > ---
> > target/arm/cpu.h | 1 -
> > target/arm/cpu64.c | 1 -
> > 2 files changed, 2 deletions(-)
> >
> > diff --git a/target/arm/cpu.h b/target/arm/cpu.h index
> > 1b0c7b91ec..9f0a5f84d5 100644
> > --- a/target/arm/cpu.h
> > +++ b/target/arm/cpu.h
> > @@ -2145,7 +2145,6 @@ enum arm_features {
> > ARM_FEATURE_M_SECURITY, /* M profile Security Extension */
> > ARM_FEATURE_M_MAIN, /* M profile Main Extension */
> > ARM_FEATURE_V8_1M, /* M profile extras only in v8.1M and later */
> > - ARM_FEATURE_A64FX, /* Fujitsu A64FX processor HPC extensions
> support */
> > };
>
> This feature doesn't exist in upstream QEMU, so this won't apply.
>
> For a v2 of a patch, the patches should be based on upstream, not on top of
> the v1
> series.
>
> thanks
> -- PMM
- RE: [PATCH v2 1/3] target-arm: delete ARM_FEATURE_A64FX,
address@hidden <=