From nobody Wed Oct 8 05:33:51 2025 Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 023EE24EF7F for ; Wed, 2 Jul 2025 09:14:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751447666; cv=none; b=JYLCCfdBZg9A7gi8n4s8eogQLICWoSI/RMmicTRjX//d1aYHP1G2N0BBQ/CRFOjLp9F3LuwARxgbiExEl2G9gv6PktsQ9NDzVQ+bXr+criGY6u+ExMJZCm40kpmZKfS3gxu9L8AvesgX8WN9DG1hUl3zVMV4Me/IiP4u1ohSSkI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751447666; c=relaxed/simple; bh=DguSM7EjLtrwVpT0x9yjX3iDCS5eIB2Mafki1V+bITE=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Fdyztwc63vGyz1KJUiN9d+sSV2DXrs2dQ9NgEqTOs0BfGC4pUhHl+Ku1YwcsCOC9LDikXKLk4qduIe/Mn1T7BURjaP/3MrGFYed8IQBJbMCQ48y9EhKUXVRorg4BtXhVefgiveLiptYzS8P9F4/CCR80IeH2gJRqTrq/beprT18= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=deGiJ0Wn; arc=none smtp.client-ip=209.85.221.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="deGiJ0Wn" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-3a54700a46eso3509783f8f.1 for ; Wed, 02 Jul 2025 02:14:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1751447662; x=1752052462; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=gpx2zRD2zETOgoqOp3qUcnPAf5yA0X9HjvfLuJL1NY0=; b=deGiJ0WnYhydWqGGsIG5V/SWs4qWXLsGJb6XlikUry/llBeFnpLRnylo26yJ1a5M5A /FjVI87Z2YMqiDO4gSMSuaM6+PyPP37rHibMtBkEF8jWkzd+4DsXmb39qNzcVB1kVUCA r1cLroc0hhbv3JVEXnr8kZAo/FntXqNW6GnaOoobIZ/GiXEvW7NST/tx+mZoV69P7gmq D7j3dPHVXQlBIaZVggB9L3GyW+zXzXY+XqDAeYJhpNq9JKns33DJ7GPJ6TXPw4+Rgxk4 eLZkPYaweqbkeE7J1fU7iftNOVs4gkHcjs4zCaZO/5EnVLazr+R2BTnduC3P3DiOjtuj hODQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751447662; x=1752052462; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=gpx2zRD2zETOgoqOp3qUcnPAf5yA0X9HjvfLuJL1NY0=; b=RbcHliSuU8oJGmusXe+bCh4LdD8u+yO2e6xeAIRaR4/hevOZh3k84LZFd9/YGM2SDN tYsW/gy6+fDMGSDltawFK+AQoacJk+EmNPI2IhROXMA+MXRuSsgwOWPcRoVkQyVlp4tb Q+hEZ0wVun3t1HUC02dGWsACo/1qcyekUzmH5ow4YKCP4ijn86Clzkhe98UjKsUxylff qzB8EAAiIsn4ZFIUFYPICSRi5Q6Ti48GHlhxlyaH9gZLnHqCoiqjGE+zTdnrIyTpBBDp 4BvJUk2+ERPyRZULZyvCvvvgP04QF0W0vzDAyyU8HqV85u1g9Rt14jpztHydpLADN0vN GTWg== X-Forwarded-Encrypted: i=1; AJvYcCUndC2hL7lhz6kddSRZnUVS1HZtz64Yg6Sg3nrqWABuXy9g1pRcwXoC6a/oIgQH6OwrojWivlS1gityseQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yyw66i2PG7NrsXlRT9ezTzKngLBGDbXXhIgVLLmlSBol98EL0Ao /mlbea3rLnSTLITesvP8+xzcyIRWrD4VO8qvjjVI19WmowgJfYb9JVDQzTBPXOqHKVQ= X-Gm-Gg: ASbGnct0KeeAZKfWFv0rVyfEpG7Kf6WptfygGGk0V9FoUg3vWoH14mPsv5wdy5W1KLg ta2DXmFUNYol6RkaPl5IZO85WqxtdAxO6evY+l/Y1ql3GRNzSEHvPHW3h1C6xiEoEEGdWTg0O8c 5RVsBxX9uGW//0cSipl/a30drKmoEMcUy8sbKqDQgpH1y3ln5MGj9tDduoB/mwLzXJQ0fz056I6 drVRnfDVnmha0qMRTNgDKF0lV0NXtqqwEQWwdfmk62BpAivGZH9xbsKePkTVc8zAc7YyDR2CvmZ a6RVmGvz1PQ5SkRJHd9NAFuHEBAC3OTtEFzcPszVnbhzZGhEEYkq7C8vdvAEbks9 X-Google-Smtp-Source: AGHT+IGVlrwZBWWDYlC/97CvbiHdRC8yezOYirmKDD5NIJ7Sc5+XMfy9yrxtEVYeo1wrlkHfZ9kXzQ== X-Received: by 2002:a5d:5c12:0:b0:3a5:52cc:346e with SMTP id ffacd0b85a97d-3b1fd74c02cmr1405354f8f.6.1751447662245; Wed, 02 Jul 2025 02:14:22 -0700 (PDT) Received: from [127.0.1.1] ([2a01:cb1d:dc:7e00:8827:d0e:25e:834a]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3a88c7e6f8bsm16027213f8f.17.2025.07.02.02.14.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Jul 2025 02:14:21 -0700 (PDT) From: Bartosz Golaszewski Date: Wed, 02 Jul 2025 11:14:11 +0200 Subject: [PATCH 10/12] gpio: tps65218: use new GPIO line value setter callbacks Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250702-gpiochip-set-rv-gpio-round3-v1-10-0d23be74f71d@linaro.org> References: <20250702-gpiochip-set-rv-gpio-round3-v1-0-0d23be74f71d@linaro.org> In-Reply-To: <20250702-gpiochip-set-rv-gpio-round3-v1-0-0d23be74f71d@linaro.org> To: Linus Walleij , Bartosz Golaszewski , Thierry Reding , Jonathan Hunter , Robert Richter , Aaro Koskinen , Andreas Kemnade , Kevin Hilman , Roger Quadros , Tony Lindgren Cc: linux-gpio@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, Bartosz Golaszewski X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=2346; i=bartosz.golaszewski@linaro.org; h=from:subject:message-id; bh=Kj1Uc078M3RGsYFWnX/pxEd96G+25cnoGzALzF84Zao=; b=owEBbQKS/ZANAwAKARGnLqAUcddyAcsmYgBoZPhhiaZaWrELXTwJFn30XUF5omE+BRuB2Etak C24B5i59vGJAjMEAAEKAB0WIQQWnetsC8PEYBPSx58Rpy6gFHHXcgUCaGT4YQAKCRARpy6gFHHX cpa0EADcu4X3M7EjtlErW4yrfl3kbML5suesL9nYnC7QAOJ2pjROvgvkpbErlcxBLY1LLejY0PI 6RCNZ4qhb/Xnxyfa/NIdW1eNSapxwp7bXQ77oeVzl2SXfaA81rsPG7W6utSW/pd2p3iw8d6vSa5 +JdGW3NbP96Rc7Uh8jyhKvXwHx3Om/C6SV5i1JU0i4Zov9uV7i/KvtTBG0CRr8mDe5iZekQoS23 YxdZ9l1K5NfUXokCr1W7M48LiAi29DYuZTydb4rupaSDpX4z+clv8rNpZOFJ8CbZ+Wx/U5s8nKo JLPogSlkKEs2OezM8zQDypAejrgdZQnmVEMuF0ASzlghXQ7BIVv2mLKSkpiN8lxx2S78eNWPdtF AFz4WtbJO0F53M/sUkCX6MedGyNZuZXwTCPr3oJ3ufpfaBmWO7ijwibvW5oLo/hJplJq/SVLYsy qzBQt7ju4HHRvQNeoMe8I2754EYNiWcyZuDbjxBdDtEq+1p68k9rbTnDf8uWQg5E6dRY4iMMUKk J5ycZ2EUp+QJVndO/iCk1MWW8FtKRQBStObnZ2z3+MYZCVDgBZccjlEH06/QcyxJq0JpcqvTPKk vsd7wirRNBst7l6HEsJVXqW/OVYYtKTQstoQw7AoExOgJAAJ+ef5M7MM3ERljiU0m7CwbhY3cJ9 clwtEVGfiY9g8Rw== X-Developer-Key: i=bartosz.golaszewski@linaro.org; a=openpgp; fpr=169DEB6C0BC3C46013D2C79F11A72EA01471D772 From: Bartosz Golaszewski struct gpio_chip now has callbacks for setting line values that return an integer, allowing to indicate failures. Convert the driver to using them. Signed-off-by: Bartosz Golaszewski --- drivers/gpio/gpio-tps65218.c | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/gpio/gpio-tps65218.c b/drivers/gpio/gpio-tps65218.c index 68e4f0a19f4eb945b33e5992fdf30a29a40bfc1e..49cd7754ed053bcdbf25a4e51e2= 27287257a5f03 100644 --- a/drivers/gpio/gpio-tps65218.c +++ b/drivers/gpio/gpio-tps65218.c @@ -34,29 +34,28 @@ static int tps65218_gpio_get(struct gpio_chip *gc, unsi= gned offset) return !!(val & (TPS65218_ENABLE2_GPIO1 << offset)); } =20 -static void tps65218_gpio_set(struct gpio_chip *gc, unsigned offset, - int value) +static int tps65218_gpio_set(struct gpio_chip *gc, unsigned int offset, + int value) { struct tps65218_gpio *tps65218_gpio =3D gpiochip_get_data(gc); struct tps65218 *tps65218 =3D tps65218_gpio->tps65218; =20 if (value) - tps65218_set_bits(tps65218, TPS65218_REG_ENABLE2, - TPS65218_ENABLE2_GPIO1 << offset, - TPS65218_ENABLE2_GPIO1 << offset, - TPS65218_PROTECT_L1); - else - tps65218_clear_bits(tps65218, TPS65218_REG_ENABLE2, - TPS65218_ENABLE2_GPIO1 << offset, - TPS65218_PROTECT_L1); + return tps65218_set_bits(tps65218, TPS65218_REG_ENABLE2, + TPS65218_ENABLE2_GPIO1 << offset, + TPS65218_ENABLE2_GPIO1 << offset, + TPS65218_PROTECT_L1); + + return tps65218_clear_bits(tps65218, TPS65218_REG_ENABLE2, + TPS65218_ENABLE2_GPIO1 << offset, + TPS65218_PROTECT_L1); } =20 static int tps65218_gpio_output(struct gpio_chip *gc, unsigned offset, int value) { /* Only drives GPOs */ - tps65218_gpio_set(gc, offset, value); - return 0; + return tps65218_gpio_set(gc, offset, value); } =20 static int tps65218_gpio_request(struct gpio_chip *gc, unsigned offset) @@ -170,7 +169,7 @@ static const struct gpio_chip template_chip =3D { .request =3D tps65218_gpio_request, .direction_output =3D tps65218_gpio_output, .get =3D tps65218_gpio_get, - .set =3D tps65218_gpio_set, + .set_rv =3D tps65218_gpio_set, .set_config =3D tps65218_gpio_set_config, .can_sleep =3D true, .ngpio =3D 3, --=20 2.48.1