Commit 86e17a51 authored by Xie Yongji's avatar Xie Yongji Committed by Michael S. Tsirkin

vdpa: Fix some coding style issues

Fix some code indent issues and following checkpatch warning:

WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
371: FILE: include/linux/vdpa.h:371:
+static inline void vdpa_get_config(struct vdpa_device *vdev, unsigned offset,
Signed-off-by: default avatarXie Yongji <xieyongji@bytedance.com>
Acked-by: default avatarJason Wang <jasowang@redhat.com>
Reviewed-by: default avatarStefano Garzarella <sgarzare@redhat.com>
Link: https://lore.kernel.org/r/20210831103634.33-5-xieyongji@bytedance.comSigned-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
parent 9c930054
...@@ -43,17 +43,17 @@ struct vdpa_vq_state_split { ...@@ -43,17 +43,17 @@ struct vdpa_vq_state_split {
* @last_used_idx: used index * @last_used_idx: used index
*/ */
struct vdpa_vq_state_packed { struct vdpa_vq_state_packed {
u16 last_avail_counter:1; u16 last_avail_counter:1;
u16 last_avail_idx:15; u16 last_avail_idx:15;
u16 last_used_counter:1; u16 last_used_counter:1;
u16 last_used_idx:15; u16 last_used_idx:15;
}; };
struct vdpa_vq_state { struct vdpa_vq_state {
union { union {
struct vdpa_vq_state_split split; struct vdpa_vq_state_split split;
struct vdpa_vq_state_packed packed; struct vdpa_vq_state_packed packed;
}; };
}; };
struct vdpa_mgmt_dev; struct vdpa_mgmt_dev;
...@@ -131,7 +131,7 @@ struct vdpa_iova_range { ...@@ -131,7 +131,7 @@ struct vdpa_iova_range {
* @vdev: vdpa device * @vdev: vdpa device
* @idx: virtqueue index * @idx: virtqueue index
* @state: pointer to returned state (last_avail_idx) * @state: pointer to returned state (last_avail_idx)
* @get_vq_notification: Get the notification area for a virtqueue * @get_vq_notification: Get the notification area for a virtqueue
* @vdev: vdpa device * @vdev: vdpa device
* @idx: virtqueue index * @idx: virtqueue index
* Returns the notifcation area * Returns the notifcation area
...@@ -350,25 +350,25 @@ static inline struct device *vdpa_get_dma_dev(struct vdpa_device *vdev) ...@@ -350,25 +350,25 @@ static inline struct device *vdpa_get_dma_dev(struct vdpa_device *vdev)
static inline void vdpa_reset(struct vdpa_device *vdev) static inline void vdpa_reset(struct vdpa_device *vdev)
{ {
const struct vdpa_config_ops *ops = vdev->config; const struct vdpa_config_ops *ops = vdev->config;
vdev->features_valid = false; vdev->features_valid = false;
ops->set_status(vdev, 0); ops->set_status(vdev, 0);
} }
static inline int vdpa_set_features(struct vdpa_device *vdev, u64 features) static inline int vdpa_set_features(struct vdpa_device *vdev, u64 features)
{ {
const struct vdpa_config_ops *ops = vdev->config; const struct vdpa_config_ops *ops = vdev->config;
vdev->features_valid = true; vdev->features_valid = true;
return ops->set_features(vdev, features); return ops->set_features(vdev, features);
} }
static inline void vdpa_get_config(struct vdpa_device *vdev,
static inline void vdpa_get_config(struct vdpa_device *vdev, unsigned offset, unsigned int offset, void *buf,
void *buf, unsigned int len) unsigned int len)
{ {
const struct vdpa_config_ops *ops = vdev->config; const struct vdpa_config_ops *ops = vdev->config;
/* /*
* Config accesses aren't supposed to trigger before features are set. * Config accesses aren't supposed to trigger before features are set.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment