From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Thu, 19 Nov 2015 20:19:14 +0000 Subject: Re: [PATCH 01/25] serial: sh-sci: Update DT binding documentation for external clock input Message-Id: <1756365.Ijac5ApxcA@avalon> List-Id: References: <1447958344-836-1-git-send-email-geert+renesas@glider.be> <1447958344-836-2-git-send-email-geert+renesas@glider.be> In-Reply-To: <1447958344-836-2-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , Simon Horman , Magnus Damm , Yoshinori Sato , Laurent Pinchart , linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Hi Geert, Thank you for the patch. On Thursday 19 November 2015 19:38:40 Geert Uytterhoeven wrote: > Amend the DT bindings to include the optional external clock on > (H)SCI(F) and some SCIFA, where this pin can serve as a clock input, > depending on board wiring. > > Signed-off-by: Geert Uytterhoeven > Cc: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/serial/renesas,sci-serial.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt index > 2c9e6b8477e92792..8efc9b6f35637fbb 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > +++ b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > @@ -43,6 +43,9 @@ Required properties: > - clocks: Must contain a phandle and clock-specifier pair for each entry > in clock-names. > - clock-names: Must contain "fck" for the SCIx UART functional clock. > + On (H)SCI(F) and some SCIFA, an additional clock may be specified: Could you list the SCIFA variants that support external clocks ? > + - "hsck" for the optional external clock input (on HSCIF), > + - "sck" for the optional external clock input (on other variants). > > Note: Each enabled SCIx UART should have an alias correctly numbered in the > "aliases" node. -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161414AbbKSUTI (ORCPT ); Thu, 19 Nov 2015 15:19:08 -0500 Received: from galahad.ideasonboard.com ([185.26.127.97]:33012 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161319AbbKSUTG (ORCPT ); Thu, 19 Nov 2015 15:19:06 -0500 From: Laurent Pinchart To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , Simon Horman , Magnus Damm , Yoshinori Sato , Laurent Pinchart , linux-serial@vger.kernel.org, linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 01/25] serial: sh-sci: Update DT binding documentation for external clock input Date: Thu, 19 Nov 2015 22:19:14 +0200 Message-ID: <1756365.Ijac5ApxcA@avalon> User-Agent: KMail/4.14.8 (Linux/4.0.9-gentoo; KDE/4.14.8; x86_64; ; ) In-Reply-To: <1447958344-836-2-git-send-email-geert+renesas@glider.be> References: <1447958344-836-1-git-send-email-geert+renesas@glider.be> <1447958344-836-2-git-send-email-geert+renesas@glider.be> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Geert, Thank you for the patch. On Thursday 19 November 2015 19:38:40 Geert Uytterhoeven wrote: > Amend the DT bindings to include the optional external clock on > (H)SCI(F) and some SCIFA, where this pin can serve as a clock input, > depending on board wiring. > > Signed-off-by: Geert Uytterhoeven > Cc: devicetree@vger.kernel.org > --- > Documentation/devicetree/bindings/serial/renesas,sci-serial.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt index > 2c9e6b8477e92792..8efc9b6f35637fbb 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > +++ b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > @@ -43,6 +43,9 @@ Required properties: > - clocks: Must contain a phandle and clock-specifier pair for each entry > in clock-names. > - clock-names: Must contain "fck" for the SCIx UART functional clock. > + On (H)SCI(F) and some SCIFA, an additional clock may be specified: Could you list the SCIFA variants that support external clocks ? > + - "hsck" for the optional external clock input (on HSCIF), > + - "sck" for the optional external clock input (on other variants). > > Note: Each enabled SCIx UART should have an alias correctly numbered in the > "aliases" node. -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 01/25] serial: sh-sci: Update DT binding documentation for external clock input Date: Thu, 19 Nov 2015 22:19:14 +0200 Message-ID: <1756365.Ijac5ApxcA@avalon> References: <1447958344-836-1-git-send-email-geert+renesas@glider.be> <1447958344-836-2-git-send-email-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <1447958344-836-2-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , Simon Horman , Magnus Damm , Yoshinori Sato , Laurent Pinchart , linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Geert, Thank you for the patch. On Thursday 19 November 2015 19:38:40 Geert Uytterhoeven wrote: > Amend the DT bindings to include the optional external clock on > (H)SCI(F) and some SCIFA, where this pin can serve as a clock input, > depending on board wiring. > > Signed-off-by: Geert Uytterhoeven > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > --- > Documentation/devicetree/bindings/serial/renesas,sci-serial.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt index > 2c9e6b8477e92792..8efc9b6f35637fbb 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > +++ b/Documentation/devicetree/bindings/serial/renesas,sci-serial.txt > @@ -43,6 +43,9 @@ Required properties: > - clocks: Must contain a phandle and clock-specifier pair for each entry > in clock-names. > - clock-names: Must contain "fck" for the SCIx UART functional clock. > + On (H)SCI(F) and some SCIFA, an additional clock may be specified: Could you list the SCIFA variants that support external clocks ? > + - "hsck" for the optional external clock input (on HSCIF), > + - "sck" for the optional external clock input (on other variants). > > Note: Each enabled SCIx UART should have an alias correctly numbered in the > "aliases" node. -- Regards, Laurent Pinchart -- 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