From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout01.posteo.de (mout01.posteo.de [185.67.36.65]) (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 A880F223328 for ; Thu, 19 Dec 2024 12:44:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.67.36.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734612288; cv=none; b=TKiQkLFbJerwYM4Dqk1ktVbyTBSwXgHGOciQtMsV2wWhM0QV4TeGZWD6Qf0H5PJ9tHczA357YRvXCrSF+JpJg1V89q0yar1uRS4A8qKJLINudRh2AOqkk4P6a6N5ue4NRs/rsGy6HCoSflU/gN/raaD0w6BKEe4sdSZECZrxSo4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734612288; c=relaxed/simple; bh=P89gRBrLr5JfkGMeHfX1YOoBe6Tlj69JV3TBj5G/KX4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=HJ4eLMAx7caZ9TO8keE2+1vdvmAM/fV1AtBakflvixB8jI4/AVSxbXqe38M4RKqMF7UPexXhjVhSX3NxPFpe6ZcRXOyw7FHB5yliBjDp6fRCJIrpzRD85QOxalgB2qaPfB1PmcKzu6EODqmKNNA86nkzg3xXCPuxwOqYubtI+Gw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net; spf=pass smtp.mailfrom=posteo.net; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b=ExtRzfjT; arc=none smtp.client-ip=185.67.36.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=posteo.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b="ExtRzfjT" Received: from submission (posteo.de [185.67.36.169]) by mout01.posteo.de (Postfix) with ESMTPS id 0FC06240027 for ; Thu, 19 Dec 2024 13:44:43 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1734612284; bh=P89gRBrLr5JfkGMeHfX1YOoBe6Tlj69JV3TBj5G/KX4=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:Content-Transfer-Encoding:From; b=ExtRzfjT4xaHbk0mb+K85L/U9/Tug56DNc9ihGDQiisl4hDevx++rNdy/KbZRfqE6 N71gBdQOenl4poHobL2RHPgP/ZGG4y0o9e6P2rPXhFPyVCm58IT7nPNEFVxs3S8wgX sKY3oWRXt1X2r5u01mOukHYeTlERy+bSHt5x03OhnC64B05cdjHF5ej3MQGE5m767d cQrd8nJxmx+zialhfvKfXnk54ydhd/UkQxT1PLBm4TxJxGBfUuUChYmAZhL0CVwfxw KKFAkw2+90qXk2ZtGp4DMx8amhuRDc++8QlmU/2UO7502ZxlynkUIVoswYZodmzOVT EjZ8JB03OJRZg== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4YDVcV5XV5z6v0G; Thu, 19 Dec 2024 13:44:42 +0100 (CET) Date: Thu, 19 Dec 2024 12:44:42 +0000 From: =?utf-8?Q?J=2E_Neusch=C3=A4fer?= To: Rob Herring Cc: =?utf-8?Q?J=2E_Neusch=C3=A4fer?= , Linus Walleij , Bartosz Golaszewski , Krzysztof Kozlowski , Conor Dooley , Maxime Ripard , linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/4] dt-bindings: gpio: fairchild,74hc595: Add latch-gpios property Message-ID: References: <20241213-gpio74-v1-0-fa2c089caf41@posteo.net> <20241213-gpio74-v1-2-fa2c089caf41@posteo.net> <20241217152522.GA1813602-robh@kernel.org> Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20241217152522.GA1813602-robh@kernel.org> On Tue, Dec 17, 2024 at 09:25:22AM -0600, Rob Herring wrote: > On Fri, Dec 13, 2024 at 06:32:48PM +0100, J. Neuschäfer wrote: > > The Fairchild MM74HC595 and other compatible parts have a latch clock > > input (also known as storage register clock input), which must be > > clocked once in order to apply any value that was serially shifted in. > > That sounds like all the existing parts have the signal and it is > required to operate? Or just needed to write settings, but not read GPIO > input state for example? These parts are output-only (so, "GPO"s, arguably). The situation with the latch signal is weirder, as I found out in the meantime: These parts don't have a chip-select built in, but the rising-edge triggered latch clock can be reinterpreted as an active-low chip-select, because that would also rise after the appropriate number of bits has been shifted through the SPI bus. _ _ _ _ shift clock ____| |_| |_..._| |_| |_________ latch clock * trigger ___ ________ chip select# |___________________| So, I now think that no additional signal and no binding change is actually needed, just perhaps an explanatory comment. > > If the new parts are usable without latch, then they should have a > fallback compatible. If they aren't usable, then it should be 1 binding > patch. AFAICT, the new part (onnn,74hc595a) behaves the same at the existing (fairchild,74hc595 and nxp,74lvc594), with regards to the latch signal, so my two binding patches are independent of other. In other words, this one can be dropped, but the other still stands. Best regards -- jn