Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751400AbdFEPKv convert rfc822-to-8bit (ORCPT ); Mon, 5 Jun 2017 11:10:51 -0400 Received: from mail.kernel.org ([198.145.29.99]:46436 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751171AbdFEPKr (ORCPT ); Mon, 5 Jun 2017 11:10:47 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org F073D23A0F Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=atull@kernel.org MIME-Version: 1.0 In-Reply-To: <14677930.o4g43KgmZP@jclayton-pc> References: <20170525172911.11467-1-stillcompiling@gmail.com> <6513199.IxvCuS6eYj@jclayton-pc> <14677930.o4g43KgmZP@jclayton-pc> From: Alan Tull Date: Mon, 5 Jun 2017 10:10:05 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v11 4/6] ARM: dts: imx6q-evi: support altera-ps-spi To: Joshua Clayton Cc: =?UTF-8?Q?Andreas_F=C3=A4rber?= , Rob Herring , Moritz Fischer , Anatolij Gustschin , Bastian Stender , Shawn Guo , Mark Rutland , "devicetree@vger.kernel.org" , linux-fpga@vger.kernel.org, Russell King , linux-kernel , Sascha Hauer , Fabio Estevam , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2238 Lines: 66 On Fri, Jun 2, 2017 at 4:10 PM, wrote: > On Friday, June 2, 2017 9:54:22 PM PDT Andreas Färber wrote: >> Am 02.06.2017 um 21:39 schrieb stillcompiling@gmail.com: >> > On Friday, June 2, 2017 6:30:12 PM PDT Andreas Färber wrote: >> >> Am 25.05.2017 um 19:29 schrieb Joshua Clayton: >> >>> diff --git a/arch/arm/boot/dts/imx6q-evi.dts >> >>> b/arch/arm/boot/dts/imx6q-evi.dts index 24fe093a66db..a0cbb2d84803 >> >>> 100644 >> >>> --- a/arch/arm/boot/dts/imx6q-evi.dts >> >>> +++ b/arch/arm/boot/dts/imx6q-evi.dts >> >>> @@ -82,6 +82,15 @@ >> >>> >> >>> pinctrl-names = "default"; >> >>> pinctrl-0 = <&pinctrl_ecspi1 &pinctrl_ecspi1cs>; >> >>> status = "okay"; >> >>> >> >>> + >> >>> + fpga_spi: cyclonespi@0 { >> >> >> >> "cyclonespi" does not strike me as the best node name. >> >> >> >> I am guessing this is a sub-node of a SPI controller node, so no need to >> >> repeat "spi", and Cyclone seems more or less implied by "altr,fpga-". >> > >> > True. >> > >> >> Note that the example in the bindings doc uses "evi-fpga-spi". Nodes >> >> don't need to be (shouldn't be?) prefixed with the board. Note that >> >> bindings examples tend to get copied a lot. >> >> >> >> Any reason not to just use "fpga@0" in both places for simplicity? >> > >> > Sure. fpga: fpga@0 is probably better. >> >> Note that I was only commenting on the node name, the latter part. >> >> I'm not aware of any rules for the label, so that could remain unchanged >> or adopt cyclone_spi from the old node name or whatever is unique and >> syntactically valid. > Too late! Patches posted. > Oh, well, I'm not changing it back. It's fine as it is in v12. fpga_mgr: fpga-mgr@0 is what I've been using most recently. It distinguishes the one block that is used to program the fpga from the fpga and hardware in the fpga. But no need to respin this. Alan >> >> > I'll change it in both the dts and the binding doc. >> >> Thanks. Maybe double-check if there's any conventions Xilinx/Lattice DTs >> are using. >> > Of the conventions I found, fpga seemed the most "hardware descriptive" > for a plain FPGA. > The other one several binding doc examples are using is "fpga-mgr". > >> Cheers, >> Andreas > > > -- > ~Joshua A Clayton