From: Chris Paterson <Chris.Paterson2@renesas.com>
To: "Simon Horman" <horms@verge.net.au>, "Andreas Färber" <afaerber@suse.de>
Cc: "Fabrizio Castro" <fabrizio.castro@bp.renesas.com>,
"Rob Herring" <robh+dt@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Wolfgang Grandegger" <wg@grandegger.com>,
"Marc Kleine-Budde" <mkl@pengutronix.de>,
"Michael Turquette" <mturquette@baylibre.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Biju Das" <biju.das@bp.renesas.com>,
"Magnus Damm" <magnus.damm@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
"Geert Uytterhoeven" <geert+renesas@glider.be>,
"Thierry Reding" <treding@nvidia.com>,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Kevin Hilman" <khilman@baylibre.com>,
"Johan Hovold" <johan@kernel.org>,
"Lukasz Majewski" <lukma@denx.de>,
"Michal Simek" <monstr@monstr.eu>,
"Michal Vokáč" <michal.vokac@ysoft.com>,
"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>,
"Ben Whitten" <ben.whitten@gmail.com>,
"linux-renesas-soc@vger.kernel.org"
<linux-renesas-soc@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-can@vger.kernel.org" <linux-can@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"ebiharaml@si-linux.co.jp" <ebiharaml@si-linux.co.jp>
Subject: RE: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon Linux.
Date: Thu, 17 Jan 2019 12:23:55 +0000 [thread overview]
Message-ID: <TYAPR01MB228595A79ABDB0D7826FED0FB7830@TYAPR01MB2285.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <20190117121919.c5u66sorut5a5nib@verge.net.au>
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)
> >
next prev parent reply other threads:[~2019-01-17 12:24 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-16 18:37 [PATCH 00/11] Add basic EK874 support Fabrizio Castro
2019-01-16 18:37 ` [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon Linux Fabrizio Castro
2019-01-17 10:44 ` Simon Horman
2019-01-17 11:17 ` Andreas Färber
2019-01-17 12:19 ` Simon Horman
2019-01-17 12:23 ` Chris Paterson [this message]
2019-01-22 1:14 ` Rob Herring
2019-02-14 16:21 ` Fabrizio Castro
2019-02-15 6:52 ` Lukasz Majewski
2019-02-15 9:09 ` Andreas Färber
2019-03-01 12:03 ` Fabrizio Castro
2019-03-04 9:40 ` Simon Horman
2019-01-16 18:37 ` [PATCH 09/11] dt-bindings: can: rcar_can: Fix RZ/G2 CAN clocks Fabrizio Castro
2019-01-17 12:01 ` Simon Horman
2019-01-22 1:15 ` Rob Herring
2019-03-01 11:52 ` Fabrizio Castro
2019-01-16 18:37 ` [PATCH 11/11] dt-bindings: can: rcar_canfd: document r8a774c0 support Fabrizio Castro
2019-01-17 12:11 ` Simon Horman
2019-01-22 1:15 ` Rob Herring
2019-03-01 11:56 ` Fabrizio Castro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=TYAPR01MB228595A79ABDB0D7826FED0FB7830@TYAPR01MB2285.jpnprd01.prod.outlook.com \
--to=chris.paterson2@renesas.com \
--cc=afaerber@suse.de \
--cc=alexandre.belloni@bootlin.com \
--cc=ben.whitten@gmail.com \
--cc=biju.das@bp.renesas.com \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=ebiharaml@si-linux.co.jp \
--cc=fabrizio.castro@bp.renesas.com \
--cc=geert+renesas@glider.be \
--cc=horms@verge.net.au \
--cc=johan@kernel.org \
--cc=khilman@baylibre.com \
--cc=linux-can@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=lukma@denx.de \
--cc=magnus.damm@gmail.com \
--cc=mark.rutland@arm.com \
--cc=martin.blumenstingl@googlemail.com \
--cc=michal.vokac@ysoft.com \
--cc=mkl@pengutronix.de \
--cc=monstr@monstr.eu \
--cc=mturquette@baylibre.com \
--cc=netdev@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=treding@nvidia.com \
--cc=wg@grandegger.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®