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 <wangyongrong@xiaomi.com>
OpenAMP · Sep 18, 2024 · e09a641 · e09a641
1 parent c468328
commit e09a641
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;
 }