On Thu, Jun 30, 2016 at 12:05 AM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote: > --- a/drivers/mmc/host/sh_mobile_sdhi.c > +++ b/drivers/mmc/host/sh_mobile_sdhi.c > +static int sh_mobile_sdhi_init_dma(enum tmio_mmc_dmac_type dmac_type) > +{ > + switch (dmac_type) { > + case TMIO_MMC_INTERNAL_DMAC: > + return sdhi_internal_dmac_init_dma(); > + > + case TMIO_MMC_SYSC_DMAC: > + return sdhi_sysc_dmac_init_dma(); > + } > +} Depending on compiler version and optimization level: drivers/mmc/host/sh_mobile_sdhi.c:172:1: warning: control reaches end of non-void function [-Wreturn-type] 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