On Wed, Mar 22, 2017 at 11:33 AM, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > On Mon, Mar 20, 2017 at 5:14 PM, Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> wrote: >> Add device tree bindings documentation for Renesas RZ/A1 gpio and pin >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt ulating the pin index from its position >> + (port where it sits on and pin offset), and alternate function >> + configuration options are provided in pin controller header file at: > > the pin ... file > >> + include/dt-bindings/pinctrl/r7s72100-pinctrl.h > > Hence I'd include that file in this patch, as it's part of the bindings. Ah, that would create a hard dependency between the DTS files and the DT bindings, which typically go in through different trees. As the driver replicates the definitions from the header, the include file can go in with the DTS updates. 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html