openamp/virtio.h: negotiate also can be call when final_features is NULL

add helpful judge for final_features is NULL in virtio_negotiate_features Signed-off-by: Yongrong Wang <[email protected]>
OpenAMP · Sep 23, 2024 · f472fe6 · f472fe6
1 parent 840cc27
commit f472fe6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/include/openamp/virtio.h b/lib/include/openamp/virtio.h
@@ -468,13 +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 = vdev->func->negotiate_features(vdev, features);
+	if (final_features)
+		*final_features = vdev->features;
 	return 0;
 }