While not technically wrong, the comma at the end of an assignment directive can lead to nasty bugs. Just replace the two occurrences with semicolons.
Signed-off-by: Francesco Valla <[email protected]> --- drivers/remoteproc/remoteproc_virtio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/remoteproc/remoteproc_virtio.c b/drivers/remoteproc/remoteproc_virtio.c index 096bcba527c1..4dacd93bf238 100644 --- a/drivers/remoteproc/remoteproc_virtio.c +++ b/drivers/remoteproc/remoteproc_virtio.c @@ -433,8 +433,8 @@ static int rproc_add_virtio_dev(struct rproc_vdev *rvdev, int id) if (!vdev) return -ENOMEM; - vdev->id.device = id, - vdev->config = &rproc_virtio_config_ops, + vdev->id.device = id; + vdev->config = &rproc_virtio_config_ops; vdev->dev.parent = dev; vdev->dev.release = rproc_virtio_dev_release; -- 2.55.0
