linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Leo Li <pku.leo@gmail.com>
Cc: Felipe Balbi <balbi@kernel.org>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Grygorii Strashko <grygorii.strashko@ti.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	Sekhar Nori <nsekhar@ti.com>, lkml <linux-kernel@vger.kernel.org>,
	David Fisher <david.fisher1@synopsys.com>,
	"Thang Q. Nguyen" <tqnguyen@apm.com>,
	Alan Stern <stern@rowland.harvard.edu>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Stuart Yoder <stuart.yoder@nxp.com>,
	Scott Wood <oss@buserror.net>
Subject: Re: [PATCH] usb: dwc3: host: inherit dma configuration from parent dev
Date: Fri, 02 Sep 2016 12:43:39 +0200	[thread overview]
Message-ID: <6414695.LEIYfGPUEg@wuerfel> (raw)
In-Reply-To: <CADRPPNTQURmuwyuXCjHZcCq6jmQDFiyEUqfcqO-us15nosGCDA@mail.gmail.com>

On Thursday, September 1, 2016 5:14:28 PM CEST Leo Li wrote:
> 
> Hi Felipe and Arnd,
> 
> It has been a while since the last response to this discussion, but we
> haven't reached an agreement yet!  Can we get to a conclusion on if it
> is valid to create child platform device for abstraction purpose?  If
> yes, can this child device do DMA by itself?

I'd say it's no problem for a driver to create child devices in order
to represent different aspects of a device, but you should not rely on
those devices working when used with the dma-mapping interfaces.

This used to be simpler back when we could configure the kernel for
only one SoC platform at a time, and the platforms could provide their
own overrides for the dma-mapping interfaces. These days, we rely on
firmware or bootloader to describe various aspects of how DMA is done,
so you can't assume that passing a device without an of_node pointer
or ACPI data into those functions will do the right thing.

	Arnd

  reply	other threads:[~2016-09-02 10:44 UTC|newest]

Thread overview: 91+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-25 19:21 [PATCH] usb: dwc3: host: inherit dma configuration from parent dev Grygorii Strashko
2016-04-26  6:17 ` Felipe Balbi
2016-04-26  8:14   ` Grygorii Strashko
2016-04-27  5:41     ` Felipe Balbi
2016-04-27 11:55       ` Grygorii Strashko
2016-04-27 13:59       ` Catalin Marinas
2016-04-27 14:11         ` Arnd Bergmann
2016-04-27 15:50           ` Catalin Marinas
2016-04-27 16:04             ` Arnd Bergmann
2016-04-27 16:53               ` Felipe Balbi
2016-04-27 17:42                 ` Arnd Bergmann
2016-04-27 17:59                   ` Alan Stern
2016-04-27 18:08                     ` Arnd Bergmann
2016-04-27 20:05                       ` Felipe Balbi
2016-04-27 21:05                         ` Arnd Bergmann
2016-04-28  6:37                           ` Felipe Balbi
2016-04-28 14:16                             ` Russell King - ARM Linux
2016-04-28 14:23                               ` Arnd Bergmann
2016-04-28 14:27                                 ` Felipe Balbi
2016-09-01 22:14                                   ` Leo Li
2016-09-02 10:43                                     ` Arnd Bergmann [this message]
2016-09-02 10:47                                       ` Russell King - ARM Linux
2016-09-02 11:08                                         ` Felipe Balbi
2016-09-02 14:11                                           ` Felipe Balbi
2016-09-02 14:21                                           ` Alan Stern
2016-09-02 15:51                                             ` Arnd Bergmann
2016-09-07  7:17                                               ` Roger Quadros
2016-09-07  8:29                                                 ` Arnd Bergmann
2016-09-07 13:04                                                   ` Roger Quadros
2016-09-07 14:38                                                     ` Arnd Bergmann
2016-09-02 16:23                                           ` Grygorii Strashko
2016-09-02 10:53                                       ` Felipe Balbi
2016-09-02 11:55                                         ` Robin Murphy
2016-09-02 12:56                                           ` Felipe Balbi
2016-09-02 13:10                                           ` Arnd Bergmann
2016-09-02 22:16                                       ` Leo Li
2016-09-05 15:39                                         ` Arnd Bergmann
2016-09-06  6:35                                           ` Peter Chen
2016-09-06  6:40                                             ` Felipe Balbi
2016-09-06 10:46                                               ` Arnd Bergmann
2016-09-06 10:50                                                 ` Felipe Balbi
2016-09-06 13:27                                                   ` Arnd Bergmann
2016-09-07  6:51                                                     ` Felipe Balbi
2016-09-07  7:44                                                     ` Peter Chen
2016-09-07  8:52                                                       ` Arnd Bergmann
2016-09-07  9:29                                                         ` Peter Chen
2016-09-07  9:35                                                           ` Russell King - ARM Linux
2016-09-07 10:18                                                             ` Felipe Balbi
2016-09-06 10:38                                             ` Arnd Bergmann
2016-09-07  6:33                                               ` Peter Chen
2016-09-07  8:48                                                 ` Arnd Bergmann
2016-09-07  9:55                                                   ` Peter Chen
2016-09-07 10:33                                                     ` Robin Murphy
2016-09-07 10:47                                                       ` Felipe Balbi
2016-09-14 16:31                                                         ` Lorenzo Pieralisi
2016-09-14 21:50                                                           ` Arnd Bergmann
2016-09-07 10:24                                                   ` Felipe Balbi
2016-09-07 15:24                                                     ` Arnd Bergmann
2016-09-07 16:08                                                       ` Alan Stern
2016-09-07 19:45                                                         ` Arnd Bergmann
2016-09-08  1:15                                                       ` Peter Chen
2016-09-08  8:02                                                         ` Arnd Bergmann
2016-09-08  8:03                                                       ` Felipe Balbi
2016-09-08  8:26                                                         ` Arnd Bergmann
2016-09-08  8:29                                                           ` Felipe Balbi
2016-09-08  8:45                                                             ` Arnd Bergmann
2016-09-08  9:43                                                               ` Felipe Balbi
2016-09-08 10:17                                                                 ` Arnd Bergmann
2016-09-08 11:00                                                                   ` Felipe Balbi
2016-09-08 11:11                                                                     ` Arnd Bergmann
2016-09-08 11:20                                                                       ` Felipe Balbi
2016-09-08 11:39                                                                         ` Arnd Bergmann
2016-09-08 11:52                                                                           ` Felipe Balbi
2016-09-08 12:46                                                                             ` Arnd Bergmann
2016-09-08 12:02                                                                     ` Grygorii Strashko
2016-09-08 12:14                                                                       ` Arnd Bergmann
2016-09-08 12:28                                                                   ` Peter Chen
2016-09-08 12:52                                                                     ` Arnd Bergmann
2016-09-09  1:37                                                                       ` Peter Chen
2016-09-08 12:59                                                                     ` Grygorii Strashko
2016-09-09  1:52                                                                       ` Peter Chen
2016-09-21 11:06                                                       ` Sriram Dash
2016-09-21 11:31                                                         ` Arnd Bergmann
2016-09-21 11:43                                                           ` Sriram Dash
2016-09-21 12:48                                                             ` Arnd Bergmann
2016-09-22  5:02                                                               ` Sriram Dash
2016-10-07 22:46                                                                 ` Leo Li
2016-09-21 17:14                                                             ` [PATCH] usb: xhci: Fix the patch inherit dma configuration from kbuild test robot
2016-04-27 20:57                   ` [PATCH] usb: dwc3: host: inherit dma configuration from parent dev Felipe Balbi
2016-04-27 14:14         ` Grygorii Strashko
2016-05-05 17:07 ` Brian Norris

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6414695.LEIYfGPUEg@wuerfel \
    --to=arnd@arndb.de \
    --cc=balbi@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=david.fisher1@synopsys.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=grygorii.strashko@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nsekhar@ti.com \
    --cc=oss@buserror.net \
    --cc=pku.leo@gmail.com \
    --cc=stern@rowland.harvard.edu \
    --cc=stuart.yoder@nxp.com \
    --cc=tqnguyen@apm.com \
    --cc=yoshihiro.shimoda.uh@renesas.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).