From mboxrd@z Thu Jan 1 00:00:00 1970 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 35DD7C77B73 for ; Wed, 31 May 2023 08:16:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231993AbjEaIQC (ORCPT ); Wed, 31 May 2023 04:16:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43128 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234708AbjEaIQB (ORCPT ); Wed, 31 May 2023 04:16:01 -0400 Received: from mail-ua1-x934.google.com (mail-ua1-x934.google.com [IPv6:2607:f8b0:4864:20::934]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1DF7CBE for ; Wed, 31 May 2023 01:15:59 -0700 (PDT) Received: by mail-ua1-x934.google.com with SMTP id a1e0cc1a2514c-78701841ccbso2913827241.2 for ; Wed, 31 May 2023 01:15:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20221208.gappssmtp.com; s=20221208; t=1685520958; x=1688112958; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=j+LYBn2d5F5zSswKXzUQPCUZnQJRWf9mOfUbaKg3l0w=; b=rMKmXPVn3E3lgiHa3lycUjJARHwVgxFmwfl5sCpdWFiTMVw2lqaS9GLBN3aux6PLVS +cphWgiDlPFjdTctdja+HfoNHSkR0Mv0SJ1LTtR0zTBvejt9mmqfgZ/Om3RxKnjLWrAg NUQbTlNH9YVzdJmhkOdWcodbnjnVp4dh8g2U6BPbrTiBEsoDqToze6m814CUVSlEyK24 1JbHnwFdNmQqp2Ht1oh7CW8xqk+8QBbFVos89g6UObdhKssiiPEWuVTMEleFB1u5hDFZ z3pWTezwyiU0731bu3W1mudxVmHGoWzOA+8uOoNZSJ5A02PIEwdwvgZGHutnvN1AZoIc NyBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1685520958; x=1688112958; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=j+LYBn2d5F5zSswKXzUQPCUZnQJRWf9mOfUbaKg3l0w=; b=iMMgfzj9UULBstgdhswJCRtiOilF1jUE7mrGk06YnLhxQloHqbwlf06qa1flsOdORY JhJjHcr2EnaTarRZ4ozjiRVGLbvQeHMMyiLm2s9/WgV1SivDHqB978yQ8fC03kYMU7Yu jWHjDN3hhNlOkgfxw9/xQkdwdn9l1p/B65NwiSEyVk8xMjUSYKgfqBc7FZLqedo+D0Hh WEsy/ZIxU/twO2KmSYpEE9W6A6dyoGgvheVw/pbSMJeXyDfA+VeXnJHtUaLDW6VKnqEs 7k2C2CoIrLlvoHa5ifyAaCFGihW5dMXm4oxmFw7yDBc4dewCIuKhL2uIM0dFYzTa+o/f 5h3A== X-Gm-Message-State: AC+VfDzd0lw9wQjvdfE75Spk4ikWtaAWUW1rJ6kOWPDpFpYR4VjbLt5H QbNok9rqQ48C53WrH4SZQcYtM7uGofe8ndaSa1w2bg== X-Google-Smtp-Source: ACHHUZ48zd6+NSboV0/QVlTXbxck4+hQ02rvQ8jT8/OjQ2yO0i7Aa0NldEv5lWNZ8h/fCfRqRYZG1yI2xFljYMFKfKQ= X-Received: by 2002:a1f:ca07:0:b0:44f:d211:2df3 with SMTP id a7-20020a1fca07000000b0044fd2112df3mr1581190vkg.13.1685520958083; Wed, 31 May 2023 01:15:58 -0700 (PDT) MIME-Version: 1.0 References: <20230530151946.2317748-1-u.kleine-koenig@pengutronix.de> In-Reply-To: From: Bartosz Golaszewski Date: Wed, 31 May 2023 10:15:47 +0200 Message-ID: Subject: Re: [PATCH v2 0/2] gpio: introduce hog properties with less ambiguity To: Kent Gibson Cc: =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Linus Walleij , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, Andy Shevchenko Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Wed, May 31, 2023 at 2:55=E2=80=AFAM Kent Gibson = wrote: > > On Tue, May 30, 2023 at 09:15:55PM +0200, Bartosz Golaszewski wrote: > > On Tue, May 30, 2023 at 5:52=E2=80=AFPM Kent Gibson wrote: > > > > > > On Tue, May 30, 2023 at 05:19:44PM +0200, Uwe Kleine-K=C3=B6nig wrote= : > > > > 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/inacti= ve, > > > > too :-) > > > > > > > > > > FWIW, this makes sense to me too - the active/inactive naming is used= in > > > both the GPIO uAPI and libgpiod v2, so it would be consistent with th= at, > > > if nothing else. > > > > > > Bart, just wondering if gpio-sim should support the aliases as well? > > > I realise they don't support active-low, so polarity isn't an issue, = and > > > it could even be confusing to support the alias, but just throwing it > > > out there... > > > > > > > I'm not sure what you need aliases for? Value is only shown, never > > stored (where you'd need "active", "inactive" strings). > > > > I was thinking wrt the definition in DT - to allow the switch to > output-active etc throughout. And I suppose also for configuring the > hogs in configfs, but mainly the DT. > But gpio-sim doesn't handle DT separately - it reuses the gpiolib-of code and pretty much works like regular DT drivers. Bart