From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933475AbcIFKjN (ORCPT ); Tue, 6 Sep 2016 06:39:13 -0400 Received: from mout.kundenserver.de ([212.227.126.130]:56033 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932799AbcIFKjJ (ORCPT ); Tue, 6 Sep 2016 06:39:09 -0400 From: Arnd Bergmann To: Peter Chen Cc: Leo Li , Felipe Balbi , Grygorii Strashko , Russell King - ARM Linux , Catalin Marinas , Yoshihiro Shimoda , "linux-usb@vger.kernel.org" , Sekhar Nori , lkml , Stuart Yoder , Scott Wood , David Fisher , "Thang Q. Nguyen" , Alan Stern , Greg Kroah-Hartman , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH] usb: dwc3: host: inherit dma configuration from parent dev Date: Tue, 06 Sep 2016 12:38:29 +0200 Message-ID: <5844875.KucAoPMrXi@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <20160906063529.GA22312@b29397-desktop> References: <4865343.bOXkeC8XtQ@wuerfel> <20160906063529.GA22312@b29397-desktop> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:HEGXNZ5za9AAIgL7oh64i1oVECR4n4swFz4GjHK7oQWgRwqBL0l REN6ruSC56jdlB3hOmZeyQAOD9zJWRpbUgG7DL07ZYWYneLqIqiV0lfn/XwA1xxGI4FHMdh vOEhnMJeS8lmDcoSz2WLXPtaJvKgRGsSeIoxZ3rQHYn6HffrCoLvulFrdcgpMge5HMqnXhL 41n3Q8FIvRFNgf8pcadVg== X-UI-Out-Filterresults: notjunk:1;V01:K0:SrCqACfSWTM=:yWy9REcRWs3qE+fa1encgz IU58yjmWKM724o+xIT0xpMpnoqzf7SPt6UzBgblOv9xUK+nupPi3M340XjpnYQA7zLwqefzw2 IE5p9CIlvLDZ/2WP1ZU+YDFFPdHCXua8M8jJDrexoIYrZtuBvVzddMOY+lRVAUHthbOfKEKQf iUYCUifqqokq5Oqnfv2vJtNFJonIIBQLNhCpb/p7FC9laB+7abRws/3HB4JhlnaG6Ltpkhjwu uFQ+2FGP1RjgD8XR/pcSewNQi/skfo9+A2jm4y0GoK4+18zX83QgM1vaY9DPQTil/5HSAP/9O Wga0AAT0ZoiSSbzi2jGKNwMHTVA3bdGx3b8rEMjbw0Lg3KTYVijZOxLHPeXJ6aaAeRypGegBJ OkVlR/wMhhLMOTUnGM4vVVachZh7r23jLWX446hIbShnSIDU3fW0Q07bWu1YOupi4r+3BfLAu Q0WuyfeK18IDVYVLdb1Ojf/D+RjHfiy+gE5qn8k2znTNINPKtC1NoxeXZ1gDQpZ22Dlf11Ubb X9pWFLr8MSFbhMQctkwD+aHLSkIu2CUsSA6v4BlPtWaMHJEqVuOlRy10CCxjfZUSKTHY/HXWO JV7k2baymiV+DRf6Cbbx8EL/0NSfignMXm8y3PpYwhjSCnX7pA1xU77Izh/uOqepklNKvD5MW MaWZy+khnUDg+T8zV4wy1+laS8lvm17s48cGEeDYzPqrfKmbvSFqibhqQ5fMPWyFv4+oA4s3B b/Te1prRpzB9Hleq Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, September 6, 2016 2:35:29 PM CEST Peter Chen wrote: > On Mon, Sep 05, 2016 at 05:39:27PM +0200, Arnd Bergmann wrote: > > On Friday, September 2, 2016 5:16:31 PM CEST Leo Li wrote: > > > > Most of these are probably never used with any nonstandard > > DMA settings (IOMMU, cache coherency, offset, ...). > > > > One thing we could possibly do is to go through these and > > replace the hardcoded dma mask setup with of_dma_configure() > > in all cases in which we actually use DT for probing, which > > should cover the interesting cases. > > > > One case I am going to work is to let USB chipidea driver support iommu, > the chipidea core device is no of_node, and created by > platform_add_device on the runtime. Using of_dma_configure with parent > of_node is a solution from my point, like [1]. > > https://lkml.org/lkml/2016/2/22/7 Right, that should make it work with iommu as well. However, it does not solve the other issue I mentioned above, with boards that have USB devices hardwired to a chipidea host controller that need configuration from DT. For that, we still need to come up with another way to associate the DT hierarchy in the host bridge node with the Linux platform_device. Arnd