All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
To: Simon Horman <horms@verge.net.au>
Cc: Thierry Reding <thierry.reding@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	"linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Chris Paterson <Chris.Paterson2@renesas.com>,
	Biju Das <biju.das@bp.renesas.com>,
	"linux-renesas-soc@vger.kernel.org"
	<linux-renesas-soc@vger.kernel.org>
Subject: RE: [PATCH v3 2/5] dt-bindings: pwm: rcar: Document r8a774[35] PWM bindings
Date: Wed, 20 Dec 2017 09:54:14 +0000	[thread overview]
Message-ID: <TY1PR06MB08955E0D55B2712C6105A2F0C00C0@TY1PR06MB0895.apcprd06.prod.outlook.com> (raw)
In-Reply-To: <20171220083137.63jj4rrkztguvvbm@verge.net.au>

Hello Simon,

> Subject: Re: [PATCH v3 2/5] dt-bindings: pwm: rcar: Document r8a774[35] PWM bindings
>
> On Tue, Dec 19, 2017 at 01:34:57PM +0000, Fabrizio Castro wrote:
> > This patch adds compatible strings specific to r8a774[35], no driver
> > change is needed as the fallback compatible string will activate the
> > right code. Also, this patch replaces the example with a DT snippet used
> > for adding PWM0 support to an r8a7743 based platform.
>
> What is the motivation for updating the example?

I am sorry I forgot to copy the explanation across from the first version of the patch:
https://patchwork.kernel.org/patch/10111831/

"the reason for replacing the example with one of a different SoC is that
the new example refers to recent code from a real use case. I hope that's ok."

Basically the r8a7743 will be the first platform with full upstream support, and I believe it is better to have up-to-date examples proven to work in the documentation.

> Could it be stated here?

Shall I send a new version for this?


Thanks,
Fab


[https://www2.renesas.eu/media/email/unicef_2017.jpg]

This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world.
We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year.



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.

  reply	other threads:[~2017-12-20  9:54 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-19 13:34 [PATCH v3 0/5] Add PWM and TPU support to r8a7743 Fabrizio Castro
2017-12-19 13:34 ` Fabrizio Castro
2017-12-19 13:34 ` Fabrizio Castro
2017-12-19 13:34 ` [PATCH v3 1/5] ARM: shmobile: defconfig: Enable PWM Fabrizio Castro
2017-12-19 13:34   ` Fabrizio Castro
2017-12-20  8:30   ` Simon Horman
2017-12-20  8:30     ` Simon Horman
     [not found] ` <1513690500-12165-1-git-send-email-fabrizio.castro-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
2017-12-19 13:34   ` [PATCH v3 2/5] dt-bindings: pwm: rcar: Document r8a774[35] PWM bindings Fabrizio Castro
2017-12-19 13:34     ` Fabrizio Castro
2017-12-20  8:31     ` Simon Horman
2017-12-20  9:54       ` Fabrizio Castro [this message]
2017-12-20 10:12         ` Simon Horman
2017-12-20 11:15           ` Fabrizio Castro
2017-12-19 13:34   ` [PATCH v3 3/5] ARM: dts: r8a7743: Add PWM SoC support Fabrizio Castro
2017-12-19 13:34     ` Fabrizio Castro
2017-12-19 13:34     ` Fabrizio Castro
2017-12-20  8:45     ` Simon Horman
2017-12-20  8:45       ` Simon Horman
2017-12-19 13:34 ` [PATCH v3 4/5] dt-bindings: pwm: renesas-tpu: Document r8a774[35] support Fabrizio Castro
     [not found]   ` <1513690500-12165-5-git-send-email-fabrizio.castro-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
2017-12-20  8:46     ` Simon Horman
2017-12-20  8:46       ` Simon Horman
2018-01-24 17:16   ` Fabrizio Castro
2018-03-06 16:52     ` Fabrizio Castro
2018-03-06 16:52       ` Fabrizio Castro
2018-03-27 21:52   ` Thierry Reding
2017-12-19 13:35 ` [PATCH v3 5/5] ARM: dts: r8a7743: Add TPU support Fabrizio Castro
2017-12-19 13:35   ` Fabrizio Castro
2017-12-20  8:47   ` Simon Horman
2017-12-20  8:47     ` Simon Horman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=TY1PR06MB08955E0D55B2712C6105A2F0C00C0@TY1PR06MB0895.apcprd06.prod.outlook.com \
    --to=fabrizio.castro@bp.renesas.com \
    --cc=Chris.Paterson2@renesas.com \
    --cc=biju.das@bp.renesas.com \
    --cc=devicetree@vger.kernel.org \
    --cc=geert+renesas@glider.be \
    --cc=horms@verge.net.au \
    --cc=linux-pwm@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.