Hi Wolfram, On Thu, Apr 7, 2022 at 4:33 PM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > We have a dedicated directory for platform_data meanwhile, don't spoil > the MMC directory with it. > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> One minor nit below. > --- a/drivers/mmc/host/sh_mmcif.c > +++ b/drivers/mmc/host/sh_mmcif.c > @@ -43,13 +43,13 @@ > #include <linux/mmc/host.h> > #include <linux/mmc/mmc.h> > #include <linux/mmc/sdio.h> > -#include <linux/mmc/sh_mmcif.h> > #include <linux/mmc/slot-gpio.h> > #include <linux/mod_devicetable.h> > #include <linux/mutex.h> > #include <linux/of_device.h> > #include <linux/pagemap.h> > #include <linux/platform_device.h> > +#include <linux/platform_data/sh_mmcif.h> Is this intended to be sorted? a < e. > #include <linux/pm_qos.h> > #include <linux/pm_runtime.h> > #include <linux/sh_dma.h> 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