diff mbox series

[01/11] dt-bindings: Add vendor prefix for Silicon Linux.

Message ID 1547663874-29411-2-git-send-email-fabrizio.castro@bp.renesas.com
State Not Applicable
Delegated to: David Miller
Headers show
Series Add basic EK874 support | expand

Commit Message

Fabrizio Castro Jan. 16, 2019, 6:37 p.m. UTC
From: Biju Das <biju.das@bp.renesas.com>

Add Silicon Linux to the list of devicetree vendor prefixes.

Website: http://www.si-linux.co.jp

Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Simon Horman Jan. 17, 2019, 10:44 a.m. UTC | #1
On Wed, Jan 16, 2019 at 06:37:44PM +0000, Fabrizio Castro wrote:
> From: Biju Das <biju.das@bp.renesas.com>
> 
> Add Silicon Linux to the list of devicetree vendor prefixes.
> 
> Website: http://www.si-linux.co.jp
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index 4b1a2a8..02a7037 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -344,6 +344,7 @@ sgx	SGX Sensortech
>  sharp	Sharp Corporation
>  shimafuji	Shimafuji Electric, Inc.
>  si-en	Si-En Technology Ltd.
> +si-linux	Silicon Linux Corporation
>  sifive	SiFive, Inc.
>  sigma	Sigma Designs, Inc.
>  sii	Seiko Instruments, Inc.
> -- 
> 2.7.4
>
Andreas Färber Jan. 17, 2019, 11:17 a.m. UTC | #2
Hi Fabrizio,

Am 16.01.19 um 19:37 schrieb Fabrizio Castro:
> From: Biju Das <biju.das@bp.renesas.com>
> 
> Add Silicon Linux to the list of devicetree vendor prefixes.
> 
> Website: http://www.si-linux.co.jp
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index 4b1a2a8..02a7037 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -344,6 +344,7 @@ sgx	SGX Sensortech
>  sharp	Sharp Corporation
>  shimafuji	Shimafuji Electric, Inc.
>  si-en	Si-En Technology Ltd.
> +si-linux	Silicon Linux Corporation

With the website being Japanese only, I haven't been able to confirm the
spelling of the company name. Looks sensible though.

Due to "en" coming before "five" I've double-checked that elsewhere we
do order dashes before letters, so placement in file and tab-indentation
is fine, too.

>  sifive	SiFive, Inc.
>  sigma	Sigma Designs, Inc.
>  sii	Seiko Instruments, Inc.

Personally I would drop the full stop at the end of the subject, but anyway:

Reviewed-by: Andreas Färber <afaerber@suse.de>

Thanks,
Andreas

P.S. Please double-check how this excessive CC list came to life!
Simon Horman Jan. 17, 2019, 12:19 p.m. UTC | #3
On Thu, Jan 17, 2019 at 12:17:56PM +0100, Andreas Färber wrote:
> Hi Fabrizio,
> 
> Am 16.01.19 um 19:37 schrieb Fabrizio Castro:
> > From: Biju Das <biju.das@bp.renesas.com>
> > 
> > Add Silicon Linux to the list of devicetree vendor prefixes.
> > 
> > Website: http://www.si-linux.co.jp
> > 
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> > ---
> >  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > index 4b1a2a8..02a7037 100644
> > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > @@ -344,6 +344,7 @@ sgx	SGX Sensortech
> >  sharp	Sharp Corporation
> >  shimafuji	Shimafuji Electric, Inc.
> >  si-en	Si-En Technology Ltd.
> > +si-linux	Silicon Linux Corporation
> 
> With the website being Japanese only, I haven't been able to confirm the
> spelling of the company name. Looks sensible though.

According to the company web site
(http://si-linux.co.jp/index.php?%E4%BC%9A%E7%A4%BE%E6%A1%88%E5%86%85)
the company name is.

Japanese: シリコンリナックス株式会社
English:  Silicon Linux Corporation

And in my opinion these are equivalent (although other translations of
the corporation bit are possible).

> Due to "en" coming before "five" I've double-checked that elsewhere we
> do order dashes before letters, so placement in file and tab-indentation
> is fine, too.
> 
> >  sifive	SiFive, Inc.
> >  sigma	Sigma Designs, Inc.
> >  sii	Seiko Instruments, Inc.
> 
> Personally I would drop the full stop at the end of the subject, but anyway:
> 
> Reviewed-by: Andreas Färber <afaerber@suse.de>
> 
> Thanks,
> Andreas
> 
> P.S. Please double-check how this excessive CC list came to life!
> 
> -- 
> SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)
>
Chris Paterson Jan. 17, 2019, 12:23 p.m. UTC | #4
Hello Andreas, Simon,

Thank you for your reviews.

> From: Simon Horman <horms@verge.net.au>
> Sent: 17 January 2019 12:19
> 
> On Thu, Jan 17, 2019 at 12:17:56PM +0100, Andreas Färber wrote:
> > Hi Fabrizio,
> >
> > Am 16.01.19 um 19:37 schrieb Fabrizio Castro:
> > > From: Biju Das <biju.das@bp.renesas.com>
> > >
> > > Add Silicon Linux to the list of devicetree vendor prefixes.
> > >
> > > Website: http://www.si-linux.co.jp
> > >
> > > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > > Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> > > ---
> > >  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt
> b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > > index 4b1a2a8..02a7037 100644
> > > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> > > @@ -344,6 +344,7 @@ sgx	SGX Sensortech
> > >  sharp	Sharp Corporation
> > >  shimafuji	Shimafuji Electric, Inc.
> > >  si-en	Si-En Technology Ltd.
> > > +si-linux	Silicon Linux Corporation
> >
> > With the website being Japanese only, I haven't been able to confirm the
> > spelling of the company name. Looks sensible though.
> 
> According to the company web site
> (http://si-
> linux.co.jp/index.php?%E4%BC%9A%E7%A4%BE%E6%A1%88%E5%86%85)
> the company name is.
> 
> Japanese: シリコンリナックス株式会社
> English:  Silicon Linux Corporation

I can also confirm that this is what we've been told by Si-Linux themselves 😊

Kind regards, Chris

> 
> And in my opinion these are equivalent (although other translations of
> the corporation bit are possible).
> 
> > Due to "en" coming before "five" I've double-checked that elsewhere we
> > do order dashes before letters, so placement in file and tab-indentation
> > is fine, too.
> >
> > >  sifive	SiFive, Inc.
> > >  sigma	Sigma Designs, Inc.
> > >  sii	Seiko Instruments, Inc.
> >
> > Personally I would drop the full stop at the end of the subject, but anyway:
> >
> > Reviewed-by: Andreas Färber <afaerber@suse.de>
> >
> > Thanks,
> > Andreas
> >
> > P.S. Please double-check how this excessive CC list came to life!
> >
> > --
> > SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> > GF: Felix Imendörffer, Jane Smithard, Graham Norton
> > HRB 21284 (AG Nürnberg)
> >
Rob Herring (Arm) Jan. 22, 2019, 1:14 a.m. UTC | #5
On Wed, 16 Jan 2019 18:37:44 +0000, Fabrizio Castro wrote:
> From: Biju Das <biju.das@bp.renesas.com>
> 
> Add Silicon Linux to the list of devicetree vendor prefixes.
> 
> Website: http://www.si-linux.co.jp
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Reviewed-by: Chris Paterson <Chris.Paterson2@renesas.com>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>  1 file changed, 1 insertion(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 4b1a2a8..02a7037 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -344,6 +344,7 @@  sgx	SGX Sensortech
 sharp	Sharp Corporation
 shimafuji	Shimafuji Electric, Inc.
 si-en	Si-En Technology Ltd.
+si-linux	Silicon Linux Corporation
 sifive	SiFive, Inc.
 sigma	Sigma Designs, Inc.
 sii	Seiko Instruments, Inc.