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.