On Wed, Dec 20, 2017 at 09:54:14AM +0000, Fabrizio Castro wrote: > 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. Thanks, that sounds reasonable. > > > Could it be stated here? > > Shall I send a new version for this? That would be best in my opinion.
