Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1725475

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

From Andreas Färber <afaerber@suse.de>
Newsgroups linux.kernel
Subject Re: [PATCH 1/3] dt-bindings: Add vendor prefix for ProBox2
Date 2017-09-02 12:10 +0200
Message-ID <uldxf-1Em-1@gated-at.bofh.it> (permalink)
References <uj5u9-86f-5@gated-at.bofh.it> <uj5ua-86f-31@gated-at.bofh.it> <ukF2y-2z7-21@gated-at.bofh.it> <uldxf-1Em-3@gated-at.bofh.it>
Organization SUSE Linux GmbH

Show all headers | View raw


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>>

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH 0/3] arm64: dts: Initial ProBox2 Ava support Andreas Färber <afaerber@suse.de> - 2017-08-27 15:10 +0200
  [PATCH 1/3] dt-bindings: Add vendor prefix for ProBox2 Andreas Färber <afaerber@suse.de> - 2017-08-27 15:10 +0200
    Re: [PATCH 1/3] dt-bindings: Add vendor prefix for ProBox2 Rob Herring <robh@kernel.org> - 2017-08-31 23:20 +0200
      Re: [PATCH 1/3] dt-bindings: Add vendor prefix for ProBox2 Andreas Färber <afaerber@suse.de> - 2017-09-02 12:10 +0200

csiph-web