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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0538BC0015E for ; Wed, 19 Jul 2023 06:04:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=hzEgT88RSnUEgTkjXuxFFzOhTkPpXg5H3yeNl03C7ys=; b=tgOh1KNwGD/x1N yoy7vLsKzlXbhM4ea2AjPIIgJxwnBoHvM1cvEcI8RN1Txa6ZcFmj0UmNyhkSPAnf6tYcJJIV+ywjI I6f5qIpGfD1kTA6Dat+/xPW3w4EGa8wP+TTicmZzednStBm/WSbIzwZk3vWy/uQDw16u2BUb881lc dqXWIlsWp4PtLENHHxfIk1bFAfrOB1KvBHogkoj7gKUl4WExocNgIn5OKKcZcVYy+/5gSyNtY3MJv s2pUv+f15/Ej9x7gwPm5CcZ4+T1FHn3MqLFR0RilFEW8LEKLNr9vnpIedj8KImTWVG5+WVOybZMrk alk5YZmvkE0p8Woliwaw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qM0Ie-005XEX-1Z; Wed, 19 Jul 2023 06:04:36 +0000 Received: from desiato.infradead.org ([90.155.92.199]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qM0Id-005XDl-1N for linux-arm-kernel@bombadil.infradead.org; Wed, 19 Jul 2023 06:04:35 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=SGXQZgN/g+wPBhhsHsjo+3LZ7ggoinuh8PEJDuHw4Go=; b=Ldw6J7rdApNwAkYRdbVmOR1+oi hQO52RWMjJMdVbMjVULaPrIizMiFW7/t2P0PYB8YzETz0JDOiAvIxo6k44GKPCcXcOdNu+w7/2eFo HfAgJA2E1uA1Ps8JcVZQRPTROjrBwGWBgTWoUci2udN/MqU34/YrGaZmRqJZxsBtaxaSrwza7P+xe +6gYaFWzv7MBE8HpEA5Uuc6Axs9v4CPzPJpew1YDHX/pyE7NUU47YHeNqqmU+MFhMSZCLgrJEpJFE aYiKVsuywQuq/IhHD8XmtE2Jz9EHh4LX3Fi2TqK22I03/lJaXLmszpvwC1gA4HLi4VqILtIK+bNm8 eeJRBHSg==; Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by desiato.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qLeLq-00BDgg-0S for linux-arm-kernel@lists.infradead.org; Tue, 18 Jul 2023 06:38:29 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id ACECA6147A; Tue, 18 Jul 2023 06:37:52 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9A2E2C433CA; Tue, 18 Jul 2023 06:37:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689662272; bh=lkqYV4xx86Ti2v26sOBZLYVym3rSZZUqRoPkM+IKoIw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ErVVeclVfGNmw++ELI2ou+/DJlthBIAb8F+aOVs2HNeAeXox2fhR4yU/63Vldz/Ho fX2HnwwDRgt44w5Hxp7h5qjarW2E4bzDoLsCzKwvckViiTKS8rRMXIT7lUpCmBiMQ4 cqvAv6Er76p9wv3yLZND7LmDKTf/V40xIEhm9ZE+4//X6h3x4rU8yb+OE/NhajahNH UtbKFVmFqB9NkvULpl+OBVFo66DEdz5hyu0yQ/nNktnKdWXttolPgyewitgThonTCs YxcLZLGCx3c4aEYC5K5N7/rQ7LRoM5fOVpYY7H0fdJ39deYqsa9Qp4FaGKCkH0SGjM W3IKaoLF/9IIg== Date: Tue, 18 Jul 2023 14:37:15 +0800 From: Shawn Guo To: Hugo Villeneuve Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Hugo Villeneuve , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] arm64: dts: imx8mn-var-som-symphony: fix USB OTG Message-ID: <20230718063715.GL9559@dragon> References: <20230705174932.3652479-1-hugo@hugovil.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230705174932.3652479-1-hugo@hugovil.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230718_073827_525914_7065BD85 X-CRM114-Status: GOOD ( 27.28 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Jul 05, 2023 at 01:49:32PM -0400, Hugo Villeneuve wrote: > From: Hugo Villeneuve > > USB OTG is currently broken on the Variscite Symphony EVK and imx8mn > nano SOM. > > The PTN5150 circuitry on newer versions of the Symphony EVK board has > a non-standard configuration in which the PTN5150 IRQ pin is left > unconnected, and the PTN5150 ID pin is connected to GPIO1_IO11. This > requires changes to the ptn5150 driver to support this new mode. > Variscite have indicated their intention to submit those changes > upstream. > > In the meantime, import device tree changes from linux-5.15 branch of > varigit repos to at least make the USB OTG port operate correctly in > host mode. > > Fixes: 7358e05bddca ("arm64: dts: imx8mn-var-som-symphony: Add Variscite Symphony board with VAR-SOM-MX8MN") Has USB OTG been ever worked at all? If the answer is no, it's not a fix but a new feature, and I would suggest you rework the patch subject and drop the Fixes tag. Shawn > Signed-off-by: Hugo Villeneuve > --- > Link: [v1] https://lkml.org/lkml/2023/7/4/702 > > Changes from v1: > - Add comments about PTN5150 IRQ/ID line connections > - Remove "typec1_con: connector" node > - Change IRQ type to IRQ_TYPE_EDGE_FALLING > > .../dts/freescale/imx8mn-var-som-symphony.dts | 32 +++++++++++++++++-- > 1 file changed, 30 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-var-som-symphony.dts b/arch/arm64/boot/dts/freescale/imx8mn-var-som-symphony.dts > index 406a711486da..a7a57442cb81 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mn-var-som-symphony.dts > +++ b/arch/arm64/boot/dts/freescale/imx8mn-var-som-symphony.dts > @@ -1,11 +1,14 @@ > // SPDX-License-Identifier: (GPL-2.0+ OR MIT) > /* > + * Supports Symphony evaluation board versions >= 1.4a. > + * > * Copyright 2019-2020 Variscite Ltd. > * Copyright (C) 2020 Krzysztof Kozlowski > */ > > /dts-v1/; > > +#include > #include "imx8mn-var-som.dtsi" > > / { > @@ -100,14 +103,26 @@ enet-sel-hog { > }; > }; > > + /* > + * For Symphony board version <= 1.4, the PTN5150 IRQ pin is connected > + * to GPIO1_IO11 on the SoM (R106 present, R132 absent). From Symphony > + * board version >= 1.4a, the PTN5150 ID pin is connected to GPIO1_IO11 > + * on the SoM (R106 absent, R132 present). > + */ > extcon_usbotg1: typec@3d { > compatible = "nxp,ptn5150"; > reg = <0x3d>; > interrupt-parent = <&gpio1>; > - interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > + interrupts = <11 IRQ_TYPE_EDGE_FALLING>; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_ptn5150>; > status = "okay"; > + > + port { > + typec1_dr_sw: endpoint { > + remote-endpoint = <&usb1_drd_sw>; > + }; > + }; > }; > }; > > @@ -148,8 +163,21 @@ &uart3 { > }; > > &usbotg1 { > + dr_mode = "otg"; > + hnp-disable; > + srp-disable; > + adp-disable; > + usb-role-switch; > disable-over-current; > - extcon = <&extcon_usbotg1>, <&extcon_usbotg1>; > + samsung,picophy-pre-emp-curr-control = <3>; > + samsung,picophy-dc-vol-level-adjust = <7>; > + status = "okay"; > + > + port { > + usb1_drd_sw: endpoint { > + remote-endpoint = <&typec1_dr_sw>; > + }; > + }; > }; > > &iomuxc { > > base-commit: d528014517f2b0531862c02865b9d4c908019dc4 > -- > 2.30.2 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel