Re: [PATCH v3] config/arm: add Phytium FT-2000+

[Thomas Monjalon <thomas@monjalon.net>]

05/07/2022 15:50, Ruifeng Wang:
> > From: luzhipeng <luzhipeng@cestc.cn>
> > 
> > Here adds configs for Phytium server.
> > 
> > Signed-off-by: luzhipeng <luzhipeng@cestc.cn>
> > Reviewed-by: Thomas Monjalon <thomas@monjalon.net>
> > ---
> > I did some changes in this v3:
> > - PHYTIUM -> Phytium (as on the website)
> > - ft2000plus -> FT-2000+ (as found online)
> > - alphabetical ordering
> > - extra commas for future extensions
> > - g++ as cpp Meson command
> > 
> > Please tell how to write your name with spaces and capitals in the order [first
> > name] [family name], thanks.
> > ---
> 
> <snip>
> 
> >  implementer_qualcomm = {
> >      'description': 'Qualcomm',
> >      'flags': [
> > @@ -214,7 +228,7 @@ implementer_qualcomm = {
> >          '0xc00': {
> >              'march': 'armv8-a',
> >              'march_features': ['crc']
> > -        }
> > +        },
> 
> Unintentional change?
> The change is valid, but not related to this patch.
> 
> Otherwise looks good to me.
> Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>

Applied (without above unintentional change), thanks.