openamp/virtio.h: make final_features optional

negotiate also can be call when final_features is NULL Signed-off-by: Yongrong Wang <wangyongrong@xiaomi.com>
OpenAMP · Oct 8, 2024 · 8c2929e · 8c2929e
1 parent 36905d7
commit 8c2929e
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/include/openamp/virtio.h b/lib/include/openamp/virtio.h
@@ -468,14 +468,15 @@ static inline int virtio_negotiate_features(struct virtio_device *vdev,
 					    uint32_t features,
 					    uint32_t *final_features)
 {
-	if (!vdev || !final_features)
+	if (!vdev)
 		return -EINVAL;
 
 	if (!vdev->func || !vdev->func->negotiate_features)
 		return -ENXIO;
 
-	*final_features = vdev->func->negotiate_features(vdev, features);
-	vdev->features = *final_features;
+	vdev->features = vdev->func->negotiate_features(vdev, features);
+	if (final_features)
+		*final_features = vdev->features;
 	return 0;
 }