From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Subject: Re: [PATCH v4 2/3] clk: vc5: Add bindings for IDT VersaClock 5P49V5935 Date: Fri, 7 Apr 2017 21:56:29 +0200 Message-ID: References: <1491556344-9465-1-git-send-email-alexey_firago@mentor.com> <1491556344-9465-3-git-send-email-alexey_firago@mentor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1491556344-9465-3-git-send-email-alexey_firago-nmGgyN9QBj3QT0dZR+AlfA@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alexey Firago , mturquette-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org, linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 04/07/2017 11:12 AM, Alexey Firago wrote: > IDT VersaClock 5 5P49V5935 has 4 clock outputs, 4 fractional dividers. > Input clock source can be taken from either integrated crystal or from > external reference clock. > > Signed-off-by: Alexey Firago I think you want a R-B from Rob on this. > --- > .../devicetree/bindings/clock/idt,versaclock5.txt | 16 +++++++++++++--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.txt b/Documentation/devicetree/bindings/clock/idt,versaclock5.txt > index 87e9c47..53d7e50 100644 > --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.txt > +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.txt > @@ -6,18 +6,21 @@ from 3 to 12 output clocks. > ==I2C device node== > > Required properties: > -- compatible: shall be one of "idt,5p49v5923" , "idt,5p49v5933". > +- compatible: shall be one of "idt,5p49v5923" , "idt,5p49v5933" , > + "idt,5p49v5935". > - reg: i2c device address, shall be 0x68 or 0x6a. > - #clock-cells: from common clock binding; shall be set to 1. > - clocks: from common clock binding; list of parent clock handles, > - 5p49v5923: (required) either or both of XTAL or CLKIN > reference clock. > - - 5p49v5933: (optional) property not present (internal > + - 5p49v5933 and > + - 5p49v5935: (optional) property not present (internal > Xtal used) or CLKIN reference > clock. > - clock-names: from common clock binding; clock input names, can be > - 5p49v5923: (required) either or both of "xin", "clkin". > - - 5p49v5933: (optional) property not present or "clkin". > + - 5p49v5933 and > + - 5p49v5935: (optional) property not present or "clkin". > > ==Mapping between clock specifier and physical pins== > > @@ -34,6 +37,13 @@ clock specifier, the following mapping applies: > 1 -- OUT1 > 2 -- OUT4 > > +5P49V5935: > + 0 -- OUT0_SEL_I2CB > + 1 -- OUT1 > + 2 -- OUT2 > + 3 -- OUT3 > + 4 -- OUT4 > + > ==Example== > > /* 25MHz reference crystal */ > -- Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html