| Message ID | 0b8ad4dcc185aa7a17655983e0eb5690d8fed460.1646311558.git.geert+renesas@glider.be (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | a0754cf3b3bfb0cb6ae30bd59e2dd0095fdbda72 |
| Headers | show |
| Series | dmaengine: fsl-dpaa2-qdma: Drop comma after SoC match table sentinel | expand |
On 03-03-22, 13:47, Geert Uytterhoeven wrote: > It does not make sense to have a comma after a sentinel, as any new > elements must be added before the sentinel. > > Add a comment to clarify the purpose of the empty element. Applied, thanks
diff --git a/drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.h b/drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.h index 7d571849c569cb32..03e2f4e0baca83d2 100644 --- a/drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.h +++ b/drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.h @@ -139,7 +139,7 @@ struct dpaa2_qdma_priv_per_prio { static struct soc_device_attribute soc_fixup_tuning[] = { { .family = "QorIQ LX2160A"}, - { }, + { /* sentinel */ } }; /* FD pool size: one FD + 3 Frame list + 2 source/destination descriptor */
It does not make sense to have a comma after a sentinel, as any new elements must be added before the sentinel. Add a comment to clarify the purpose of the empty element. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)