Hi Shimoda-san, On Fri, Apr 20, 2018 at 2:27 PM, Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: > From: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > > This patch adds all R-Car E3 Clock Pulse Generator Core Clock Outputs. > > Note that internal CPG clocks (S0, S1, S2, S3, SDSRC) are not included, > as they are used as internal clock sources only, and never referenced > from DT. > > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > [shimoda: add SPDX-License-Identifier] > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> Thanks for your patch! > --- /dev/null > +++ b/include/dt-bindings/clock/r8a77990-cpg-mssr.h > @@ -0,0 +1,62 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* > + * Copyright (C) 2018 Renesas Electronics Corp. > + */ > +#ifndef __DT_BINDINGS_CLOCK_R8A77990_CPG_MSSR_H__ > +#define __DT_BINDINGS_CLOCK_R8A77990_CPG_MSSR_H__ > + > +#include <dt-bindings/clock/renesas-cpg-mssr.h> > + > +/* r8a77990 CPG Core Clocks */ > +#define R8A77990_CLK_Z2 0 [...] > +#define R8A77990_CLK_CSI0 47 > +#define R8A77990_CLK_CP 49 > +#define R8A77990_CLK_CPEX 50 The numbering should be contiguous. I'll fix it up while applying, and add "POST3" to the list of internal clocks in the commit message. > + > +#endif /* __DT_BINDINGS_CLOCK_R8A77990_CPG_MSSR_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 -- 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