Thread (20 messages) 20 messages, 4 authors, 2021-07-09

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

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


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help