From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wfout1-smtp.messagingengine.com (wfout1-smtp.messagingengine.com [64.147.123.144]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 73837140367; Thu, 13 Jun 2024 08:50:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=64.147.123.144 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718268650; cv=none; b=OWoUXV4fTHR77VQ9cAi0ij594HscfmlU0uyWbSHZuWqTGhnW+lLAuZowRZoBof/K2MQJoRNRt5qlLncRdwX+d+cdJOk2/PTeCch8Xs9cFzeDV2qxRQcd+iOellwRVLlnd1/I7SRcFtm2rYmnAWNswTtqPWnHBohStORO4FqaFp0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718268650; c=relaxed/simple; bh=KXCXAgQ/8mCSDJW0lQx7h8JuO6D20M17zUa7fCUzY1Q=; h=MIME-Version:Message-Id:In-Reply-To:References:Date:From:To:Cc: Subject:Content-Type; b=K5vQ+8MWzJo9YzP4V6RVnPpVAL/bhblix6Si2v67J7ND0fdUxR5UkMJRDamrk/qS8kQuZabWSNQtQQ37AOShBfHOF5v+27yCDMQpEN9iq97rAzNitG3f/WRbQVWi0ig1KnvVYNqjIpSxQSeMFySWMcuHbtnMUweZRJTj+CZpXGs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arndb.de; spf=pass smtp.mailfrom=arndb.de; dkim=pass (2048-bit key) header.d=arndb.de header.i=@arndb.de header.b=rPfp7S3H; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b=IbOIkppC; arc=none smtp.client-ip=64.147.123.144 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arndb.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arndb.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=arndb.de header.i=@arndb.de header.b="rPfp7S3H"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="IbOIkppC" Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailfout.west.internal (Postfix) with ESMTP id ECD9C1C000E9; Thu, 13 Jun 2024 04:50:46 -0400 (EDT) Received: from imap51 ([10.202.2.101]) by compute5.internal (MEProxy); Thu, 13 Jun 2024 04:50:47 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm1; t=1718268646; x=1718355046; bh=yc30Sq5q8flxrIVa1InN0nKiCPVTeJsKHwqnezpTGqQ=; b= rPfp7S3HfSzQZ3r9TA5s2ZHDPOXTrwNkWIw5662zm0dly4h+MMKk2bsy7zs33tjL bWX9QCO87ghraapH352S8zblY7ooBWRPTwfcyjdKNIFdEZOS5f16tqk66sleuFHZ QxdkIrHw7QTXU1kiV2/dGVwF6IcMsWUttgipLBC8IhuqsVeBEnUUmOXjTDQ1G8wO iIqCdpak2egq6zGMQhaF2pqoAKJsaDMY9ftD9QeZTg5KXRCk5tWxKxU3OEmTj4oH /BaPRRzeV71ozwmo7NTwKRzczUzBnQj5bEbiXSVCdAc1ZPpteq+5XUgjBMUx4XO/ qYUmjCkBaYfj0zepKGC/3Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1718268646; x= 1718355046; bh=yc30Sq5q8flxrIVa1InN0nKiCPVTeJsKHwqnezpTGqQ=; b=I bOIkppC2M7GaC485imy4rYPjPZ3gxBuxuO9jDkBZb/kbAmXd2+T/iKMyDppNWEhy UJeN0q8a27xrmttFWZ2+fc+Hfm7v60JU0uKMDaK3x1byQKnJYo4M+A8BaLqQ22Rb Z6jA2JbPIs+Cv+9Lf09qU0bdHB6bNDLXTrFEdXXPY5m+WXGxofi1jmN05Z0ih5JX 8I4nBX4MvvlVvoRxHyUBNPC0U5Ms4UswvVo5hcvZCUyQaWE75YIsDN+18Gk1/5e8 AN6qpKmcsH3GrNS+R7JRCjL1V5zHlSLpjIK+lRx+UFHw9YepPy9ALWkbocwXWWpZ RsuPvF4VTiCrxml5SEA3Q== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvledrfedujedgtdejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtgfesthhqredtreerjeenucfhrhhomhepfdet rhhnugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrg htthgvrhhnpeegfeejhedvledvffeijeeijeeivddvhfeliedvleevheejleetgedukedt gfejveenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe grrhhnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 2AFE7B6008D; Thu, 13 Jun 2024 04:50:46 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.11.0-alpha0-515-g87b2bad5a-fm-20240604.001-g87b2bad5 Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: In-Reply-To: References: <20240610112700.80819-1-wsa+renesas@sang-engineering.com> <20240610112700.80819-2-wsa+renesas@sang-engineering.com> Date: Thu, 13 Jun 2024 10:50:25 +0200 From: "Arnd Bergmann" To: "Linus Walleij" , "Bartosz Golaszewski" Cc: "Wolfram Sang" , Linux-Renesas , "Jonathan Corbet" , "Kent Gibson" , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, "open list:GPIO SUBSYSTEM" Subject: Re: [PATCH v9 1/1] gpio: add sloppy logic analyzer using polling Content-Type: text/plain;charset=utf-8 Content-Transfer-Encoding: quoted-printable On Thu, Jun 13, 2024, at 10:27, Linus Walleij wrote: > On Thu, Jun 13, 2024 at 10:17=E2=80=AFAM Bartosz Golaszewski wrote: > >> On second thought >> though: are you sure drivers/gpio/ is the right place for it? > > Actually that is something I requested. > > I think it fits in drivers/gpio as it is such a clear cut usage of GPIO > lines, and it doesn't really fit into any other subsystem. > >> May I suggest moving it over to drivers/misc/? > > Misc is a bit... > messy. I remember Arnd being very sceptical about putting stuff there > rather than creating new subsystems, so since I've tried to avoid it, > albeit recently more and more stuff gets merged there again :/ Right, and that is mostly to avoid having code in there because there is no other place for it. Some parts of drivers/misc should have been a separate subsystem, some should have use an existing subsystem, and other parts should have never been merged. The parts of drivers/misc that make the most sense to me are those that expose a one-of-a-kind piece of hardware as a single character device. This one would probably fit into drivers/misc/ better than some other drivers we have in there, but leaving it in drivers/gpio/ also seems fine. I could also imagine the functionality being exposed through drivers/iio/ in a way that is similar to an adc, but I don't know if that would work in practice or how much of a rewrite that would be. Arnd