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 X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4D5C6C433E0 for ; Tue, 14 Jul 2020 08:22:07 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 194AE21897 for ; Tue, 14 Jul 2020 08:22:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="z0n43pOX"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=toradex.com header.i=@toradex.com header.b="EdOOluN3" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 194AE21897 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=toradex.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To:References: Message-ID:Date:Subject:To:From:Reply-To:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=n7e23tTtswFXoORFOB4W3Zzb/S+g0L1SuHzsoUqh8Yg=; b=z0n43pOXB1LOLNPR8eh6WQdIB WEABW8IG6Nj9oCrWnHWp8md16NxbSdmSxTYsjtmf3qqYnrMC6GKXCaOdPYkqJDIaRUVoJeAoeB0T7 KcZ/K+LvKPQqGNLCu+vKVmiUQl+boV6664NOE2YaTk9Dkmop5KCkgNPmNvd1Rv0geT47X+javw1E4 J7rAtuhDjfsc+EEKyppAswDuqT61jCPhF8jhFtp82HbQNYST3MOt6QrV9YQK1PPafGd5gxGxoyQQv uYK7F5AlmMjii9YoLaWpaNsEV8wzKZ8l3EvKhiQNsmtC6bPpQXsXUhzzAelp5j6cBxYLOf53c0ntj RRLiXdQpQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jvGBC-00036J-1v; Tue, 14 Jul 2020 08:20:46 +0000 Received: from mail-eopbgr20124.outbound.protection.outlook.com ([40.107.2.124] helo=EUR02-VE1-obe.outbound.protection.outlook.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jvGB9-00035B-Vs for linux-arm-kernel@lists.infradead.org; Tue, 14 Jul 2020 08:20:44 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R8d8pC/mcYvOkXPM6BqZQXewdH8YZxjYZr45BttVIXJ2lvBLXqSOLX0DJueCFF0NZZV0K8OGRbyWE5iNOXqfm6n3zkUZaekm0k3n2FHsbmbHmIVzxPYJK6ZaHp3jBkJeM0QGU/m/cFBD0lyWK05FNhZ4lpEkGuNYl+K2Z/Qp6mDGmt4kySJU8aFI15dBAPamQabvyGY3tIuZtgQ3Yqq65d+/zySsZq92w63wrDZM7j7kN3mBbwTiMZIqNtn6BkhGAnrho71sJCJOMOmycd4EtpqgkpABpHsb0KEMZ6fD31OFkCL+/sG4zRB978mE04c7kygNJ14q9jRoVHngLaByZw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Mu3LL8WGBx6/arpP7s7dDydU6qV2Orm739eV+CssGgc=; b=MHAQrzQ6xMdSKhUj/qoGK6ObfaAEHcXJtirEAH72rWZ8jDyS/+PjO4uab3Nrcw04MSpf1/61/ZGGowB60FzFeHIl55uwI4vhaIdn+YQ58OtFs1YfWxEaLGnhesBrcCaFt8bSZ3U2bUnsjcqAxmDfQz8POTxbBXvnhTJUGYfmpqYbicpPAqFOlDvo+zmBHuPgxi62ud3EmZvvJuKEkMbTk4PGerhlzVYMdG4Ckn+wIZhBwSGIK7bucYbYS6+hqQdRB9y44MJX7yPoFvN2mSbaEOX05QpmwR2f/nWpZ4RqeVRsBVYvmbkqCz5xEEC27VSwPnxpaTkIUU+tn0b255rb/A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=toradex.com; dmarc=pass action=none header.from=toradex.com; dkim=pass header.d=toradex.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=toradex.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Mu3LL8WGBx6/arpP7s7dDydU6qV2Orm739eV+CssGgc=; b=EdOOluN3Wy4kUKBopXCVk+InMz7sZq5RxVISXYi/u96JOXypcz1CrDPQQQScuuAamLbsVUxXRasWz+6HCglkin85Z7WNxq9WDl0YADb/yOtzIP2ezfOvrd0JWcK5wzUB4T+7UbtLaGWTimA8V2X4091UzKLhzozfIhYZI/Z4eXY= Received: from AM6PR05MB6120.eurprd05.prod.outlook.com (2603:10a6:20b:a8::25) by AM6PR05MB4951.eurprd05.prod.outlook.com (2603:10a6:20b:8::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3174.23; Tue, 14 Jul 2020 08:20:35 +0000 Received: from AM6PR05MB6120.eurprd05.prod.outlook.com ([fe80::1d81:6a9b:8c26:3b7d]) by AM6PR05MB6120.eurprd05.prod.outlook.com ([fe80::1d81:6a9b:8c26:3b7d%4]) with mapi id 15.20.3174.026; Tue, 14 Jul 2020 08:20:34 +0000 From: Philippe Schenker To: "devicetree@vger.kernel.org" , "a.fatoum@pengutronix.de" , "shawnguo@kernel.org" Subject: Re: [PATCH 3/3] ARM: dts: colibri-imx7: add usb dual-role switch capability Thread-Topic: [PATCH 3/3] ARM: dts: colibri-imx7: add usb dual-role switch capability Thread-Index: AQHWVr15sZvxLwnO3E+b+KcqajdLWakFR14AgAAjuQCAAU+xgIAABwcA Date: Tue, 14 Jul 2020 08:20:34 +0000 Message-ID: <83b230cef109e1a413ab1036c21ad763a1d127eb.camel@toradex.com> References: <20200710132423.497230-1-philippe.schenker@toradex.com> <20200710132423.497230-3-philippe.schenker@toradex.com> <73240ab0-5069-40c5-4ade-7fcc2207dfaf@pengutronix.de> <7e6f8135-120f-0831-20ad-25648ea646b8@pengutronix.de> In-Reply-To: <7e6f8135-120f-0831-20ad-25648ea646b8@pengutronix.de> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Evolution 3.36.3 authentication-results: vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=none action=none header.from=toradex.com; x-originating-ip: [31.10.206.124] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 1b0f3fdb-3340-4f59-0c2c-08d827cec849 x-ms-traffictypediagnostic: AM6PR05MB4951: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:10000; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 3WLSZpgY2Zv8NpQEF1ER/7koPJhilvkTwsLpHkLWJ2yJPTEcjAXPN7dSvi1sq9Nv7h2gx+fonrc5iY0MuDx7P6ETs4D8oJNypslPFvY1slfya4NHLyaXGKy4coJ3dTJgSVHQ20t5Eb1EItbBUHDjW/jJvVOglAx6qylb2Ib5ZyCuvxRV0U2ELj1IDMFdqQir2t7idd+0WXxKGYtlH4Cv75/yZipCDkETeJn+cS9yN/EZU+1R6afZFt7KaU5u1rNygM2XQfxTJd8Y9IG6o6rnui92uS5JC5N2+FEBJfPtuQblMLx+o11NIZQSSnFzIdCQvI+WC/kbjAQOUskFRAS9Cw== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR05MB6120.eurprd05.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(346002)(396003)(366004)(376002)(136003)(39850400004)(6512007)(8936002)(26005)(6486002)(76116006)(91956017)(110136005)(71200400001)(4326008)(186003)(6506007)(53546011)(478600001)(2906002)(7416002)(44832011)(66446008)(66946007)(66556008)(64756008)(66476007)(86362001)(316002)(54906003)(8676002)(2616005)(83380400001)(36756003)(5660300002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata: b+Pr3dWar97rK5XDnDlTMwDWJuQ1wE0P6NZroSpN8/be6QHKV7pHrJ7unAYBgk8MmMZwu4kkCVmKaon468iw/cCxTHMBlN02ejb5nE0zeyZXS8rmahcI4C2bRe/0Op7xxG4nwjSo43m/SnpHzWMeG36w9JsgMpSn1wLbFgWlKYlwEKrqXs8OIcKdz17uxRFDqpto2feCcpD/HSqLms9OMtSY3qQb3Kz9xRo1FkjhCExXdezBDe39+FvFzKTLp154qarEU5MZZLyKFCYSFVXXbgnXCoiMljITLQPPmwOk0xNJWhtHkmnj//+WIRVni+/NC85Q3+f9uXKusiqdQAtsovkvmG3HEwk7/qQ/x9BvQPZ6kz497jrdUqmlYKBLO1DLkpLvN8AoYBXfDHvkpea2uay6gZDj5tqGNCFcUhb5pb2MOfobkwzl9scUMtEBZ/htDAyTbkj4NBaqvSZ74itq+TqNMXtp2m4gu4DwrfpL7OA= x-ms-exchange-transport-forked: True Content-ID: MIME-Version: 1.0 X-OriginatorOrg: toradex.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AM6PR05MB6120.eurprd05.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1b0f3fdb-3340-4f59-0c2c-08d827cec849 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jul 2020 08:20:34.8832 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: d9995866-0d9b-4251-8315-093f062abab4 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: en2v4lVXPj6270KB6HRB0+KbJaceXHPr3qfMD3Koy8Z/NmOeYqV8YFZ4Xg1xG/XQmvaf+GJYhHSsjGmTXip9GmIhEIH1dyx/fdGPx8gdX8g= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR05MB4951 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200714_042044_045647_46DFED4B X-CRM114-Status: GOOD ( 22.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "s.hauer@pengutronix.de" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" , "linux-imx@nxp.com" , "kernel@pengutronix.de" , "festevam@gmail.com" , "linux-arm-kernel@lists.infradead.org" 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 Tue, 2020-07-14 at 09:55 +0200, Ahmad Fatoum wrote: > Hello Philippe, > > On 7/13/20 1:53 PM, Philippe Schenker wrote: > > On Mon, 2020-07-13 at 11:46 +0200, Ahmad Fatoum wrote: > > > Hello Philippe, > > > > > > > + extcon_usbc_det: usbc_det { > > > > + compatible = "linux,extcon-usb-gpio"; > > > > > > According to 4602f3bff266 ("usb: common: add USB GPIO based > > > connection > > > detection driver"): > > > "the old way using extcon to support USB Dual-Role switch is now > > > deprecated > > > when use Type-B connector." > > > > > > Have you considered using a compatible = "gpio-usb-b-connector" > > > child > > > node instead? > > > > > > Cheers, > > > Ahmad > > > > Thanks for the Hint Ahmad, > > > > I already tried and just now tried again but it doesn't work on our > > hardware. Are you sure this works with chipidea driver? > > I haven't, just wanted to point its existence out in case you didn't > know. > Seems we need to call of_platform_populate somewhere in the driver. > Unsure what other changes are necessary. Yep, thanks for that! It seems like that need to be really called as I can't get a probe of the new usb connector driver. I quickly grepped through code and saw that tegra USB driver is calling of_platform_populate and they also use that gpio-usb-b-connector. > > > Should this new usb-connector stuff work in general with every old > > driver? > > If the driver support isn't there yet, I think use of extcon is fine > as is. > > Thanks > Ahmad @Shawn: Could you pull patches #2 and #3 back in? Thanks guys! Philippe > > > Philippe > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel