From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 4 Dec 2018 16:56:03 -0600 From: Rob Herring Subject: Re: [PATCH 1/2] dt-bindings: can: rcar_canfd: document r8a77965 support Message-ID: <20181204225603.GA16478@bogus> References: <20181118173201.4645-1-marek.vasut+renesas@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181118173201.4645-1-marek.vasut+renesas@gmail.com> To: Marek Vasut Cc: devicetree@vger.kernel.org, Marek Vasut , Eugeniu Rosca , Geert Uytterhoeven , Marc Kleine-Budde , Simon Horman , Wolfram Sang , linux-renesas-soc@vger.kernel.org List-ID: On Sun, Nov 18, 2018 at 06:32:00PM +0100, Marek Vasut wrote: > Document the support for rcar_canfd on R8A77965 SoC devices. > > Signed-off-by: Marek Vasut > Cc: Eugeniu Rosca > Cc: Geert Uytterhoeven > Cc: Marc Kleine-Budde > Cc: Rob Herring > Cc: Simon Horman > Cc: Wolfram Sang > Cc: linux-renesas-soc@vger.kernel.org > --- > .../devicetree/bindings/net/can/rcar_canfd.txt | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > index ac71daa46195..4720e916fbdd 100644 > --- a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > +++ b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > @@ -6,6 +6,7 @@ Required properties: > - "renesas,rcar-gen3-canfd" for R-Car Gen3 compatible controller. > - "renesas,r8a7795-canfd" for R8A7795 (R-Car H3) compatible controller. > - "renesas,r8a7796-canfd" for R8A7796 (R-Car M3-W) compatible controller. > + - "renesas,r8a77965-canfd" for R8A77965 (R-Car M3-N) compatible controller. > - "renesas,r8a77970-canfd" for R8A77970 (R-Car V3M) compatible controller. > - "renesas,r8a77980-canfd" for R8A77980 (R-Car V3H) compatible controller. > > @@ -26,12 +27,12 @@ The name of the child nodes are "channel0" and "channel1" respectively. Each > child node supports the "status" property only, which is used to > enable/disable the respective channel. > > -Required properties for "renesas,r8a7795-canfd" and "renesas,r8a7796-canfd" > -compatible: > -In R8A7795 and R8A7796 SoCs, canfd clock is a div6 clock and can be used by both > -CAN and CAN FD controller at the same time. It needs to be scaled to maximum > -frequency if any of these controllers use it. This is done using the below > -properties: > +Required properties for "renesas,r8a7795-canfd", "renesas,r8a7796-canfd" and > +"renesas,r8a77965-canfd" compatible: > +In R8A7795, R8A7796 and R8A77965 SoCs, canfd clock is a div6 clock and can Do we have to list the SoCs twice so that the paragraph has to be reformatted every time? Just the compatibles above should be enough. > +be used by both CAN and CAN FD controller at the same time. It needs to be > +scaled to maximum frequency if any of these controllers use it. This is done > +using the below properties: > > - assigned-clocks: phandle of canfd clock. > - assigned-clock-rates: maximum frequency of this clock. > -- > 2.18.0 >