From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) (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 CAEA13446B8 for ; Wed, 19 Nov 2025 09:38:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763545095; cv=none; b=GhJNjoYklqKp9hE/cqzcbcoQv5hl+1aTZxGLx0/WLr2avFPkLy662xPyK2ANGhObCJwea9K100mj1wHga/6NjYaSXMibMnnBaZGXCeUktHkiKmt+1pqZxDGvBa9jrm7AP7pWVBvlaAZdx/R5U8+Ch7K17QcHNum5OYYxkyezZjs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763545095; c=relaxed/simple; bh=BA/QhjEOAvnHt2a7jJ7yUqi14IVlgeBAOD2GlMeu5zQ=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=DERMekg22xhsbWiNQ2dW9zumxzKqhuil94hD7Dw9RaqU6sobSOk0/66qOlOHbSekFNg0rQWntGBs5BIV3/D9W/1Jq0nKNYO4GkzgCQ7w+oADae6hDFP50FhhjLkgq/w6FqV1QMoeiOK3BVT9vRhLTH/xPp15IwJLlPKG/lPet9I= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=XX1ZCmvK; arc=none smtp.client-ip=209.85.221.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="XX1ZCmvK" Received: by mail-wr1-f50.google.com with SMTP id ffacd0b85a97d-42b32900c8bso3694633f8f.0 for ; Wed, 19 Nov 2025 01:38:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763545092; x=1764149892; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=BA/QhjEOAvnHt2a7jJ7yUqi14IVlgeBAOD2GlMeu5zQ=; b=XX1ZCmvKPU+4XAfKnHtjUnQ2v8HRoLpbk9zb8tU/BrgV6j4APOR8aZuOV66BDYICyy 5hKGHRpFzTMiE85Igz2sGoC6sWHaK31Bi6YFrl1Lva6raOhaQvBnKiYOSzRnGD+ahxuS M4xtD2sWcF6rsg3g5JaXyHhFY0sB1Y+isHHZXDu69EWBqSC+U2zJLkKcHL6SFRzcQNuL /l37MQDLNZD8SeCLM5Ql3S7AUxU3gbl7xsdE0eYwBgBhqcVy/E6+JbmlBlkFh0D0vP8D LGUPk307OLT+34JYm3VRY6nkvXkBPXaK4+BLvGWSfCiG3u4y7ZfEFXUNWqvTVgxvG/R2 Qzbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763545092; x=1764149892; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=BA/QhjEOAvnHt2a7jJ7yUqi14IVlgeBAOD2GlMeu5zQ=; b=q/oMN69fER/yy8IEEiwinpV2bdi/E44LauaW2tPqhuD95rYnLK7imP6LSq4daq3X2u 4c8WADDOhaKdx7ksTadeJfDoheUl95TG0efO2fThGg9TKnpZL/NeuNZ1Gfkayq7t2ktn Myo9F4NFwCxGyf0kF8MC5u0hRgaXjY+snS9M6PmXZD8j0rytzelcbAFzXWZ/6ASJ2u+l qanhsNXUXEniH3O4HbiVfAros6lj+S+HdTjoyS0E4cCft1jgSBoFvIAp+0z3ZVNrAffP a581oXiAfr69TdSvGhtiTpxGb81lXUmxVhtoA3PIV/QyDgeP6xnY/Ogpcl4hwAhmaZMk dGmw== X-Forwarded-Encrypted: i=1; AJvYcCVO21kDE2WBoOgnSDqA/ooLvM2wAKAgsI5h/7ikUwPtnLiyg53UeFRi4hjESRXYl6BlTkoCKy4aOPyr7e8=@vger.kernel.org X-Gm-Message-State: AOJu0YwISTq1KzKzC2Mn6yVp+N01D/eUkes8qimpXwQuBOCa05nbaLiZ kgf9Mz8sbz+zRG5PQeirfkv7ULK2mvN8dZdsJ0Xzqd+sd682Apjd6IV9 X-Gm-Gg: ASbGncvUJPv4+niqF9Bqr8559pujsfO+3j5b6LMKodw21RNEPPaarpelQvF2EAoHtyl B7TSlH4lnf4OS91WPHOn3AfmTszEEd7rO87zNhBj8mN9PqGvNJ960uDv0YG+UwQD8+CXqWw6Gtt TAftE3N8VdKAIdt+MKOGzkdZoy6nhFm+dzoKl3QnIoEqQHm8jUX5L04OuFlNlAiZkEvdcCTrj1z RpcqIx3sjvIZOsxeo2FEnQHtKe2mLocPMacQt/z8d4iEHZEtBt08NeU2nClPAOm7mjbBcd0WO2k bxWpduufUKDXXT1W9UODnP5R2EDs8TekAEfLrvs+llY5WW4f1sERJOE7fPKL56lOnTb7IvvQ1G0 RBjF/BA++wjYBgjeXgKmW6dx9JNQgx1/yR9B7/w943VKUV6ZFrXjlG6HkQYMTmJaFn3DJJH9d60 ALHrXlthArIH5sxbOKmfHPjF+ZaIw/OQ== X-Google-Smtp-Source: AGHT+IEWzNEjxbuCnmwYlkD23l9ZyNOLZ82woEGJ46u5TpbCp1tYUrkPolXaHpgj5iFjnral8XQ5kw== X-Received: by 2002:a05:6000:3101:b0:42b:36f4:cd20 with SMTP id ffacd0b85a97d-42cb1f9e9bamr1758191f8f.27.1763545091879; Wed, 19 Nov 2025 01:38:11 -0800 (PST) Received: from [192.168.1.187] ([161.230.67.253]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b53e84b12sm38667604f8f.15.2025.11.19.01.38.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 01:38:11 -0800 (PST) Message-ID: <67d177ef5fa41e597459f88159fd289354f7dde3.camel@gmail.com> Subject: Re: [PATCH 2/2] gpio: adg1712: add driver support From: Nuno =?ISO-8859-1?Q?S=E1?= To: Linus Walleij Cc: Antoniu Miclaus , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Date: Wed, 19 Nov 2025 09:38:49 +0000 In-Reply-To: References: <20251031160710.13343-1-antoniu.miclaus@analog.com> <20251031160710.13343-3-antoniu.miclaus@analog.com> <3ead5d7aa5e6be2b6df3bb91b35fec37e23353f3.camel@gmail.com> <0cd7cf10d47683f22a4358635f243a2dfb6be564.camel@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Tue, 2025-11-18 at 23:54 +0100, Linus Walleij wrote: > On Tue, Nov 11, 2025 at 5:01=E2=80=AFPM Nuno S=C3=A1 wrote: >=20 > [Me] > > > - Implement .set_config() and implement the generic pin > > > =C2=A0 control property PIN_CONFIG_OUTPUT_ENABLE as 1 > > > =C2=A0 to switch "on" and 0 for switch "off". > > > =C2=A0 See include/linux/pinctrl/pinconf-generic.h >=20 > > The above makes sense to me. I'll let Antoniu take it from here and che= ck if > > the above fits the usecases he is aware of. Not sure if it makes sense = for a piece > > of HW like this but if the usecase is for userspace to control the on/o= ff states, > > then I guess we would need .get() and .set(). Or some kind of "frontend= " driver > > making use of the consumer helpers. >=20 > There is already GPIO_V2_LINE_SET_CONFIG_IOCTL > in so setting configs from userspace is no issue, > just use the character device. >=20 > You will need to add I think two new config flags for userspace: > GPIO_V2_LINE_FLAG_OUTPUT_ENABLE > GPIO_V2_LINE_FLAG_OUTPUT_DISABLE >=20 > And update gpio_v2_line_config_flags_to_desc_flags() in > drivers/gpio/gpiolib-cdev.c accordingly. >=20 > Then you probably want some tests or examples in libgpiod to make > sure userspace is fine. Bartosz knows all about how to do this. >=20 It seems there's no need for userspace control. If you look at v3, it seems we don't really need it to be a gpiochip (at least, I think). Maybe take a = look, you might have some good pointers :) Thx! - Nuno S=C3=A1