* [PATCH] dts: add specific compatible type for Terasic DE0-NANO-SoC Board
@ 2016-02-12 13:35 Tim Sander
2016-02-24 16:50 ` Dinh Nguyen
0 siblings, 1 reply; 7+ messages in thread
From: Tim Sander @ 2016-02-12 13:35 UTC (permalink / raw)
To: linux-kernel
Cc: Rob Herring, Dinh Nguyen, Pawel Moll, Mark Rutland, Ian Campbell,
Kumar Gala, Russell King, devicetree, linux-arm-kernel
From: Tim Sander <tim@krieglstein.org>
Add a more specific compatible string:"terasic,de0-nano-soc" for respective board.
Background: when checking for bootspec entries, some board specific fixups
are not apropriate for board of the same platform ("altr,socfpga-cyclone5").
The same aproach is taken with the EBV-Socrates board.
Signed-off-by: Tim Sander <tim@krieglstein.org>
---
arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts
index 555e9caf21e1..3a427423168e 100644
--- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts
+++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts
@@ -18,7 +18,7 @@
/ {
model = "Terasic DE-0(Atlas)";
- compatible = "altr,socfpga-cyclone5", "altr,socfpga";
+ compatible = "terasic,de0-nano-soc"," altr,socfpga-cyclone5", "altr,socfpga";
chosen {
bootargs = "earlyprintk";
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH] dts: add specific compatible type for Terasic DE0-NANO-SoC Board 2016-02-12 13:35 [PATCH] dts: add specific compatible type for Terasic DE0-NANO-SoC Board Tim Sander @ 2016-02-24 16:50 ` Dinh Nguyen 2016-02-25 10:05 ` [PATCH v2] " Tim Sander 0 siblings, 1 reply; 7+ messages in thread From: Dinh Nguyen @ 2016-02-24 16:50 UTC (permalink / raw) To: Tim Sander Cc: linux-kernel, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree, linux-arm-kernel Hi Tim, On Fri, 12 Feb 2016, Tim Sander wrote: > From: Tim Sander <tim@krieglstein.org> > > Add a more specific compatible string:"terasic,de0-nano-soc" for respective board. > Background: when checking for bootspec entries, some board specific fixups > are not apropriate for board of the same platform ("altr,socfpga-cyclone5"). > The same aproach is taken with the EBV-Socrates board. > Please fix up the spacing in the commit message. > Signed-off-by: Tim Sander <tim@krieglstein.org> > --- > arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > index 555e9caf21e1..3a427423168e 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > @@ -18,7 +18,7 @@ > > / { > model = "Terasic DE-0(Atlas)"; > - compatible = "altr,socfpga-cyclone5", "altr,socfpga"; > + compatible = "terasic,de0-nano-soc"," altr,socfpga-cyclone5", "altr,socfpga"; > "terasic" is not listed in Documentation/devicetree/bindings/vendor-prefixes.txt. BR, Dinh ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] dts: add specific compatible type for Terasic DE0-NANO-SoC Board 2016-02-24 16:50 ` Dinh Nguyen @ 2016-02-25 10:05 ` Tim Sander 2016-02-25 10:38 ` Steffen Trumtrar 0 siblings, 1 reply; 7+ messages in thread From: Tim Sander @ 2016-02-25 10:05 UTC (permalink / raw) To: Dinh Nguyen Cc: linux-kernel, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree, linux-arm-kernel From: Tim Sander <tim@krieglstein.org> Add a more specific compatible string:"terasic,de0-nano-soc" for respective board. Background: when checking for bootspec entries, some board specific fixups are not apropriate for board of the same platform ("altr,socfpga-cyclone5"). The same aproach is taken with the EBV-Socrates board. Signed-off-by: Tim Sander <tim@krieglstein.org> --- Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index 72e2c5a..d1f7803 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.txt +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt @@ -230,6 +230,7 @@ synology Synology, Inc. tbs TBS Technologies tcl Toby Churchill Ltd. technologic Technologic Systems +terasic Terasic Inc. thine THine Electronics, Inc. ti Texas Instruments tlm Trusted Logic Mobility diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts index afea364..704aa9d 100644 --- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts +++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts @@ -18,7 +18,7 @@ / { model = "Terasic DE-0(Atlas)"; - compatible = "altr,socfpga-cyclone5", "altr,socfpga"; + compatible = "terasic,de0-nano-soc","altr,socfpga-cyclone5", "altr,socfpga"; chosen { bootargs = "earlyprintk"; -- 1.9.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dts: add specific compatible type for Terasic DE0-NANO-SoC Board 2016-02-25 10:05 ` [PATCH v2] " Tim Sander @ 2016-02-25 10:38 ` Steffen Trumtrar 2016-02-25 16:56 ` Dinh Nguyen 0 siblings, 1 reply; 7+ messages in thread From: Steffen Trumtrar @ 2016-02-25 10:38 UTC (permalink / raw) To: Tim Sander Cc: Dinh Nguyen, linux-kernel, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree, linux-arm-kernel Hi Tim! On Thu, Feb 25, 2016 at 11:05:05AM +0100, Tim Sander wrote: > From: Tim Sander <tim@krieglstein.org> > > Add a more specific compatible string:"terasic,de0-nano-soc" for respective board. > Background: when checking for bootspec entries, some board specific fixups are > not apropriate for board of the same platform ("altr,socfpga-cyclone5"). > The same aproach is taken with the EBV-Socrates board. > > Signed-off-by: Tim Sander <tim@krieglstein.org> > --- > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +- > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt > index 72e2c5a..d1f7803 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt > @@ -230,6 +230,7 @@ synology Synology, Inc. > tbs TBS Technologies > tcl Toby Churchill Ltd. > technologic Technologic Systems > +terasic Terasic Inc. > thine THine Electronics, Inc. > ti Texas Instruments > tlm Trusted Logic Mobility You should IMHO split this up in two patches. First patch: add terasic > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > index afea364..704aa9d 100644 > --- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > +++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > @@ -18,7 +18,7 @@ > > / { > model = "Terasic DE-0(Atlas)"; > - compatible = "altr,socfpga-cyclone5", "altr,socfpga"; > + compatible = "terasic,de0-nano-soc","altr,socfpga-cyclone5", "altr,socfpga"; Second patch: this. > > chosen { > bootargs = "earlyprintk"; The naming of this board still confuses me though. It has 3 different names now: - de0_sockit.dts - Terasic DE-0(Atlas) - de0-nano-soc And according to Terasic DE0-Nano-SoC is the same as Atlas-SoC with a different software?! So all three names are actually correct ?! Weird. Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dts: add specific compatible type for Terasic DE0-NANO-SoC Board 2016-02-25 10:38 ` Steffen Trumtrar @ 2016-02-25 16:56 ` Dinh Nguyen [not found] ` <56CF323C.90202-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> 0 siblings, 1 reply; 7+ messages in thread From: Dinh Nguyen @ 2016-02-25 16:56 UTC (permalink / raw) To: Steffen Trumtrar, Tim Sander Cc: linux-kernel, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree, linux-arm-kernel On 02/25/2016 04:38 AM, Steffen Trumtrar wrote: > Hi Tim! > > On Thu, Feb 25, 2016 at 11:05:05AM +0100, Tim Sander wrote: >> From: Tim Sander <tim@krieglstein.org> >> >> Add a more specific compatible string:"terasic,de0-nano-soc" for respective board. >> Background: when checking for bootspec entries, some board specific fixups are >> not apropriate for board of the same platform ("altr,socfpga-cyclone5"). >> The same aproach is taken with the EBV-Socrates board. >> >> Signed-off-by: Tim Sander <tim@krieglstein.org> >> --- >> Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + >> arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +- >> 2 files changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt >> index 72e2c5a..d1f7803 100644 >> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt >> @@ -230,6 +230,7 @@ synology Synology, Inc. >> tbs TBS Technologies >> tcl Toby Churchill Ltd. >> technologic Technologic Systems >> +terasic Terasic Inc. >> thine THine Electronics, Inc. >> ti Texas Instruments >> tlm Trusted Logic Mobility > > You should IMHO split this up in two patches. > First patch: add terasic > That's right. That patch will go through the DTS maintainer's tree. >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts >> index afea364..704aa9d 100644 >> --- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts >> +++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts >> @@ -18,7 +18,7 @@ >> >> / { >> model = "Terasic DE-0(Atlas)"; >> - compatible = "altr,socfpga-cyclone5", "altr,socfpga"; >> + compatible = "terasic,de0-nano-soc","altr,socfpga-cyclone5", "altr,socfpga"; > So perhaps, "terasic,de0-sockit"? > Second patch: this. > And I can take this one. >> >> chosen { >> bootargs = "earlyprintk"; > > The naming of this board still confuses me though. > It has 3 different names now: > > - de0_sockit.dts > - Terasic DE-0(Atlas) > - de0-nano-soc > > And according to Terasic DE0-Nano-SoC is the same as Atlas-SoC with a different software?! > So all three names are actually correct ?! Weird. > I had a hard time understanding this myself. But from what I gather from[1], I just name the file de0_sockit. Dinh [1] http://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&CategoryNo=13&List=Simple#Category167 ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <56CF323C.90202-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>]
* Re: [PATCH v2] dts: add specific compatible type for Terasic DE0-NANO-SoC Board [not found] ` <56CF323C.90202-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org> @ 2016-02-25 23:34 ` Tim Sander 2016-02-25 23:38 ` Dinh Nguyen 0 siblings, 1 reply; 7+ messages in thread From: Tim Sander @ 2016-02-25 23:34 UTC (permalink / raw) To: Dinh Nguyen Cc: Steffen Trumtrar, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Dinh On Thursday 25 February 2016 10:56:28 Dinh Nguyen wrote: > On 02/25/2016 04:38 AM, Steffen Trumtrar wrote: > > Hi Tim! > > > > On Thu, Feb 25, 2016 at 11:05:05AM +0100, Tim Sander wrote: > >> From: Tim Sander <tim-cVflAIDvGO/+u4ArqExSyQ@public.gmane.org> > >> > >> Add a more specific compatible string:"terasic,de0-nano-soc" for > >> respective board. Background: when checking for bootspec entries, some > >> board specific fixups are not apropriate for board of the same platform > >> ("altr,socfpga-cyclone5"). The same aproach is taken with the > >> EBV-Socrates board. > >> > >> Signed-off-by: Tim Sander <tim-cVflAIDvGO/+u4ArqExSyQ@public.gmane.org> > >> --- > >> > >> Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > >> arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts | 2 +- > >> 2 files changed, 2 insertions(+), 1 deletion(-) > >> > >> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt > >> b/Documentation/devicetree/bindings/vendor-prefixes.txt index > >> 72e2c5a..d1f7803 100644 > >> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt > >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt > >> @@ -230,6 +230,7 @@ synology Synology, Inc. > >> > >> tbs TBS Technologies > >> tcl Toby Churchill Ltd. > >> technologic Technologic Systems > >> > >> +terasic Terasic Inc. > >> > >> thine THine Electronics, Inc. > >> ti Texas Instruments > >> tlm Trusted Logic Mobility > > > > You should IMHO split this up in two patches. > > First patch: add terasic > > That's right. That patch will go through the DTS maintainer's tree. Ah well for such a simple patch it turns out more complicated than thought :-) Will do as soon as there is agreement on a name which does not seem that easy... > > >> diff --git a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > >> b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts index > >> afea364..704aa9d 100644 > >> --- a/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > >> +++ b/arch/arm/boot/dts/socfpga_cyclone5_de0_sockit.dts > >> @@ -18,7 +18,7 @@ > >> > >> / { > >> > >> model = "Terasic DE-0(Atlas)"; > >> > >> - compatible = "altr,socfpga-cyclone5", "altr,socfpga"; > >> + compatible = "terasic,de0-nano-soc","altr,socfpga-cyclone5", > >> "altr,socfpga"; > So perhaps, "terasic,de0-sockit"? > > > Second patch: this. > > And I can take this one. > > >> chosen { > >> > >> bootargs = "earlyprintk"; > > > > The naming of this board still confuses me though. > > > > It has 3 different names now: > > - de0_sockit.dts > > - Terasic DE-0(Atlas) > > - de0-nano-soc > > > > And according to Terasic DE0-Nano-SoC is the same as Atlas-SoC with a > > different software?! So all three names are actually correct ?! Weird. > > I had a hard time understanding this myself. But from what I gather > from[1], I just name the file de0_sockit. As far as i remember there are different de0 and different sockit boards, so the name does not seem to be as concise? I don't care but i would say that de0-nano-soc is the most concise and easier to search for than atlas which might turn up more false postives? But as long as there is a more selective name than cyclone5 everthing is fine with me. Best regards Tim -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dts: add specific compatible type for Terasic DE0-NANO-SoC Board 2016-02-25 23:34 ` Tim Sander @ 2016-02-25 23:38 ` Dinh Nguyen 0 siblings, 0 replies; 7+ messages in thread From: Dinh Nguyen @ 2016-02-25 23:38 UTC (permalink / raw) To: Tim Sander Cc: Steffen Trumtrar, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On 02/25/2016 05:34 PM, Tim Sander wrote: > As far as i remember there are different de0 and different sockit boards, so > the name does not seem to be as concise? I don't care but i would say that > de0-nano-soc is the most concise and easier to search for than atlas which > might turn up more false postives? > I'm fine with "terasic,de0-nano-soc". Dinh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-02-25 23:38 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-12 13:35 [PATCH] dts: add specific compatible type for Terasic DE0-NANO-SoC Board Tim Sander
2016-02-24 16:50 ` Dinh Nguyen
2016-02-25 10:05 ` [PATCH v2] " Tim Sander
2016-02-25 10:38 ` Steffen Trumtrar
2016-02-25 16:56 ` Dinh Nguyen
[not found] ` <56CF323C.90202-yzvPICuk2ABMcg4IHK0kFoH6Mc4MB0Vx@public.gmane.org>
2016-02-25 23:34 ` Tim Sander
2016-02-25 23:38 ` Dinh Nguyen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).