On Mon, Mar 7, 2022 at 7:48 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > max_seg_size is unsigned int and it can have a value up to 2^32 > (for eg:-RZ_DMAC driver sets dma_set_max_seg_size as U32_MAX) > When this value is used in min_t() as an integer type, it becomes > -1 and the value of sgs becomes 0. > > Fix this issue by replacing the 'int' data type with 'unsigned int' > in min_t(). > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > --- > v1->v2: > * Removed the cast for ctlr->max_dma_len in min_t() 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