2019-11-21 19:06:41

by Jason Kridner

[permalink] [raw]
Subject: [PATCH] dt-bindings: Add vendor prefix for BeagleBoard.org

Add vendor prefix for BeagleBoard.org Foundation

Signed-off-by: Jason Kridner <[email protected]>
---
Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index 967e78c5ec0a..3e3d8e3c28d3 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -139,6 +139,8 @@ patternProperties:
description: Shenzhen AZW Technology Co., Ltd.
"^bananapi,.*":
description: BIPAI KEJI LIMITED
+ "^beagleboard.org,.*":
+ description: BeagleBoard.org Foundation
"^bhf,.*":
description: Beckhoff Automation GmbH & Co. KG
"^bitmain,.*":
--
2.17.1


2019-11-21 23:18:54

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Add vendor prefix for BeagleBoard.org

On Thu, Nov 21, 2019 at 1:02 PM Jason Kridner <[email protected]> wrote:
>
> Add vendor prefix for BeagleBoard.org Foundation
>
> Signed-off-by: Jason Kridner <[email protected]>

Author and Sob emails don't match.

> ---
> Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> index 967e78c5ec0a..3e3d8e3c28d3 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -139,6 +139,8 @@ patternProperties:
> description: Shenzhen AZW Technology Co., Ltd.
> "^bananapi,.*":
> description: BIPAI KEJI LIMITED
> + "^beagleboard.org,.*":

'.' would need need to be escaped as this is a regex.

The '.' may cause some problems with schemas. We can adapt or just
drop the '.org'.

> + description: BeagleBoard.org Foundation
> "^bhf,.*":
> description: Beckhoff Automation GmbH & Co. KG
> "^bitmain,.*":
> --
> 2.17.1
>

2019-11-21 23:47:06

by Jason Kridner

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Add vendor prefix for BeagleBoard.org

On Thu, Nov 21, 2019 at 6:17 PM Rob Herring <[email protected]> wrote:
>
> On Thu, Nov 21, 2019 at 1:02 PM Jason Kridner <[email protected]> wrote:
> >
> > Add vendor prefix for BeagleBoard.org Foundation
> >
> > Signed-off-by: Jason Kridner <[email protected]>
>
> Author and Sob emails don't match.

I'll set the from address to [email protected] in another submission.

>
> > ---
> > Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > index 967e78c5ec0a..3e3d8e3c28d3 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > @@ -139,6 +139,8 @@ patternProperties:
> > description: Shenzhen AZW Technology Co., Ltd.
> > "^bananapi,.*":
> > description: BIPAI KEJI LIMITED
> > + "^beagleboard.org,.*":
>
> '.' would need need to be escaped as this is a regex.
>
> The '.' may cause some problems with schemas. We can adapt or just
> drop the '.org'.
>

I tried to run some YAML syntax checkers and they didn't like a
backslash ('\') before the period ('.'). YAML specification
documentation didn't specify how to escape a period.

My preference is to keep beagleboard.org, but, out of simplicity and
brevity, I can resubmit without it.

The device tree submission for BeagleBone AI will need to updated accordingly.

--
https://beagleboard.org/about