From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752113AbcGARfm (ORCPT ); Fri, 1 Jul 2016 13:35:42 -0400 Received: from mail-pa0-f41.google.com ([209.85.220.41]:33772 "EHLO mail-pa0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751426AbcGARfk (ORCPT ); Fri, 1 Jul 2016 13:35:40 -0400 Date: Fri, 1 Jul 2016 10:35:37 -0700 From: Bjorn Andersson To: Linus Walleij Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, Stephen Boyd Subject: Re: [PATCH] pinctrl: qcom: add support for EBI2 Message-ID: <20160701173537.GL1190@tuxbot> References: <1467388364-22223-1-git-send-email-linus.walleij@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1467388364-22223-1-git-send-email-linus.walleij@linaro.org> 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 On Fri 01 Jul 08:52 PDT 2016, Linus Walleij wrote: > Add support to mux in the second external bus interface as > function 1 on pins 123 thru 158. This external bus is used on > the APQ8060 Dragonboard to connect an external SMSC9211 ethernet > adapter. > > Cc: Stephen Boyd > Cc: Bjorn Andersson > Signed-off-by: Linus Walleij > --- > drivers/pinctrl/qcom/pinctrl-msm8660.c | 82 +++++++++++++++++++--------------- > 1 file changed, 46 insertions(+), 36 deletions(-) > > diff --git a/drivers/pinctrl/qcom/pinctrl-msm8660.c b/drivers/pinctrl/qcom/pinctrl-msm8660.c [..] > @@ -875,42 +885,42 @@ static const struct msm_pingroup msm8660_groups[] = { [..] > + PINGROUP(158, ebi2, _, _, _, _, _, _), Looking at the default gpiomux-8x60.c in an old tree (the only one I've found that defines ebi2) here seems to indicate that pin 158 is not part of ebi2. It also denotes pin 40 as "cs2" and skips 131 for some reason. So I suggest dropping 158 and you have my Acked-by: Bjorn Andersson Regards, Bjorn