From mboxrd@z Thu Jan 1 00:00:00 1970 From: marex@denx.de (Marek Vasut) Date: Wed, 29 Aug 2018 17:07:51 +0200 Subject: [RFC] Rename socfpga_cyclone5_de0_sockit.dts to socfpga_cyclone5_de0_nano_soc.dts In-Reply-To: References: <73bm9lwn3t.fsf@pengutronix.de> <9bdee446-5bc6-1171-eded-3c01623de820@denx.de> <2a991ba5-9746-007b-720a-91e4c4615d4d@denx.de> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 08/29/2018 05:05 PM, Nguyen, Dinh wrote: > > >> -----Original Message----- >> From: Marek Vasut >> Sent: Wednesday, August 29, 2018 10:03 AM >> To: Nguyen, Dinh ; Steffen Trumtrar >> >> Cc: linux ARM ; Dinh Nguyen >> ; Jan Kiszka ; Rob Herring >> ; Westergreen, Dalon >> 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 >>>> Sent: Wednesday, August 29, 2018 9:47 AM >>>> To: Nguyen, Dinh ; Steffen Trumtrar >>>> >>>> Cc: linux ARM ; Dinh Nguyen >>>> ; Jan Kiszka ; Rob >>>> Herring ; Westergreen, Dalon >>>> >>>> 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 >>>>>> Sent: Wednesday, August 29, 2018 5:34 AM >>>>>> To: Steffen Trumtrar >>>>>> Cc: linux ARM ; Dinh Nguyen >>>>>> ; Jan Kiszka ; Rob >>>>>> Herring ; See, Chin Liang >>>>>> ; Tan, Ley Foon ; >>>>>> Nguyen, Dinh >>>>>> 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 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