From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 28B7922F388; Wed, 11 Mar 2026 12:59:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773233953; cv=none; b=PMXmwDahQRJa+AQtCFcUcfLhTf/0iqg0Ndy5HCn4kBLIclGibftk0ATZ3st6/pJo9PiqAgjPTJVxBRVPJrN7Qdnpcg5PndkS5HFggWmlxsosXrfTwS3jgEEDdf67JwAUR3wQX8zGUw1r5FRBt6H8CPDKxQf1sLJUw5g2oSr4oa4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773233953; c=relaxed/simple; bh=TjeokAYJpp1s8ejVuxn4YOBMcXKuwV+jjPwk9H6WUlg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=DD07WhyYx72Ql4R5wJR2kWK9WwcZK6iXLFoLl9/fDDpOg5g4eC4E4DYyiCojOTXzoh2pojO4PoA1BuYHyXeYXdZQqzBwxfuPoB0Jley56FIHcZ1SGBKcPdKxSYk/mtitPOQzBSCgkPnDXvmfPS1PeJHxwQmFmwLwZDO9M794ZCE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=FaKnJAMi; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="FaKnJAMi" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7479CC4CEF7; Wed, 11 Mar 2026 12:59:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1773233953; bh=TjeokAYJpp1s8ejVuxn4YOBMcXKuwV+jjPwk9H6WUlg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FaKnJAMibdSL50v7fK4cMBJpjtPIPNeAej7LyaIVWAvGD0yBFrer4ROIhH9C534n5 K7tdPLdGyoK7MrbvUYTsw0MT3oI4QXHTlu9xKS2iDVqYfQ0ZgS2wo8Rw0+XLSY8Sor 1YHEwiF18POygmMmizzjPzlEG60Tiu/2QCUfX6xcxyK/wbNaeshm/b3ik/ilWIQmA1 bPvt9Lilf+PQJRMWBhZRqLRYLeHJFyDF3KYEeXrS9XnCioDvl94GjbWwW2okE/hOhs ZFyoNPVlYDCJOg7oQ0urN7+phPwrwDcXH84C9+wM8Ht+iLFgl9j+yVmVuHT4+nEgUX QQfrEY1elo0Qw== Date: Wed, 11 Mar 2026 13:59:10 +0100 From: Krzysztof Kozlowski To: Xu Yang Cc: Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Li Jun , Badhri Jagan Sridharan , Heikki Krogerus , linux-usb@vger.kernel.org, imx@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] dt-bindings: usb: nxp,ptn5110: add optional orientation-gpio property Message-ID: <20260311-hasty-talented-shark-5fdecd@quoll> References: <20260310-support-setting-orientation-use-gpio-v1-0-da31dc6cd641@nxp.com> <20260310-support-setting-orientation-use-gpio-v1-1-da31dc6cd641@nxp.com> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20260310-support-setting-orientation-use-gpio-v1-1-da31dc6cd641@nxp.com> On Tue, Mar 10, 2026 at 04:13:56PM +0800, Xu Yang wrote: > The Type-C chip know the cable orientation and then normally will set the > switch channel to correctly configure the data path. Some chips itself > support to output the control signal by indicating the capability in > bit[0] of STANDARD_OUTPUT_CAPABILITIES register and do it in > CONFIG_STANDARD_OUTPUT register. For other chips which don't present this > capability currently there are no way to achieve the orientation setting. > Add an optional "orientation-gpio" property to achieve the same purpose. > > Signed-off-by: Xu Yang > --- > Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml b/Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml > index 65a8632b4d9e..866b5d033f4e 100644 > --- a/Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml > +++ b/Documentation/devicetree/bindings/usb/nxp,ptn5110.yaml > @@ -26,6 +26,11 @@ properties: > $ref: /schemas/connector/usb-connector.yaml# > unevaluatedProperties: false > > + orientation-gpio: > + description: Optional orientation select control if the chip doesn't > + present "Connector Orientation" bit[0] in STANDARD_OUTPUT_CAPABILITIES What do you mean by "if the chip"? There is only one chip, so its registers are fixed. Why would this device sometimes report it and sometimes not? Best regards, Krzysztof