From patchwork Tue Mar 17 07:15:21 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wolfram Sang X-Patchwork-Id: 6027531 X-Patchwork-Delegate: geert@linux-m68k.org Return-Path: X-Original-To: patchwork-linux-sh@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 53289BF90F for ; Tue, 17 Mar 2015 07:15:18 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 5DB6820454 for ; Tue, 17 Mar 2015 07:15:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0827020459 for ; Tue, 17 Mar 2015 07:15:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751906AbbCQHPP (ORCPT ); Tue, 17 Mar 2015 03:15:15 -0400 Received: from sauhun.de ([89.238.76.85]:41124 "EHLO pokefinder.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752044AbbCQHPP (ORCPT ); Tue, 17 Mar 2015 03:15:15 -0400 Received: from p4fe2508e.dip0.t-ipconnect.de ([79.226.80.142]:37297 helo=localhost) by pokefinder.org with esmtpsa (TLS1.2:RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from ) id 1YXlib-0007AT-Hl; Tue, 17 Mar 2015 08:15:13 +0100 From: Wolfram Sang To: linux-sh@vger.kernel.org Cc: Magnus Damm , Simon Horman , Laurent Pinchart , Geert Uytterhoeven , Wolfram Sang Subject: [RFC V2 1/4] pinctrl: allow to unselect a state Date: Tue, 17 Mar 2015 08:15:21 +0100 Message-Id: <1426576524-22315-2-git-send-email-wsa@the-dreams.de> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1426576524-22315-1-git-send-email-wsa@the-dreams.de> References: <1426576524-22315-1-git-send-email-wsa@the-dreams.de> Sender: linux-sh-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-sh@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Wolfram Sang Needed to implement an i2c n-to-1 bus demultiplexer where n different I2C IP cores should be able to serve the same bus (like i2c-rcar, i2c-sh_mobile, and i2c-gpio on Renesas RCar Gen2 SoCs). Alternative: We could also skip this patch and define two states instead of one to be used in the demuxer, like "active" and "passive". The drawback there is, that the passive state needs some dummy pins which are not actually used. They would only exist to free the active pins, so those could be used by another I2C core. Signed-off-by: Wolfram Sang --- Changes since RFC V1: none, only rebased drivers/pinctrl/core.c | 34 +++++++++++++++++++++++----------- include/linux/pinctrl/consumer.h | 5 +++++ 2 files changed, 28 insertions(+), 11 deletions(-) diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c index 89dca77ca0382e..48d77d04b5d5ed 100644 --- a/drivers/pinctrl/core.c +++ b/drivers/pinctrl/core.c @@ -977,18 +977,12 @@ struct pinctrl_state *pinctrl_lookup_state(struct pinctrl *p, EXPORT_SYMBOL_GPL(pinctrl_lookup_state); /** - * pinctrl_select_state() - select/activate/program a pinctrl state to HW - * @p: the pinctrl handle for the device that requests configuration - * @state: the state handle to select/activate/program + * pinctrl_deselect_state() - deselect a pinctrl state to HW + * @p: the pinctrl handle for the device that requests deconfiguration */ -int pinctrl_select_state(struct pinctrl *p, struct pinctrl_state *state) +void pinctrl_deselect_state(struct pinctrl *p) { - struct pinctrl_setting *setting, *setting2; - struct pinctrl_state *old_state = p->state; - int ret; - - if (p->state == state) - return 0; + struct pinctrl_setting *setting; if (p->state) { /* @@ -1002,9 +996,27 @@ int pinctrl_select_state(struct pinctrl *p, struct pinctrl_state *state) continue; pinmux_disable_setting(setting); } + + p->state = NULL; } +} +EXPORT_SYMBOL_GPL(pinctrl_deselect_state); + +/** + * pinctrl_select_state() - select/activate/program a pinctrl state to HW + * @p: the pinctrl handle for the device that requests configuration + * @state: the state handle to select/activate/program + */ +int pinctrl_select_state(struct pinctrl *p, struct pinctrl_state *state) +{ + struct pinctrl_setting *setting, *setting2; + struct pinctrl_state *old_state = p->state; + int ret; + + if (p->state == state) + return 0; - p->state = NULL; + pinctrl_deselect_state(p); /* Apply all the settings for the new state */ list_for_each_entry(setting, &state->settings, node) { diff --git a/include/linux/pinctrl/consumer.h b/include/linux/pinctrl/consumer.h index 72c0415d6c2175..5e949122529db8 100644 --- a/include/linux/pinctrl/consumer.h +++ b/include/linux/pinctrl/consumer.h @@ -36,6 +36,7 @@ extern struct pinctrl_state * __must_check pinctrl_lookup_state( struct pinctrl *p, const char *name); extern int pinctrl_select_state(struct pinctrl *p, struct pinctrl_state *s); +extern void pinctrl_deselect_state(struct pinctrl *p); extern struct pinctrl * __must_check devm_pinctrl_get(struct device *dev); extern void devm_pinctrl_put(struct pinctrl *p); @@ -102,6 +103,10 @@ static inline int pinctrl_select_state(struct pinctrl *p, return 0; } +static inline void pinctrl_deselect_state(struct pinctrl *p) +{ +} + static inline struct pinctrl * __must_check devm_pinctrl_get(struct device *dev) { return ERR_PTR(-ENOSYS);