Re: [PATCH] dt-bindings: sifive: describe sifive-blocks versioning

From: Paul Walmsley
Date: Thu Dec 06 2018 - 19:46:11 EST




On Thu, 6 Dec 2018, Rob Herring wrote:

> On Wed, Nov 21, 2018 at 05:06:56PM -0800, Paul Walmsley wrote:
> >
> > For IP blocks that are generated from the public, open-source
> > sifive-blocks repository, describe the version numbering policy
> > that its maintainers intend to use, upon request from Rob
> > Herring <robh@xxxxxxxxxx>.
> >
> > Cc: Rob Herring <robh+dt@xxxxxxxxxx>
> > Cc: Palmer Dabbelt <palmer@xxxxxxxxxx>
> > Cc: Megan Wachs <megan@xxxxxxxxxx>
> > Cc: Wesley Terpstra <wesley@xxxxxxxxxx>
> > Cc: Mark Rutland <mark.rutland@xxxxxxx>
> > Cc: devicetree@xxxxxxxxxxxxxxx
> > Cc: linux-riscv@xxxxxxxxxxxxxxxxxxx
> > Cc: linux-kernel@xxxxxxxxxxxxxxx
> > Signed-off-by: Paul Walmsley <paul.walmsley@xxxxxxxxxx>
> > Signed-off-by: Paul Walmsley <paul@xxxxxxxxx>
> > ---
> >
> > Hi Rob, please let me know if this document works with your
> > requirements, or if some changes are needed. - Paul
>
> Looks pretty good to me.
>
> > .../sifive/sifive-blocks-ip-versioning.txt | 38 +++++++++++++++++++
> > 1 file changed, 38 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/sifive/sifive-blocks-ip-versioning.txt
>
> Use the path that was suggested.

Could you remind me which one that is?

> > +
> > +An example is "sifive,uart0" from:
> > +
> > +https://github.com/sifive/sifive-blocks/blob/master/src/main/scala/devices/uart/UART.scala#L43
>
> That's nice, but will be out of date as soon as someone adds or removes
> a line above it. Can you point to a tagged version?

Will do

- Paul