Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757320AbcJ3TZc (ORCPT ); Sun, 30 Oct 2016 15:25:32 -0400 Received: from mail-oi0-f65.google.com ([209.85.218.65]:34018 "EHLO mail-oi0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753069AbcJ3TZ3 (ORCPT ); Sun, 30 Oct 2016 15:25:29 -0400 MIME-Version: 1.0 In-Reply-To: <1477776746-3678-2-git-send-email-joel@airwebreathe.org.uk> References: <1477776746-3678-1-git-send-email-joel@airwebreathe.org.uk> <1477776746-3678-2-git-send-email-joel@airwebreathe.org.uk> From: Geert Uytterhoeven Date: Sun, 30 Oct 2016 20:25:27 +0100 X-Google-Sender-Auth: _fHyy7kZy-DZZWMg_1kCV7_3Jrg Message-ID: Subject: Re: [PATCH v4 2/2] fpga: Add support for Lattice iCE40 FPGAs To: Joel Holdsworth Cc: atull@opensource.altera.com, moritz.fischer@ettus.com, "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-spi Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1697 Lines: 36 On Sat, Oct 29, 2016 at 11:32 PM, Joel Holdsworth wrote: > --- /dev/null > +++ b/Documentation/devicetree/bindings/fpga/lattice-ice40-fpga-mgr.txt > @@ -0,0 +1,23 @@ > +- cdone-gpio: GPIO connected to CDONE pin > +- creset_b-gpio: GPIO connected to CRESET_B pin. Note that CRESET_B is > + treated as an active-low output because the signal is > + treated as an enable signal, rather than a reset. This > + is necessary because the FPGA will enter Master SPI > + mode and drive SCK with a clock signal, potentially > + jamming other devices on the bus, unless CRESET_B is > + held high until the firmware is loaded. Plural, please, like in the previous version, cfr. Documentation/devicetree/bindings/gpio/gpio.txt: "GPIO properties should be named "[-]gpios", with being the purpose of this GPIO for the device. While a non-existent is considered valid for compatibility reasons (resolving to the "gpios" property), it is not allowed for new bindings. Also, GPIO properties named "[-]gpio" are valid and old bindings use it, but are only supported for compatibility reasons and should not be used for newer bindings since it has been deprecated." Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds