| Message ID | 20200220083508.792071-4-anarsoul@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add LCD support for Pine64 Pinebook 1080p | expand |
Hi Vasily On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> > --- > 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 6456a6dfd83d..a390a793422b 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -665,6 +665,8 @@ patternProperties: > description: Nexbox > "^nextthing,.*": > description: Next Thing Co. > + "^neweast,.*": > + description: Guangdong Neweast Optoelectronics CO., LT Alphabetical order. "new" comes before "nex". Sam
Hi Vasily, Thank you for the patch. On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> > --- > 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 6456a6dfd83d..a390a793422b 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -665,6 +665,8 @@ patternProperties: > description: Nexbox > "^nextthing,.*": > description: Next Thing Co. > + "^neweast,.*": > + description: Guangdong Neweast Optoelectronics CO., LT Google only returns two hits for this name, beside the ones related to this patch series. Are you sure this is the correct company name ? > "^newhaven,.*": > description: Newhaven Display International > "^ni,.*":
Hi Laurent. > > + "^neweast,.*": > > + description: Guangdong Neweast Optoelectronics CO., LT > > Google only returns two hits for this name, beside the ones related to > this patch series. Are you sure this is the correct company name ? Seems legit: http://www.eastbl.com/ But maybe their chinese name was better a basis for vendor prefix? Guangdong New Oriental Optoelectronics Sam
On Thu, Feb 20, 2020 at 1:21 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > Hi Laurent. > > > > + "^neweast,.*": > > > + description: Guangdong Neweast Optoelectronics CO., LT > > > > Google only returns two hits for this name, beside the ones related to > > this patch series. Are you sure this is the correct company name ? > > Seems legit: > http://www.eastbl.com/ > > But maybe their chinese name was better a basis for vendor prefix? > > Guangdong New Oriental Optoelectronics They call themselves "Guangdong NewEast Optoelectronics" in English, so I think it's better to keep it as is. > > Sam
On Thu, Feb 20, 2020 at 1:35 AM Sam Ravnborg <sam@ravnborg.org> wrote: > > Hi Vasily > > On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > > > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> > > --- > > 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 6456a6dfd83d..a390a793422b 100644 > > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > @@ -665,6 +665,8 @@ patternProperties: > > description: Nexbox > > "^nextthing,.*": > > description: Next Thing Co. > > + "^neweast,.*": > > + description: Guangdong Neweast Optoelectronics CO., LT > > Alphabetical order. > "new" comes before "nex". Will fix in v2 > > Sam
On Thu, Feb 20, 2020 at 5:56 AM Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > > Hi Vasily, Hi Laurent, > Thank you for the patch. > > On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > > > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> > > --- > > 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 6456a6dfd83d..a390a793422b 100644 > > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > @@ -665,6 +665,8 @@ patternProperties: > > description: Nexbox > > "^nextthing,.*": > > description: Next Thing Co. > > + "^neweast,.*": > > + description: Guangdong Neweast Optoelectronics CO., LT > > Google only returns two hits for this name, beside the ones related to > this patch series. Are you sure this is the correct company name ? That is what datasheet says: http://files.pine64.org/doc/datasheet/pinebook/11.6inches-1080P-IPS-LCD-Panel-spec-WJFH116008A.pdf > > "^newhaven,.*": > > description: Newhaven Display International > > "^ni,.*": > > -- > Regards, > > Laurent Pinchart
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 6456a6dfd83d..a390a793422b 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -665,6 +665,8 @@ patternProperties: description: Nexbox "^nextthing,.*": description: Next Thing Co. + "^neweast,.*": + description: Guangdong Neweast Optoelectronics CO., LT "^newhaven,.*": description: Newhaven Display International "^ni,.*":
Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+)