diff mbox series

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

Message ID a4930fb7a7464e4feedbdae993fe25080c8f5490.1605635248.git.agx@sigxcpu.org (mailing list archive)
State New, archived
Headers show
Series drm/panel: mantix and st7703 fixes and additions | expand

Commit Message

Guido Günther Nov. 17, 2020, 5:49 p.m. UTC 
Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.

Signed-off-by: Guido Günther <agx@sigxcpu.org>
---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Linus Walleij Nov. 17, 2020, 8:42 p.m. UTC | #1 
On Tue, Nov 17, 2020 at 6:49 PM Guido Günther <agx@sigxcpu.org> wrote:

> Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
>
> Signed-off-by: Guido Günther <agx@sigxcpu.org>
> ---
>  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
Guido Günther Nov. 18, 2020, 8:30 a.m. UTC | #2 
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 <agx@sigxcpu.org> wrote:
> 
> > Add prefix for Shenzhen Yashi Changhua Intelligent Technology Co., Ltd.
> >
> > Signed-off-by: Guido Günther <agx@sigxcpu.org>
> > ---
> >  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
>
diff mbox series

Patch

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,.*":