On Thu, Jul 6, 2023 at 1:22 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > From: Hien Huynh <hien.huynh.px@xxxxxxxxxxx> > > Before setting DDS and SDS values, we need to clear its value first > otherwise, we get incorrect results when we change/update the DMA bus > width several times due to the 'OR' expression. > > Fixes: 5000d37042a6 ("dmaengine: sh: Add DMAC driver for RZ/G2L SoC") > Cc: stable@xxxxxxxxxx > Signed-off-by: Hien Huynh <hien.huynh.px@xxxxxxxxxxx> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > --- > v2->v3: > * Added bitfield.h header file and replaced CHCFG_FILL_{SDS,DDS} > macros with FIELD_PREP. > * Updated commit header 'dma: rz-dmac'->'dmaengine: sh: rz-dmac'. Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds