Received: by 2002:a25:86ce:0:0:0:0:0 with SMTP id y14csp957656ybm; Tue, 21 May 2019 06:32:47 -0700 (PDT) X-Google-Smtp-Source: APXvYqy9TzsK8PMrHd0ZpnuYhIolsQQ8bbP65faRVEa8XGz64SbNUSDrk567DsKOpfuK29M/4ySL X-Received: by 2002:a63:5726:: with SMTP id l38mr21536290pgb.344.1558445567678; Tue, 21 May 2019 06:32:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558445567; cv=none; d=google.com; s=arc-20160816; b=MQM2p2R//XiZuxvbw07n+6bH26DI6cAHJ8vaeg2QI6P/xFkqHH3auefIdlgpsnt3WS DiymZd/eap9rHYeKSoxfGbrTfZGxQxdcoVtpsmYLlQc/WkBSXczqdUO73gFCPNktOZ+q xqnzDA8YV7yBQGoMaaCCKvdO4YBvGHii8Rb604K5EP4Znj6znJ2nf5YXr77wz98bAQlC yzkA+EUzFapJ28ZVYHjXuqh1yPhYmJS0vKnFVmngjynQWlZsiEjqNxcsRVajuSJnxSYQ BCslyUq0fuT8a3KIfmesEISECPHf7wM2WnWuPx9jf/D419jIzsk/grf5jn7qb/zuzpXN Eh9A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:message-id:in-reply-to:date:references:subject:cc:to :from; bh=e4dA/6f/SoryfwWh3twxrB4cfwxECXiJiIRUTo2i4b8=; b=Rt6JUACsavlMaOx1QqxHlkKuk0jhaZPqaupgRfkH9gq+FewLdnOMOL9Blt/8zowkhF jvDpq1Fr7xg6YUj8HKovnd6WfAX+KRVO9VeAVryqrV81JAE3laWZHmMxgYUp3FnyQRrD Uq7DB+F0+az+WihFabl++o2YwAgKpuRmEwQhFeVI20FtLOZ4agiEWawspfU5KSSpKKhT YP9+Zv7aoUeuHjJqiNKYF9IS4TSu/y5k+4U0TFFxeof3hI56Tm5w+tKDGl6cMZKfjEIv hA3gOhG/jxPAe7NYFrgPnm80PO/D0RCV5BdnJ+mO+aemOAlguj/BPs24gD0P8PECxg4l mR1w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x14si556567pln.72.2019.05.21.06.32.32; Tue, 21 May 2019 06:32:47 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728175AbfEUNaR convert rfc822-to-8bit (ORCPT + 99 others); Tue, 21 May 2019 09:30:17 -0400 Received: from unicorn.mansr.com ([81.2.72.234]:37392 "EHLO unicorn.mansr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726995AbfEUNaQ (ORCPT ); Tue, 21 May 2019 09:30:16 -0400 Received: by unicorn.mansr.com (Postfix, from userid 51770) id 6E34217102; Tue, 21 May 2019 14:30:14 +0100 (BST) From: =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?= To: Marek Szyprowski Cc: linux-usb@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Greg Kroah-Hartman , Bartlomiej Zolnierkiewicz , Markus Reichl , Krzysztof Kozlowski , Peter Chen , Alan Stern , Rob Herring Subject: Re: [PATCH 0/5] Exynos EHCI/OHCI: resolve conflict with the generic USB device bindings References: <20190521115849.9882-1-m.szyprowski@samsung.com> Date: Tue, 21 May 2019 14:30:14 +0100 In-Reply-To: <20190521115849.9882-1-m.szyprowski@samsung.com> (Marek Szyprowski's message of "Tue, 21 May 2019 13:58:44 +0200") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Marek Szyprowski writes: > Dear All, > > Commit 69bec7259853 ("USB: core: let USB device know device node") added > support for attaching devicetree node for USB devices. Those nodes are > children of their USB host controller. However Exynos EHCI and OHCI > driver bindings already define child-nodes for each physical root hub > port and assigns respective PHY controller and parameters to them. This > leads to the conflict. A workaround for it has been merged as commit > 01d4071486fe ("usb: exynos: add workaround for the USB device bindings > conflict"), but it disabled support for USB device binding for Exynos > EHCI/OHCI controllers. > > This patchset tries to resolve this binding conflict by changing Exynos > EHCI/OHCI bindings: PHYs are moved from the sub-nodes to a standard array > under the 'phys' property. Such solution has been suggested by M?ns > Rullg?rd in the following thread: https://lkml.org/lkml/2019/5/13/228 > > To keep everything working during the transitional time, the changes has > been split into 2 steps. First step (patches 1-3) need to be merged before > the second one (patches 4-5). Patches from each step can be merged to > respective trees without any dependencies - the only requirement is that > second step has to be merged after merging all patches from the first one. > > This patchset has been tested on various Exynos4 boards with different > USB host controller configurations (Odroids family: X2, U3, XU3). > > Best regards > Marek Szyprowski > Samsung R&D Institute Poland > > Marek Szyprowski (5): > dt-bindings: switch Exynos EHCI/OHCI bindings to use array of generic > PHYs > ARM: dts: exynos: Add array of generic PHYs to EHCI/OHCI devices > usb: exynos: add support for getting PHYs from the standard dt array > ARM: dts: exynos: Remove obsolete port sub-nodes from EHCI/OHCI > devices > usb: exynos: Remove support for legacy PHY bindings You could retain compatibility with old devicetrees (which may be useful) by using the "phys" property if it exists and falling back on the old method if it doesn't. Then you would get this sequence of changes: 1. Update binding definition. 2. Support new binding in driver, with fallback to old. 3. Switch dts files to new binding. -- M?ns Rullg?rd