Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1164548AbdD1FXC (ORCPT ); Fri, 28 Apr 2017 01:23:02 -0400 Received: from kirsty.vergenet.net ([202.4.237.240]:45635 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755970AbdD1FWy (ORCPT ); Fri, 28 Apr 2017 01:22:54 -0400 Date: Fri, 28 Apr 2017 07:22:48 +0200 From: Simon Horman To: Geert Uytterhoeven Cc: Jacopo Mondi , Linus Walleij , Geert Uytterhoeven , Laurent Pinchart , Chris Brandt , Rob Herring , Mark Rutland , Russell King , Linux-Renesas , "linux-gpio@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v5 00/10] Renesas RZ/A1 pin and gpio controller Message-ID: <20170428052247.GG18349@verge.net.au> References: <1493281194-5200-1-git-send-email-jacopo+renesas@jmondi.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organisation: Horms Solutions BV User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1718 Lines: 49 On Thu, Apr 27, 2017 at 10:42:02AM +0200, Geert Uytterhoeven wrote: > Hi Jacopo, > > On Thu, Apr 27, 2017 at 10:19 AM, Jacopo Mondi > wrote: > > this is 5th round of gpio/pincontroller for RZ/A1 devices. > > > > I have updated the pin controller driver to use the newly introduced > > "pinctrl_enable()" function. > > This is required since v4.11-rc7 as otherwise, as reported by Chris Brandt, > > the pin controller does not start. > > > > I have incorporated your comments on the device tree bindings documentation, > > and added to pinctrl-generic.h header file two macros to unpack generic > > properties and their arguments. > > > > Tested with SCIF, RIIC, ETHER and gpio-leds on Genmai board. > > Thanks for the update! > > > Jacopo Mondi (10): > > pinctrl: generic: Add bi-directional and output-enable > > Already applied by LinusW. > > > pinctrl: generic: Add macros to unpack properties > > LinusW: do you want me to queue this together with the driver for v4.13, > or will you take this single patch for v4.12? > > > pinctrl: Renesas RZ/A1 pin and gpio controller > > dt-bindings: pinctrl: Add RZ/A1 bindings doc > > Will queue in sh-pfc-for-v4.13. > > > arm: dts: dt-bindings: Add Renesas RZ/A1 pinctrl header > > arm: dts: r7s72100: Add pin controller node > > arm: dts: genmai: Add SCIF2 pin group > > arm: dts: genmai: Add RIIC2 pin group > > arm: dts: genmai: Add user led device nodes > > arm: dts: genmai: Add ethernet pin group > > These are for Simon. > > Does applying the DTS changes before the driver introduce regressions? > If no, Simon can queue them for v4.13. > If yes, they'll have to wait for v4.14. That is my question too.