Hi Wolfram, On Tue, Dec 10, 2019 at 11:28 PM Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: > > BTW, why is tap_set unsigned long instead of unsigned int? > > Because we use bitmap functions on it, and those have all unsigned long > as arguments. Do we? I only see bitops on host->taps? > > And perhaps it should be moved from host to priv? > > That would be great, but other tap_* variables are still used in > tmio_mmc_core.c. We maybe can refactor all tap handling into > renesas_sdhi_core.c meanwhile, but this is a seperate issue. tap_num is shared by the Renesas and TMIO code. tap_set is Renesas-specific. 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