Re: [dpdk-dev] [PATCH V6] config/arm: add Qualcomm Centriq 2400 part number

[Thomas Monjalon <thomas@monjalon.net>]

21/06/2021 03:52, Ruifeng Wang:
> From: Thomas Monjalon <thomas@monjalon.net>
> > 18/06/2021 10:53, Thierry Herbelot:
> > > On 6/18/21 10:51 AM, Thomas Monjalon wrote:
> > > > 18/06/2021 04:09, Ruifeng Wang:
> > > >> From: Thierry Herbelot <thierry.herbelot@6wind.com>
> > > >>>       'part_number_config': {
> > > >>> -        '0xc00': {'machine_args':  ['-march=armv8-a+crc']}
> > > >>> +        '0x800': {'machine_args':  ['-march=armv8-a+crc']},
> > > >>> +        '0xc00': {'machine_args':  ['-march=armv8-a+crc']},
> > > >> Nit, redundant comma at the end of the line.
> > > >
> > > > What is redundant?
> > >
> > > The comma at the end of the second line is not necessary.
> > 
> > It is a good practice to have comma like other lines, so no need to update this
> > line when adding more.
> > 
> Looked at style in the rest of the file. Just wanted them to be aligned.
> I'm fine with a trailing comma at the last line.
> 
> Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>

Applied, thanks