Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753259AbdDJR5D (ORCPT ); Mon, 10 Apr 2017 13:57:03 -0400 Received: from mail-oi0-f67.google.com ([209.85.218.67]:33193 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751685AbdDJR5A (ORCPT ); Mon, 10 Apr 2017 13:57:00 -0400 Date: Mon, 10 Apr 2017 12:56:58 -0500 From: Rob Herring To: Richard Fitzgerald Cc: linus.walleij@linaro.org, gnurou@gmail.com, tglx@linutronix.de, jason@lakedaemon.net, lee.jones@linaro.org, broonie@kernel.org, alsa-devel@alsa-project.org, patches@opensource.wolfsonmicro.com, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 09/16] pinctrl: madera: Add driver for Cirrus Logic Madera codecs Message-ID: <20170410175658.a64tmev7hxzon2el@rob-hp-laptop> References: <1491386884-30689-1-git-send-email-rf@opensource.wolfsonmicro.com> <1491386884-30689-10-git-send-email-rf@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1491386884-30689-10-git-send-email-rf@opensource.wolfsonmicro.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1553 Lines: 30 On Wed, Apr 05, 2017 at 11:07:57AM +0100, Richard Fitzgerald wrote: > These codecs have a variable number of I/O lines each of which > is individually selectable to a wide range of possible functions. > > The functionality is slightly different from the traditional muxed > GPIO since most of the functions can be mapped to any pin (and even > the same function to multiple pins). Most pins have a dedicated > "alternate" function that is only available on that pin. The > alternate functions are usually a group of signals, though it is > not always necessary to enable the full group, depending on the > alternate function and how it is to be used. The mapping between > alternate functions and GPIO pins varies between codecs depending > on the number of alternate functions and available pins. > > Signed-off-by: Richard Fitzgerald > --- > .../bindings/pinctrl/cirrus,madera-pinctrl.txt | 103 ++ As Linus said, separate patch is preferred. But I don't have any other comments, so I'm not going to require it: Acked-by: Rob Herring > MAINTAINERS | 2 + > drivers/pinctrl/Kconfig | 22 + > drivers/pinctrl/Makefile | 1 + > drivers/pinctrl/pinctrl-madera.c | 1092 ++++++++++++++++++++ > 5 files changed, 1220 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pinctrl/cirrus,madera-pinctrl.txt > create mode 100644 drivers/pinctrl/pinctrl-madera.c