diff mbox

[1/3] dt-bindings: Add vendor prefix for ProBox2

Message ID 20170827125425.3400-2-afaerber@suse.de
State New
Headers show

Commit Message

Andreas Färber Aug. 27, 2017, 12:54 p.m. UTC 
PROBOX2 is a Hong Kong based manufacturer of TV boxes.

Cc: support@probox2.com
Signed-off-by: Andreas Färber <afaerber@suse.de>
---
 I did not find a fully spelled out company name for this brand.
 
 http://probox2.com/contact-us/
 https://www.facebook.com/pg/probox2/about/
 
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Rob Herring (Arm) Aug. 31, 2017, 9:14 p.m. UTC | #1 
On Sun, Aug 27, 2017 at 02:54:23PM +0200, Andreas Färber wrote:
> PROBOX2 is a Hong Kong based manufacturer of TV boxes.
> 
> Cc: support@probox2.com
> Signed-off-by: Andreas Färber <afaerber@suse.de>
> ---
>  I did not find a fully spelled out company name for this brand.
>  
>  http://probox2.com/contact-us/
>  https://www.facebook.com/pg/probox2/about/
>  
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>  1 file changed, 1 insertion(+)

Acked-by: Rob Herring <robh@kernel.org>
Andreas Färber Sept. 2, 2017, 10:08 a.m. UTC | #2 
Hi Wilson,

Please note that HTML-format mails are not permitted on most Linux
mailing lists, so your below message did not arrive for the other reviewers.

Am 01.09.2017 um 04:11 schrieb PROBOX2-CS:
> Hi Rob,
> 
> For the company of the brand is W2COMP COMPANY LIMITED or you can write
> PROBOX2.

My updated proposal is:

probox2         PROBOX2 (by W2COMP Co., Ltd.)

Rationale: Repeating the same name with just different casing on the
right does not provide additional information to a reader wondering whom
this prefix belongs to and whether to use it for new compatible strings.

If that proposal is okay with you, we still need to figure out how to
describe your company in the commit message below.

> thanks,
> Wilson
> PROBOX2
> 
> On Fri, Sep 1, 2017 at 5:14 AM, Rob Herring <robh@kernel.org
> <mailto:robh@kernel.org>> wrote:
> 
>     On Sun, Aug 27, 2017 at 02:54:23PM +0200, Andreas Färber wrote:
>     > PROBOX2 is a Hong Kong based manufacturer of TV boxes.

W2COMP describes itself as an "online reseller", and PROBOX2 does not
have any About Us page to derive a fair description from - "reseller"
appears to contradict my above description as "manufacturer".
Therefore my (unanswered) off-list inquiry from Aug 28, when you first
mentioned W2COMP to me.

So what about: "PROBOX2 is a TV box brand by Hong Kong based online
reseller W2COMP Company Limited."?

CC'ing William.

Thanks,
Andreas


>     >
>     > Cc: support@probox2.com <mailto:support@probox2.com>
>     > Signed-off-by: Andreas Färber <afaerber@suse.de
>     <mailto:afaerber@suse.de>>
>     > ---
>     >  I did not find a fully spelled out company name for this brand.
>     >
>     >  http://probox2.com/contact-us/
>     >  https://www.facebook.com/pg/probox2/about/
>     <https://www.facebook.com/pg/probox2/about/>
>     >
>     >  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>     >  1 file changed, 1 insertion(+)
> 
>     Acked-by: Rob Herring <robh@kernel.org <mailto:robh@kernel.org>>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 37867c3cb94b..683b1693078f 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -261,6 +261,7 @@  plathome	Plat'Home Co., Ltd.
 plda	PLDA
 poslab	Poslab Technology Co., Ltd.
 powervr	PowerVR (deprecated, use img)
+probox2	PROBOX2
 pulsedlight	PulsedLight, Inc
 qca	Qualcomm Atheros, Inc.
 qcom	Qualcomm Technologies, Inc