From patchwork Tue May 30 15:19:44 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= X-Patchwork-Id: 688004 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 544FEC77B7A for ; Tue, 30 May 2023 15:20:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230353AbjE3PUF (ORCPT ); Tue, 30 May 2023 11:20:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51300 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229697AbjE3PUD (ORCPT ); Tue, 30 May 2023 11:20:03 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C312EC7 for ; Tue, 30 May 2023 08:20:02 -0700 (PDT) Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1q418d-0007zm-KT; Tue, 30 May 2023 17:19:55 +0200 Received: from [2a0a:edc0:0:900:1d::77] (helo=ptz.office.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.94.2) (envelope-from ) id 1q418c-003uGX-It; Tue, 30 May 2023 17:19:54 +0200 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1q418b-009XZl-Lk; Tue, 30 May 2023 17:19:53 +0200 From: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= To: Linus Walleij , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, Andy Shevchenko , Kent Gibson Subject: [PATCH v2 0/2] gpio: introduce hog properties with less ambiguity Date: Tue, 30 May 2023 17:19:44 +0200 Message-Id: <20230530151946.2317748-1-u.kleine-koenig@pengutronix.de> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=1658; i=u.kleine-koenig@pengutronix.de; h=from:subject; bh=wiI74e46ToWHZc/6ArbrTrYVH4iGdfuKtXso1nG5FLs=; b=owEBbQGS/pANAwAKAY+A+1h9Ev5OAcsmYgBkdhQA++q78TcFbOUXN57P/8g0FuP9WF5sfuwzZ OEauF+GOLaJATMEAAEKAB0WIQQ/gaxpOnoeWYmt/tOPgPtYfRL+TgUCZHYUAAAKCRCPgPtYfRL+ TvX+B/913rRAOd36p3+tdaHmShzFshqd2HddX4VTqsprTW8wSjA4926VYIZfevmypYshVDt0kFx j0ZHuCKVjix64ioUR/xqigdOddTzMbt47eBlap1POLJ+8FgmTNyiFnbtzaTkziVnYms2V0WUm59 xYNbkmynuHH8OtW5UvtLijmsAu48ahEv4PfM15hHWQrxJeQHc/F/oiMmbGv7onTvqp3JCoYpTSb Psf9Yp8NNPiIELRt+skmO2tU9PGZhnYWZGwd1c0HYDWP6Fq7wXRYUm1WP49plEQKWWik9phmQgL PEvFu5tsGVQ+j5qzfl1HXw7v2RpxZTcX4Xb6NfjFjC6xL16t X-Developer-Key: i=u.kleine-koenig@pengutronix.de; a=openpgp; fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6 X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-gpio@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org Hello, this is another approach after https://lore.kernel.org/linux-gpio/20210503210526.43455-1-u.kleine-koenig@pengutronix.de two years ago. I switched back to "active" and "inactive" from "asserted" and "deasserted". The poll about the naming is ambigous, but I think with a slight preference of active/inactive over asserted/deasserted (with my unbiased self preferring active/inactive, too :-) Rob Herring acked patch #1 and Linus Walleij acked patch #2 of the asserted/deasserted variant back then, I dropped these as the change is too relevant IMHO. In the previous post I had some inconsistencies that should be gone now. And Andy wrote: > I like the idea (with active-inactive terms), but... > > It’s a half baked solution. Please use fwnode to cover all possible > providers. which I didn't address because I want to change what is currently supported and not introduce new support. Adding fwnode is orthogonal to this renaming. I didn't find any other more general implementation that would affect more than "of" for output-high and output-low. Please tell me if I missed something. Best regards Uwe Uwe Kleine-König (2): dt-bindings: gpio: introduce hog properties with less ambiguity gpio: use "active" and "inactive" instead of "high" and "low" for output hogs Documentation/devicetree/bindings/gpio/gpio.txt | 16 +++++++++++----- drivers/gpio/gpiolib-of.c | 10 ++++++++-- include/linux/gpio/consumer.h | 14 ++++++++++---- 3 files changed, 29 insertions(+), 11 deletions(-) base-commit: 8c33787278ca8db73ad7d23f932c8c39b9f6e543