2020-11-17 17:52:31

by Guido Günther

Subject: [PATCH v1 5/6] dt-bindings: vendor-prefixes: Add ys vendor prefix

Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.

Signed-off-by: Guido Günther <[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 e40ee369f808..d0f3abf2f12c 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1220,6 +1220,8 @@ patternProperties:
description: Shenzhen Yangliming Electronic Technology Co., Ltd.
"^yna,.*":
description: YSH & ATIL
+ "^ys,.*":
+ description: Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
"^yones-toptech,.*":
description: Yones Toptech Co., Ltd.
"^ysoft,.*":
--
2.29.2

2020-11-17 20:46:00

by Linus Walleij

Subject: Re: [PATCH v1 5/6] dt-bindings: vendor-prefixes: Add ys vendor prefix

On Tue, Nov 17, 2020 at 6:49 PM Guido Günther <[email protected]> wrote:

> Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
>
> Signed-off-by: Guido Günther <[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 e40ee369f808..d0f3abf2f12c 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -1220,6 +1220,8 @@ patternProperties:
> description: Shenzhen Yangliming Electronic Technology Co., Ltd.
> "^yna,.*":
> description: YSH & ATIL
> + "^ys,.*":
> + description: Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
> "^yones-toptech,.*":
> description: Yones Toptech Co., Ltd.
> "^ysoft,.*":

I think this should be in alphabetical order of the compatible string, i.e.
under yones.

Yours,
Linus Walleij

2020-11-18 08:34:25

by Guido Günther

Subject: Re: [PATCH v1 5/6] dt-bindings: vendor-prefixes: Add ys vendor prefix

Hi,
On Tue, Nov 17, 2020 at 09:42:33PM +0100, Linus Walleij wrote:
> On Tue, Nov 17, 2020 at 6:49 PM Guido G?nther <[email protected]> wrote:
>
> > Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
> >
> > Signed-off-by: Guido G?nther <[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 e40ee369f808..d0f3abf2f12c 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> > @@ -1220,6 +1220,8 @@ patternProperties:
> > description: Shenzhen Yangliming Electronic Technology Co., Ltd.
> > "^yna,.*":
> > description: YSH & ATIL
> > + "^ys,.*":
> > + description: Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
> > "^yones-toptech,.*":
> > description: Yones Toptech Co., Ltd.
> > "^ysoft,.*":
>
> I think this should be in alphabetical order of the compatible string, i.e.
> under yones.

Messed that up during rebase. Fixed in v2.
Thanks for the review!
-- Guido

>
> Yours,
> Linus Walleij
>