On Tue, 29 Oct 2024 21:45:22 +0800
Yong Zhang <zhang.yon...@zte.com.cn> wrote:

> +char zxdh_gdma_driver_name[] = "rawdev_zxdh_gdma";
> +char dev_name[] = "zxdh_gdma";
> +
> +uint32_t
> +zxdh_gdma_read_reg(struct rte_rawdev *dev, uint16_t queue_id, uint32_t 
> offset)
> +{
> +     struct zxdh_gdma_rawdev *gdmadev = zxdh_gdma_rawdev_get_priv(dev);
> +     uint32_t addr = 0;
> +     uint32_t val = 0;
> +
> +     addr = offset + queue_id * ZXDH_GDMA_CHAN_SHIFT;
> +     val = *(uint32_t *)(gdmadev->base_addr + addr);
> +
> +     return val;
> +}
> +
> +void
> +zxdh_gdma_write_reg(struct rte_rawdev *dev, uint16_t queue_id, uint32_t 
> offset, uint32_t val)
> +{
> +     struct zxdh_gdma_rawdev *gdmadev = zxdh_gdma_rawdev_get_priv(dev);
> +     uint32_t addr = 0;
> +
> +     addr = offset + queue_id * ZXDH_GDMA_CHAN_SHIFT;
> +     *(uint32_t *)(gdmadev->base_addr + addr) = val;
> +}

This driver is made up one C file. Please make all these helper functions
and the names static. If static checkers and optimizers can work better.

Reply via email to