| Message ID | 1547663874-29411-2-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 295768f6207e412e145487c87993481774c55411 |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Add basic EK874 support | expand |
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 >
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!
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) >
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
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>
Dear All, Who is the best person for taking this patch? Thanks, Fab > From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > Sent: 16 January 2019 18:38 > Subject: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon Linux. > > 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 @@ sgxSGX Sensortech > sharpSharp Corporation > shimafujiShimafuji Electric, Inc. > si-enSi-En Technology Ltd. > +si-linuxSilicon Linux Corporation > sifiveSiFive, Inc. > sigmaSigma Designs, Inc. > siiSeiko Instruments, Inc. > -- > 2.7.4 Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647
Hi Fabrizio, The best approach is to run: git format patch -1 <your patch sha> And then scripts/get_maintainer.pl 0001-xxxxxxx You shall see the the list of people to send the patch. > Dear All, > > Who is the best person for taking this patch? > > Thanks, > Fab > > > From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > Sent: 16 January 2019 18:38 > > Subject: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon > > Linux. > > > > 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 @@ sgxSGX Sensortech sharpSharp Corporation > > shimafujiShimafuji Electric, Inc. > > si-enSi-En Technology Ltd. > > +si-linuxSilicon Linux Corporation > > sifiveSiFive, Inc. > > sigmaSigma Designs, Inc. > > siiSeiko Instruments, Inc. > > -- > > 2.7.4 > > > > Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael > Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, > Arcadiastrasse 10, 40472 Duesseldorf, > Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 > USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE > reg. no.: DE 14978647 Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
Hi Lukasz and Fabrizio, Am 15.02.19 um 07:52 schrieb Lukasz Majewski: > The best approach is to run: > > git format patch -1 <your patch sha> > > And then > > scripts/get_maintainer.pl 0001-xxxxxxx > > You shall see the the list of people to send the patch. No, that does not answer his question. And please don't top-post. >> Who is the best person for taking this patch? The answer is: whomever is taking dt-bindings patches in the following 10 patches that make use of this prefix, once you have a Reviewed-by from Rob. So, most likely the Renesas kernel maintainer(s), i.e. Simon, which differs from get_maintainers.pl of vendor-prefixes.txt. Regards, Andreas >>> From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> >>> Sent: 16 January 2019 18:38 >>> Subject: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon >>> Linux. >>> >>> 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 @@ sgxSGX Sensortech sharpSharp Corporation >>> shimafujiShimafuji Electric, Inc. >>> si-enSi-En Technology Ltd. >>> +si-linuxSilicon Linux Corporation >>> sifiveSiFive, Inc. >>> sigmaSigma Designs, Inc. >>> siiSeiko Instruments, Inc. >>> -- >>> 2.7.4 >> >> >> >> Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael >> Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, >> Arcadiastrasse 10, 40472 Duesseldorf, >> Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 >> USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE >> reg. no.: DE 14978647 > > > > > Best regards, > > Lukasz Majewski > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de >
Thank you Andreas. > From: Andreas Färber <afaerber@suse.de> > Sent: 15 February 2019 09:10 > Subject: Re: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon Linux. > > Hi Lukasz and Fabrizio, > > Am 15.02.19 um 07:52 schrieb Lukasz Majewski: > > The best approach is to run: > > > > git format patch -1 <your patch sha> > > > > And then > > > > scripts/get_maintainer.pl 0001-xxxxxxx > > > > You shall see the the list of people to send the patch. > > No, that does not answer his question. And please don't top-post. > > >> Who is the best person for taking this patch? > > The answer is: whomever is taking dt-bindings patches in the following > 10 patches that make use of this prefix, once you have a Reviewed-by > from Rob. So, most likely the Renesas kernel maintainer(s), i.e. Simon, > which differs from get_maintainers.pl of vendor-prefixes.txt. Simon, do you think you can take this patch? Thanks, Fab > > Regards, > Andreas > > >>> From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > >>> Sent: 16 January 2019 18:38 > >>> Subject: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon > >>> Linux. > >>> > >>> 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 @@ sgxSGX Sensortech sharpSharp Corporation > >>> shimafujiShimafuji Electric, Inc. > >>> si-enSi-En Technology Ltd. > >>> +si-linuxSilicon Linux Corporation > >>> sifiveSiFive, Inc. > >>> sigmaSigma Designs, Inc. > >>> siiSeiko Instruments, Inc. > >>> -- > >>> 2.7.4 > >> > >> > >> > >> Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael > >> Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, > >> Arcadiastrasse 10, 40472 Duesseldorf, > >> Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 > >> USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE > >> reg. no.: DE 14978647 > > > > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de > > > > > -- > SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Felix Imendörffer, Jane Smithard, Graham Norton > HRB 21284 (AG Nürnberg)
On Fri, Mar 01, 2019 at 12:03:06PM +0000, Fabrizio Castro wrote: > Thank you Andreas. > > > From: Andreas Färber <afaerber@suse.de> > > Sent: 15 February 2019 09:10 > > Subject: Re: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon Linux. > > > > Hi Lukasz and Fabrizio, > > > > Am 15.02.19 um 07:52 schrieb Lukasz Majewski: > > > The best approach is to run: > > > > > > git format patch -1 <your patch sha> > > > > > > And then > > > > > > scripts/get_maintainer.pl 0001-xxxxxxx > > > > > > You shall see the the list of people to send the patch. > > > > No, that does not answer his question. And please don't top-post. > > > > >> Who is the best person for taking this patch? > > > > The answer is: whomever is taking dt-bindings patches in the following > > 10 patches that make use of this prefix, once you have a Reviewed-by > > from Rob. So, most likely the Renesas kernel maintainer(s), i.e. Simon, > > which differs from get_maintainers.pl of vendor-prefixes.txt. > > Simon, do you think you can take this patch? Sure, I have queued it up for v5.2. > > Thanks, > Fab > > > > > Regards, > > Andreas > > > > >>> From: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > >>> Sent: 16 January 2019 18:38 > > >>> Subject: [PATCH 01/11] dt-bindings: Add vendor prefix for Silicon > > >>> Linux. > > >>> > > >>> 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 @@ sgxSGX Sensortech sharpSharp Corporation > > >>> shimafujiShimafuji Electric, Inc. > > >>> si-enSi-En Technology Ltd. > > >>> +si-linuxSilicon Linux Corporation > > >>> sifiveSiFive, Inc. > > >>> sigmaSigma Designs, Inc. > > >>> siiSeiko Instruments, Inc. > > >>> -- > > >>> 2.7.4 > > >> > > >> > > >> > > >> Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael > > >> Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, > > >> Arcadiastrasse 10, 40472 Duesseldorf, > > >> Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 > > >> USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE > > >> reg. no.: DE 14978647 > > > > > > > > > > > > > > > Best regards, > > > > > > Lukasz Majewski > > > > > > -- > > > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > > Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de > > > > > > > > > -- > > SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > > GF: Felix Imendörffer, Jane Smithard, Graham Norton > > HRB 21284 (AG Nürnberg) >
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.