RE: [PATCH v2 0/3] Renesas R9A06G032 PINCTRL Driver

From: Phil Edworthy
Date: Mon Sep 10 2018 - 05:54:23 EST


Hi Geert, Laurent,

On 10 September 2018 10:42 Geert Uytterhoeven wrote:
> On Mon, Sep 10, 2018 at 9:49 AM Linus Walleij wrote:
> > On Wed, Sep 5, 2018 at 11:59 AM Phil Edworthy wrote:
> > > On 05 September 2018 10:37, Geert Uytterhoeven wrote:
> >
> > > > Have the VxWorks DT bindings been submitted for review to the
> > > > devicetree mailing list?
> >
> > > I'm not involved with the VxWorks port, but I am pretty sure that
> > > they have not been submitted for review.
> >
> > We have had other cases where deployments on other OSes have been
> > establishing bindings (even not very pretty ones).
> >
> > Examples include typically the Power MacIntosh and the original SPARC
> > bindings.
>
> Which predated FDT support in Linux by more than a decade, with the latter
> predating even p1275?
>
> While I see merit in compatibility with other existing OSes, I prefer not to
> deviate from standard DT bindings, if they exist.
>
> Over time, we've even migrated from Renesas-specific DT bindings to
> standard DT bindings, in already upstreamed SoC support.
>
> If needed, I guess a script can be written to convert DTS files from the old to
> the new syntax?

Thanks for your comments, that is also my preference. Since I have to juggle
effort vs. features I thought it best to send the driver with existing out-of-tree
bindings first just to see.

I'll modify the driver for the standard bindings.

Thanks
Phil