Hi Jacopo, On Mon, Jan 9, 2017 at 8:31 PM, Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> wrote: > From: Magnus Damm <damm@xxxxxxxxxxxxx> > > Squash commits in Geert's renesas-driver/genmai-gpio-and-pfc branch that > add support for r7s72100 PFC. > This squash combines commits for Magnus' original driver and minor fixes > to forward-port it to a more recent kernel (v4.10) > > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> > --- a/drivers/pinctrl/sh-pfc/core.c > +++ b/drivers/pinctrl/sh-pfc/core.c > @@ -473,6 +473,12 @@ static const struct of_device_id sh_pfc_of_table[] = { > .data = &emev2_pinmux_info, > }, > #endif > +#ifdef CONFIG_PINCTRL_PFC_R7S72100 > + { > + .compatible = "renesas,pfc-r7s72100", > + .data = &r7s72100_pinmux_info, > + }, > +#endif > #ifdef CONFIG_PINCTRL_PFC_R8A73A4 > { > .compatible = "renesas,pfc-r8a73a4", > @@ -626,6 +632,9 @@ static int sh_pfc_probe(struct platform_device *pdev) > } > > static const struct platform_device_id sh_pfc_id_table[] = { > +#ifdef CONFIG_PINCTRL_PFC_R7S72100 > + { "pfc-r7s72100", (kernel_ulong_t)&r7s72100_pinmux_info }, > +#endif The above chunk should be removed, as now r7s72100 always uses DT. > #ifdef CONFIG_PINCTRL_PFC_SH7203 > { "pfc-sh7203", (kernel_ulong_t)&sh7203_pinmux_info }, > #endif 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