Re: [dpdk-dev] [PATCH V6] config/arm: add Qualcomm Centriq 2400 part number
From: Thomas Monjalon <hidden>
Date: 2021-07-09 15:27:30
From: Thomas Monjalon <hidden>
Date: 2021-07-09 15:27:30
21/06/2021 03:52, Ruifeng Wang:
From: Thomas Monjalon <redacted>quoted
18/06/2021 10:53, Thierry Herbelot:quoted
On 6/18/21 10:51 AM, Thomas Monjalon wrote:quoted
18/06/2021 04:09, Ruifeng Wang:quoted
From: Thierry Herbelot <redacted>quoted
'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 <redacted>
Applied, thanks