On Fri, 2019-06-14 at 11:22 +0530, Vinod Koul wrote: > [External] > > > On 06-06-19, 13:45, Alexandru Ardelean wrote: > > > +static bool axi_dmac_regmap_rdwr(struct device *dev, unsigned int reg) > > +{ > > + switch (reg) { > > + case AXI_DMAC_REG_IRQ_MASK: > > + case AXI_DMAC_REG_IRQ_SOURCE: > > + case AXI_DMAC_REG_IRQ_PENDING: > > + case AXI_DMAC_REG_CTRL: > > + case AXI_DMAC_REG_TRANSFER_ID: > > + case AXI_DMAC_REG_START_TRANSFER: > > + case AXI_DMAC_REG_FLAGS: > > + case AXI_DMAC_REG_DEST_ADDRESS: > > + case AXI_DMAC_REG_SRC_ADDRESS: > > + case AXI_DMAC_REG_X_LENGTH: > > + case AXI_DMAC_REG_Y_LENGTH: > > + case AXI_DMAC_REG_DEST_STRIDE: > > + case AXI_DMAC_REG_SRC_STRIDE: > > + case AXI_DMAC_REG_TRANSFER_DONE: > > + case AXI_DMAC_REG_ACTIVE_TRANSFER_ID : > > Space before : ...? oops sorry about that thanks for fixing Alex > > -- > ~Vinod