* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts @ 2018-08-28 10:00 Marek Vasut 2018-08-29 9:14 ` Steffen Trumtrar 0 siblings, 1 reply; 9+ messages in thread From: Marek Vasut @ 2018-08-28 10:00 UTC (permalink / raw) To: linux-arm-kernel Hi, I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I had a hard time finding the right DT for the board, but then was suggested that the DT for the board is socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, since SoCKit is a different board altogether. But the DTS seems to be indeed for the DE0 Nano SoC. To fix up this confusion, I'd propose renaming of the socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts to match the label on the package . What do you think ? -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-28 10:00 [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts Marek Vasut @ 2018-08-29 9:14 ` Steffen Trumtrar 2018-08-29 10:33 ` Marek Vasut 0 siblings, 1 reply; 9+ messages in thread From: Steffen Trumtrar @ 2018-08-29 9:14 UTC (permalink / raw) To: linux-arm-kernel Marek Vasut <marex@denx.de> writes: > Hi, > > I was recently experimenting with the DE0 Nano SoC / Atlas SoC . > I had a > hard time finding the right DT for the board, but then was > suggested > that the DT for the board is socfpga_cyclone5_de0_sockit.dts . > This did > not make sense to me, since SoCKit is a different board > altogether. But > the DTS seems to be indeed for the DE0 Nano SoC. > > To fix up this confusion, I'd propose renaming of the > socfpga_cyclone5_de0_sockit.dts to > socfpga_cyclone5_de0_nano_soc.dts to > match the label on the package . > > What do you think ? Ack. This confusion came up multiple times. The barebox-specific dts "overlay" is already named socfpga_cyclone5_de0_nano_soc.dts because of this. Regards, Steffen -- Pengutronix e.K. | Steffen Trumtrar | 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] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 9:14 ` Steffen Trumtrar @ 2018-08-29 10:33 ` Marek Vasut 2018-08-29 14:40 ` Nguyen, Dinh 0 siblings, 1 reply; 9+ messages in thread From: Marek Vasut @ 2018-08-29 10:33 UTC (permalink / raw) To: linux-arm-kernel On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: > > Marek Vasut <marex@denx.de> writes: > >> Hi, >> >> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I had a >> hard time finding the right DT for the board, but then was suggested >> that the DT for the board is socfpga_cyclone5_de0_sockit.dts . This did >> not make sense to me, since SoCKit is a different board altogether. But >> the DTS seems to be indeed for the DE0 Nano SoC. >> >> To fix up this confusion, I'd propose renaming of the >> socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts to >> match the label on the package . >> >> What do you think ? > > Ack. This confusion came up multiple times. The barebox-specific dts > "overlay" > is already named socfpga_cyclone5_de0_nano_soc.dts because of this. The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts since forever. I'd be happy to send a patch if the Altera folks are fine with it? -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 10:33 ` Marek Vasut @ 2018-08-29 14:40 ` Nguyen, Dinh 2018-08-29 14:46 ` Marek Vasut 0 siblings, 1 reply; 9+ messages in thread From: Nguyen, Dinh @ 2018-08-29 14:40 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Marek Vasut <marex@denx.de> > Sent: Wednesday, August 29, 2018 5:34 AM > To: Steffen Trumtrar <s.trumtrar@pengutronix.de> > Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring > <robh+dt@kernel.org>; See, Chin Liang <chin.liang.see@intel.com>; Tan, Ley > Foon <ley.foon.tan@intel.com>; Nguyen, Dinh <dinh.nguyen@intel.com> > Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > socfpga_cyclone5_de0_nano_soc.dts > > On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: > > > > Marek Vasut <marex@denx.de> writes: > > > >> Hi, > >> > >> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I > >> had a hard time finding the right DT for the board, but then was > >> suggested that the DT for the board is > >> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, > >> since SoCKit is a different board altogether. But the DTS seems to be indeed > for the DE0 Nano SoC. > >> > >> To fix up this confusion, I'd propose renaming of the > >> socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts > >> to match the label on the package . > >> > >> What do you think ? > > > > Ack. This confusion came up multiple times. The barebox-specific dts > > "overlay" > > is already named socfpga_cyclone5_de0_nano_soc.dts because of this. > > The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts since > forever. I'd be happy to send a patch if the Altera folks are fine with it? > I'm fine with it. But this is the commit message when this board was originally create: The Terasic DE0 Atlas board is also known as the DE0-Nano board. This patch adds the DTS board file for the DE0-Nano Sockit board, and not the DE0 Nano "Development Board". Dinh ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 14:40 ` Nguyen, Dinh @ 2018-08-29 14:46 ` Marek Vasut 2018-08-29 14:54 ` Nguyen, Dinh 0 siblings, 1 reply; 9+ messages in thread From: Marek Vasut @ 2018-08-29 14:46 UTC (permalink / raw) To: linux-arm-kernel On 08/29/2018 04:40 PM, Nguyen, Dinh wrote: > > >> -----Original Message----- >> From: Marek Vasut <marex@denx.de> >> Sent: Wednesday, August 29, 2018 5:34 AM >> To: Steffen Trumtrar <s.trumtrar@pengutronix.de> >> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring >> <robh+dt@kernel.org>; See, Chin Liang <chin.liang.see@intel.com>; Tan, Ley >> Foon <ley.foon.tan@intel.com>; Nguyen, Dinh <dinh.nguyen@intel.com> >> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >> socfpga_cyclone5_de0_nano_soc.dts >> >> On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: >>> >>> Marek Vasut <marex@denx.de> writes: >>> >>>> Hi, >>>> >>>> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I >>>> had a hard time finding the right DT for the board, but then was >>>> suggested that the DT for the board is >>>> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, >>>> since SoCKit is a different board altogether. But the DTS seems to be indeed >> for the DE0 Nano SoC. >>>> >>>> To fix up this confusion, I'd propose renaming of the >>>> socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts >>>> to match the label on the package . >>>> >>>> What do you think ? >>> >>> Ack. This confusion came up multiple times. The barebox-specific dts >>> "overlay" >>> is already named socfpga_cyclone5_de0_nano_soc.dts because of this. >> >> The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts since >> forever. I'd be happy to send a patch if the Altera folks are fine with it? >> > > I'm fine with it. But this is the commit message when this board was originally create: > > The Terasic DE0 Atlas board is also known as the DE0-Nano board. > This patch adds the DTS board file for the DE0-Nano Sockit board, and not > the DE0 Nano "Development Board". But it was never called SoCkit anywhere , no documentation lists it as sockit. It is either DE0-Nano-SoC or Atlas-SoC , but that's it. Note that there is a DE0-Nano board (without the -SoC suffix) with Cyclone IV , but that one doesn't have the ARM cores and has different FPGA. So if that's clear, I'll just send a patch ? -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 14:46 ` Marek Vasut @ 2018-08-29 14:54 ` Nguyen, Dinh 2018-08-29 15:03 ` Marek Vasut 0 siblings, 1 reply; 9+ messages in thread From: Nguyen, Dinh @ 2018-08-29 14:54 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Marek Vasut <marex@denx.de> > Sent: Wednesday, August 29, 2018 9:47 AM > To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar > <s.trumtrar@pengutronix.de> > Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring > <robh+dt@kernel.org>; Westergreen, Dalon <dalon.westergreen@intel.com> > Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > socfpga_cyclone5_de0_nano_soc.dts > > On 08/29/2018 04:40 PM, Nguyen, Dinh wrote: > > > > > >> -----Original Message----- > >> From: Marek Vasut <marex@denx.de> > >> Sent: Wednesday, August 29, 2018 5:34 AM > >> To: Steffen Trumtrar <s.trumtrar@pengutronix.de> > >> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > >> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob > >> Herring <robh+dt@kernel.org>; See, Chin Liang > >> <chin.liang.see@intel.com>; Tan, Ley Foon <ley.foon.tan@intel.com>; > >> Nguyen, Dinh <dinh.nguyen@intel.com> > >> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > >> socfpga_cyclone5_de0_nano_soc.dts > >> > >> On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: > >>> > >>> Marek Vasut <marex@denx.de> writes: > >>> > >>>> Hi, > >>>> > >>>> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I > >>>> had a hard time finding the right DT for the board, but then was > >>>> suggested that the DT for the board is > >>>> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, > >>>> since SoCKit is a different board altogether. But the DTS seems to > >>>> be indeed > >> for the DE0 Nano SoC. > >>>> > >>>> To fix up this confusion, I'd propose renaming of the > >>>> socfpga_cyclone5_de0_sockit.dts to > >>>> socfpga_cyclone5_de0_nano_soc.dts to match the label on the package . > >>>> > >>>> What do you think ? > >>> > >>> Ack. This confusion came up multiple times. The barebox-specific dts > >>> "overlay" > >>> is already named socfpga_cyclone5_de0_nano_soc.dts because of this. > >> > >> The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts > >> since forever. I'd be happy to send a patch if the Altera folks are fine with it? > >> > > > > I'm fine with it. But this is the commit message when this board was originally > create: > > > > The Terasic DE0 Atlas board is also known as the DE0-Nano board. > > This patch adds the DTS board file for the DE0-Nano Sockit board, and not > > the DE0 Nano "Development Board". > > But it was never called SoCkit anywhere , no documentation lists it as sockit. It is > either DE0-Nano-SoC or Atlas-SoC , but that's it. https://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&No=941 > > Note that there is a DE0-Nano board (without the -SoC suffix) with Cyclone IV , > but that one doesn't have the ARM cores and has different FPGA. > > So if that's clear, I'll just send a patch ? > > -- > Best regards, > Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 14:54 ` Nguyen, Dinh @ 2018-08-29 15:03 ` Marek Vasut 2018-08-29 15:05 ` Nguyen, Dinh 0 siblings, 1 reply; 9+ messages in thread From: Marek Vasut @ 2018-08-29 15:03 UTC (permalink / raw) To: linux-arm-kernel On 08/29/2018 04:54 PM, Nguyen, Dinh wrote: > > >> -----Original Message----- >> From: Marek Vasut <marex@denx.de> >> Sent: Wednesday, August 29, 2018 9:47 AM >> To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar >> <s.trumtrar@pengutronix.de> >> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring >> <robh+dt@kernel.org>; Westergreen, Dalon <dalon.westergreen@intel.com> >> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >> socfpga_cyclone5_de0_nano_soc.dts >> >> On 08/29/2018 04:40 PM, Nguyen, Dinh wrote: >>> >>> >>>> -----Original Message----- >>>> From: Marek Vasut <marex@denx.de> >>>> Sent: Wednesday, August 29, 2018 5:34 AM >>>> To: Steffen Trumtrar <s.trumtrar@pengutronix.de> >>>> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >>>> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob >>>> Herring <robh+dt@kernel.org>; See, Chin Liang >>>> <chin.liang.see@intel.com>; Tan, Ley Foon <ley.foon.tan@intel.com>; >>>> Nguyen, Dinh <dinh.nguyen@intel.com> >>>> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >>>> socfpga_cyclone5_de0_nano_soc.dts >>>> >>>> On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: >>>>> >>>>> Marek Vasut <marex@denx.de> writes: >>>>> >>>>>> Hi, >>>>>> >>>>>> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . I >>>>>> had a hard time finding the right DT for the board, but then was >>>>>> suggested that the DT for the board is >>>>>> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, >>>>>> since SoCKit is a different board altogether. But the DTS seems to >>>>>> be indeed >>>> for the DE0 Nano SoC. >>>>>> >>>>>> To fix up this confusion, I'd propose renaming of the >>>>>> socfpga_cyclone5_de0_sockit.dts to >>>>>> socfpga_cyclone5_de0_nano_soc.dts to match the label on the package . >>>>>> >>>>>> What do you think ? >>>>> >>>>> Ack. This confusion came up multiple times. The barebox-specific dts >>>>> "overlay" >>>>> is already named socfpga_cyclone5_de0_nano_soc.dts because of this. >>>> >>>> The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts >>>> since forever. I'd be happy to send a patch if the Altera folks are fine with it? >>>> >>> >>> I'm fine with it. But this is the commit message when this board was originally >> create: >>> >>> The Terasic DE0 Atlas board is also known as the DE0-Nano board. >>> This patch adds the DTS board file for the DE0-Nano Sockit board, and not >>> the DE0 Nano "Development Board". >> >> But it was never called SoCkit anywhere , no documentation lists it as sockit. It is >> either DE0-Nano-SoC or Atlas-SoC , but that's it. > > https://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&No=941 Half of the boards from terasic have this "kit" suffix in the listing, but with a space in front of it. The SoCkit (without space) is: https://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&CategoryNo=167&No=816 -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 15:03 ` Marek Vasut @ 2018-08-29 15:05 ` Nguyen, Dinh 2018-08-29 15:07 ` Marek Vasut 0 siblings, 1 reply; 9+ messages in thread From: Nguyen, Dinh @ 2018-08-29 15:05 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Marek Vasut <marex@denx.de> > Sent: Wednesday, August 29, 2018 10:03 AM > To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar > <s.trumtrar@pengutronix.de> > Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring > <robh+dt@kernel.org>; Westergreen, Dalon <dalon.westergreen@intel.com> > Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > socfpga_cyclone5_de0_nano_soc.dts > > On 08/29/2018 04:54 PM, Nguyen, Dinh wrote: > > > > > >> -----Original Message----- > >> From: Marek Vasut <marex@denx.de> > >> Sent: Wednesday, August 29, 2018 9:47 AM > >> To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar > >> <s.trumtrar@pengutronix.de> > >> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > >> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob > >> Herring <robh+dt@kernel.org>; Westergreen, Dalon > >> <dalon.westergreen@intel.com> > >> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > >> socfpga_cyclone5_de0_nano_soc.dts > >> > >> On 08/29/2018 04:40 PM, Nguyen, Dinh wrote: > >>> > >>> > >>>> -----Original Message----- > >>>> From: Marek Vasut <marex@denx.de> > >>>> Sent: Wednesday, August 29, 2018 5:34 AM > >>>> To: Steffen Trumtrar <s.trumtrar@pengutronix.de> > >>>> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen > >>>> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob > >>>> Herring <robh+dt@kernel.org>; See, Chin Liang > >>>> <chin.liang.see@intel.com>; Tan, Ley Foon <ley.foon.tan@intel.com>; > >>>> Nguyen, Dinh <dinh.nguyen@intel.com> > >>>> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to > >>>> socfpga_cyclone5_de0_nano_soc.dts > >>>> > >>>> On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: > >>>>> > >>>>> Marek Vasut <marex@denx.de> writes: > >>>>> > >>>>>> Hi, > >>>>>> > >>>>>> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . > >>>>>> I had a hard time finding the right DT for the board, but then > >>>>>> was suggested that the DT for the board is > >>>>>> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, > >>>>>> since SoCKit is a different board altogether. But the DTS seems > >>>>>> to be indeed > >>>> for the DE0 Nano SoC. > >>>>>> > >>>>>> To fix up this confusion, I'd propose renaming of the > >>>>>> socfpga_cyclone5_de0_sockit.dts to > >>>>>> socfpga_cyclone5_de0_nano_soc.dts to match the label on the package > . > >>>>>> > >>>>>> What do you think ? > >>>>> > >>>>> Ack. This confusion came up multiple times. The barebox-specific > >>>>> dts "overlay" > >>>>> is already named socfpga_cyclone5_de0_nano_soc.dts because of this. > >>>> > >>>> The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts > >>>> since forever. I'd be happy to send a patch if the Altera folks are fine with > it? > >>>> > >>> > >>> I'm fine with it. But this is the commit message when this board was > >>> originally > >> create: > >>> > >>> The Terasic DE0 Atlas board is also known as the DE0-Nano board. > >>> This patch adds the DTS board file for the DE0-Nano Sockit board, and not > >>> the DE0 Nano "Development Board". > >> > >> But it was never called SoCkit anywhere , no documentation lists it > >> as sockit. It is either DE0-Nano-SoC or Atlas-SoC , but that's it. > > > > https://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&No > > =941 > > Half of the boards from terasic have this "kit" suffix in the listing, but with a > space in front of it. The SoCkit (without space) is: > > https://www.terasic.com.tw/cgi- > bin/page/archive.pl?Language=English&CategoryNo=167&No=816 > Right, just noticed that. Pretty clear naming scheme, I think! ? Dinh ^ permalink raw reply [flat|nested] 9+ messages in thread
* [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts 2018-08-29 15:05 ` Nguyen, Dinh @ 2018-08-29 15:07 ` Marek Vasut 0 siblings, 0 replies; 9+ messages in thread From: Marek Vasut @ 2018-08-29 15:07 UTC (permalink / raw) To: linux-arm-kernel On 08/29/2018 05:05 PM, Nguyen, Dinh wrote: > > >> -----Original Message----- >> From: Marek Vasut <marex@denx.de> >> Sent: Wednesday, August 29, 2018 10:03 AM >> To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar >> <s.trumtrar@pengutronix.de> >> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob Herring >> <robh+dt@kernel.org>; Westergreen, Dalon <dalon.westergreen@intel.com> >> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >> socfpga_cyclone5_de0_nano_soc.dts >> >> On 08/29/2018 04:54 PM, Nguyen, Dinh wrote: >>> >>> >>>> -----Original Message----- >>>> From: Marek Vasut <marex@denx.de> >>>> Sent: Wednesday, August 29, 2018 9:47 AM >>>> To: Nguyen, Dinh <dinh.nguyen@intel.com>; Steffen Trumtrar >>>> <s.trumtrar@pengutronix.de> >>>> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >>>> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob >>>> Herring <robh+dt@kernel.org>; Westergreen, Dalon >>>> <dalon.westergreen@intel.com> >>>> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >>>> socfpga_cyclone5_de0_nano_soc.dts >>>> >>>> On 08/29/2018 04:40 PM, Nguyen, Dinh wrote: >>>>> >>>>> >>>>>> -----Original Message----- >>>>>> From: Marek Vasut <marex@denx.de> >>>>>> Sent: Wednesday, August 29, 2018 5:34 AM >>>>>> To: Steffen Trumtrar <s.trumtrar@pengutronix.de> >>>>>> Cc: linux ARM <linux-arm-kernel@lists.infradead.org>; Dinh Nguyen >>>>>> <dingnuyen@kernel.org>; Jan Kiszka <jan.kiszka@siemens.com>; Rob >>>>>> Herring <robh+dt@kernel.org>; See, Chin Liang >>>>>> <chin.liang.see@intel.com>; Tan, Ley Foon <ley.foon.tan@intel.com>; >>>>>> Nguyen, Dinh <dinh.nguyen@intel.com> >>>>>> Subject: Re: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to >>>>>> socfpga_cyclone5_de0_nano_soc.dts >>>>>> >>>>>> On 08/29/2018 11:14 AM, Steffen Trumtrar wrote: >>>>>>> >>>>>>> Marek Vasut <marex@denx.de> writes: >>>>>>> >>>>>>>> Hi, >>>>>>>> >>>>>>>> I was recently experimenting with the DE0 Nano SoC / Atlas SoC . >>>>>>>> I had a hard time finding the right DT for the board, but then >>>>>>>> was suggested that the DT for the board is >>>>>>>> socfpga_cyclone5_de0_sockit.dts . This did not make sense to me, >>>>>>>> since SoCKit is a different board altogether. But the DTS seems >>>>>>>> to be indeed >>>>>> for the DE0 Nano SoC. >>>>>>>> >>>>>>>> To fix up this confusion, I'd propose renaming of the >>>>>>>> socfpga_cyclone5_de0_sockit.dts to >>>>>>>> socfpga_cyclone5_de0_nano_soc.dts to match the label on the package >> . >>>>>>>> >>>>>>>> What do you think ? >>>>>>> >>>>>>> Ack. This confusion came up multiple times. The barebox-specific >>>>>>> dts "overlay" >>>>>>> is already named socfpga_cyclone5_de0_nano_soc.dts because of this. >>>>>> >>>>>> The DT in U-Boot is also called socfpga_cyclone5_de0_nano_soc.dts >>>>>> since forever. I'd be happy to send a patch if the Altera folks are fine with >> it? >>>>>> >>>>> >>>>> I'm fine with it. But this is the commit message when this board was >>>>> originally >>>> create: >>>>> >>>>> The Terasic DE0 Atlas board is also known as the DE0-Nano board. >>>>> This patch adds the DTS board file for the DE0-Nano Sockit board, and not >>>>> the DE0 Nano "Development Board". >>>> >>>> But it was never called SoCkit anywhere , no documentation lists it >>>> as sockit. It is either DE0-Nano-SoC or Atlas-SoC , but that's it. >>> >>> https://www.terasic.com.tw/cgi-bin/page/archive.pl?Language=English&No >>> =941 >> >> Half of the boards from terasic have this "kit" suffix in the listing, but with a >> space in front of it. The SoCkit (without space) is: >> >> https://www.terasic.com.tw/cgi- >> bin/page/archive.pl?Language=English&CategoryNo=167&No=816 >> > > Right, just noticed that. Pretty clear naming scheme, I think! ? Jupp, it's confusing all right. -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-08-29 15:07 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-08-28 10:00 [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts Marek Vasut 2018-08-29 9:14 ` Steffen Trumtrar 2018-08-29 10:33 ` Marek Vasut 2018-08-29 14:40 ` Nguyen, Dinh 2018-08-29 14:46 ` Marek Vasut 2018-08-29 14:54 ` Nguyen, Dinh 2018-08-29 15:03 ` Marek Vasut 2018-08-29 15:05 ` Nguyen, Dinh 2018-08-29 15:07 ` Marek Vasut
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.