From nobody Tue Feb 10 16:44:02 2026 Received: from mail-wm1-f48.google.com (mail-wm1-f48.google.com [209.85.128.48]) (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 B5C87263F22 for ; Tue, 8 Apr 2025 07:18:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744096689; cv=none; b=b4maSM7jhyiWIlutEZ+Ygor3IWBUdJdLfFc6JSYPbcBTIPbR93CXYF1ERkZI3gg65q8rU8GJu2ZylifR9HBDrNU26REPrOc5I5A9u2n1OttyjhbXMOFaRteoteIWVsbdcA0BtkcCJOyYaG7WC5c+0PBXcqXx0BAC21jdeiS4wMU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744096689; c=relaxed/simple; bh=S1eRm9ruT/z8xa6PV2GYxT3sAM1tH/Z37d9OcC4qqFo=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=rBdLbkoIU1/pxRz6rMThLdm1Y7TpQP8FMinb/X7Y7QySnLU4+5NovSqwexHAzo6+KvcQI6/UZJk/F1ifGCLHmzJE6Bsus/A6ZcwUj3X41vw+i3VHXj67S5atTrO2Pzv6AoTdlU/mNIGyU9oIXfPZc6CGZNwYJGvm7omqBhs1oJQ= 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=lmUj2Ewh; arc=none smtp.client-ip=209.85.128.48 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="lmUj2Ewh" Received: by mail-wm1-f48.google.com with SMTP id 5b1f17b1804b1-43ce70f9afbso56441705e9.0 for ; Tue, 08 Apr 2025 00:18:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1744096686; x=1744701486; 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=lsLjcRrGKuzDX9mvNJ2l1Pj3maW/MPJjpEEItwHgjBM=; b=lmUj2Ewhzr4TtxbBgCyouo8GVt1gku318lTNsVJjCswA8TJKI4s/kXDj/GKUiAj7IQ MIecDrh0N0XsujNBxfhy9FV3pfe+rgJccIKUXaZB7E4Xd4akWBaXmXY6IwFswEGCgSxh I88fGg10AvFZr2zphUP7meE3F5eFx+mEd4gVzFuXQC7DuLOtUBMxfdP7xj77lh2lf36o fLFCFk0i5pI50js3SEzAqEtgXUTP8aA1TzkIAkvbZRwVu8S0m3DsOHT/lVAUg5HaN8RF KgLPHpA94P2YiyTIWO5qx/NFWkH7i+9eEsoe0sKn6RX1WSTiEOEhHm7rc8tZZPzzm8tK RO9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1744096686; x=1744701486; 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=lsLjcRrGKuzDX9mvNJ2l1Pj3maW/MPJjpEEItwHgjBM=; b=hOAIKS80ZQCMd9zZCiOU8egZM/H70PqJxtYEy+szVmFe8G6FLa+p1OSLFE2rg9Cn4J rIuTt61AlRZKmL+BxKgu6DIKyZbpULuYME3mO/fYiqKk2Bo47rM9JDhRCmbDFp8DXkmS IJwHK6R0ba9GUHlWr51sDf9ZG9DbcD6GMeTZ68BJkMaq4ZAJKoM7fdeO1ZLS+l0hKyME C4L5Nxb+M1hU9jUFSvYGRhPK4RrKufkZrDf12BzEdv3MxPuTdY7sPjsojVBpcxcMAKg+ AFflxOjwVI74b17mUH3ekMQZ0KmoZDhy2AarNZI01x+Hsk8bIeX2c3GzTUVB+oy3UEiF ajeA== X-Forwarded-Encrypted: i=1; AJvYcCX8EicwZlT/UY4qQ+zg12Hj40Z2H72wFDRl3YnQz7f0XOMkl/MaiRSQZBX2Uw+H8ZTvGKnVBldPeYUHLJ8=@vger.kernel.org X-Gm-Message-State: AOJu0Yw9X4HoFqSlKj8ITe6V1CNoe+8mhVB7cstXUNDwAFwRYExfzf1V /soiqnCy3HFB0xRnH0qFWEuAk5IUov8ugmFRxfCldDQIK2VJcjT3OPMCEtp/4KY= X-Gm-Gg: ASbGncufpMyBHudwESahAE/u/C4mwl/TX0E0BuEq9uhFmmFpURqKrO3O0RMZCy/IDDC f+cAwdY+MUxXNSabHMnVwlh4BWjOXevBZKuf0uI3TrKcxMvrIif4oLRwHU72t1ixV1zBsyVmWBk ow8mCfeRTHK6BNlGEqjvtkFoSRBpuF9tjHHZVihX+fx/JnJBf1x6TO+QuHiV8BVKkGyPjaWms5/ OIaccYNK4dOrrxl8WccT44z950eH35jwuYlYsnSFg588qKEfUOmr6TNS9tbbuBkg8luxGFmRTB0 SX2CDBR6skX8Siqkl+ChP4GnWvlU80CYOato9g== X-Google-Smtp-Source: AGHT+IFVLY1WNvAayiNBB006bFO4nSra7Yx4DcKw1yaLoj6yk4wJfEbc8yNlS2Lbk6qsNo9ohgkRog== X-Received: by 2002:a05:6000:40cc:b0:39c:142a:ed6c with SMTP id ffacd0b85a97d-39d0de66a15mr12602862f8f.40.1744096686025; Tue, 08 Apr 2025 00:18:06 -0700 (PDT) Received: from [127.0.1.1] ([2a01:cb1d:dc:7e00:2adf:eaae:f6ea:1a73]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-39c3020d5d1sm14257694f8f.77.2025.04.08.00.18.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Apr 2025 00:18:05 -0700 (PDT) From: Bartosz Golaszewski Date: Tue, 08 Apr 2025 09:17:47 +0200 Subject: [PATCH 10/10] pinctrl: cy8c95x0: 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: <20250408-gpiochip-set-rv-pinctrl-part1-v1-10-c9d521d7c8c7@linaro.org> References: <20250408-gpiochip-set-rv-pinctrl-part1-v1-0-c9d521d7c8c7@linaro.org> In-Reply-To: <20250408-gpiochip-set-rv-pinctrl-part1-v1-0-c9d521d7c8c7@linaro.org> To: Ludovic Desroches , Linus Walleij , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Bartosz Golaszewski , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Xianwei Zhao , Patrick Rudolph Cc: linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org, Bartosz Golaszewski X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=2609; i=bartosz.golaszewski@linaro.org; h=from:subject:message-id; bh=nFz0ZQu1wB25C83nAY0bNQ02w+UniTfC+j0g7wZb9Vw=; b=owEBbQKS/ZANAwAKARGnLqAUcddyAcsmYgBn9M2b7PYpEd1QYwzkZOLJp94uziWDLFCT9+cRZ 6jtV1w9v+CJAjMEAAEKAB0WIQQWnetsC8PEYBPSx58Rpy6gFHHXcgUCZ/TNmwAKCRARpy6gFHHX cphiEAC83D/t8tFMiLHRxWuKzVZS6TAwB9yfLq3bsdR9gOgBqJaavIsx28w0liHuZZIj8w3UHJ3 te5PhEz/S5S7Osm2h1k+wsZovOT4+g7Tn/BqJ04NbpBDccfDwNlRHz9dp5oTqhX/I9BxlNdAM6j nw2qiHcJBTD+wKKdTworBCco10+UdxKyIVJZ1xIEUUha9S7j83VF0X/lo/D6JFLOPf7hVAxpMUg UAs4h9Zmbh+zfoE3OX3QPGFNlwtL/UHFHuKx8uHYrmKv0tOmFEJL2FpJ1Dpt21Ue0har4mN7d+z XYTh2aZ5lZBpTRONkVjKs17D58wvSoBeJSXBp60u0KOD5isuDZxUWicE89i2IY9+jkffOPFNBBj hwVA2QcAYCAUMMtWb4BzW4sg02wn+SwW/2TRCl5aUXcF7n+rRnYBTlfmliRX2kHJ3PA3532UUS4 0qF0kn7hDbX9yY7oX/aD3N97/Eyh9HrOfWH13ZdvF5Hh+MT56nrovr0kjLQFazEgd8IYV8CEPDg ZoCCrIrePRmAOUTYwyZAb/qrH/mEh2qLVE5EAVKijNbE+LV0QR4DWDglwYmR6LbjrmcfnyoOS3U 1/73A3TxYyCRipzULEk7VLJXFMw5u8ROVTWk+4s4XCehemi1NJRDj9ORfPUJMTW0b8refz8JnQW hv7fa/dGOTU4VTg== 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/pinctrl/pinctrl-cy8c95x0.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/pinctrl/pinctrl-cy8c95x0.c b/drivers/pinctrl/pinctrl-c= y8c95x0.c index 3cfbcaee9e65..8a2fd632bdd4 100644 --- a/drivers/pinctrl/pinctrl-cy8c95x0.c +++ b/drivers/pinctrl/pinctrl-cy8c95x0.c @@ -742,14 +742,15 @@ static int cy8c95x0_gpio_get_value(struct gpio_chip *= gc, unsigned int off) return reg_val ? 1 : 0; } =20 -static void cy8c95x0_gpio_set_value(struct gpio_chip *gc, unsigned int off, - int val) +static int cy8c95x0_gpio_set_value(struct gpio_chip *gc, unsigned int off, + int val) { struct cy8c95x0_pinctrl *chip =3D gpiochip_get_data(gc); u8 port =3D cypress_get_port(chip, off); u8 bit =3D cypress_get_pin_mask(chip, off); =20 - cy8c95x0_regmap_write_bits(chip, CY8C95X0_OUTPUT, port, bit, val ? bit : = 0); + return cy8c95x0_regmap_write_bits(chip, CY8C95X0_OUTPUT, port, bit, + val ? bit : 0); } =20 static int cy8c95x0_gpio_get_direction(struct gpio_chip *gc, unsigned int = off) @@ -908,12 +909,12 @@ static int cy8c95x0_gpio_get_multiple(struct gpio_chi= p *gc, return cy8c95x0_read_regs_mask(chip, CY8C95X0_INPUT, bits, mask); } =20 -static void cy8c95x0_gpio_set_multiple(struct gpio_chip *gc, - unsigned long *mask, unsigned long *bits) +static int cy8c95x0_gpio_set_multiple(struct gpio_chip *gc, + unsigned long *mask, unsigned long *bits) { struct cy8c95x0_pinctrl *chip =3D gpiochip_get_data(gc); =20 - cy8c95x0_write_regs_mask(chip, CY8C95X0_OUTPUT, bits, mask); + return cy8c95x0_write_regs_mask(chip, CY8C95X0_OUTPUT, bits, mask); } =20 static int cy8c95x0_add_pin_ranges(struct gpio_chip *gc) @@ -938,10 +939,10 @@ static int cy8c95x0_setup_gpiochip(struct cy8c95x0_pi= nctrl *chip) gc->direction_input =3D cy8c95x0_gpio_direction_input; gc->direction_output =3D cy8c95x0_gpio_direction_output; gc->get =3D cy8c95x0_gpio_get_value; - gc->set =3D cy8c95x0_gpio_set_value; + gc->set_rv =3D cy8c95x0_gpio_set_value; gc->get_direction =3D cy8c95x0_gpio_get_direction; gc->get_multiple =3D cy8c95x0_gpio_get_multiple; - gc->set_multiple =3D cy8c95x0_gpio_set_multiple; + gc->set_multiple_rv =3D cy8c95x0_gpio_set_multiple; gc->set_config =3D gpiochip_generic_config; gc->can_sleep =3D true; gc->add_pin_ranges =3D cy8c95x0_add_pin_ranges; --=20 2.45.2