Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-24 Thread Peter Chen
On Tue, Mar 08, 2016 at 07:40:08PM -0800, Bjorn Andersson wrote: > On Tue, Mar 8, 2016 at 11:52 AM, Li Yang wrote: > > On Wed, Mar 2, 2016 at 4:59 PM, Li Yang wrote: > >> On Mon, Feb 22, 2016 at 4:07 PM, Bjorn Andersson > >>

RE: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-20 Thread Rajesh Bhagat
r.kernel.org>; linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT > > On Thu, Mar 17, 2016 at 04:52:55PM +0100, Arnd Bergmann wrote: > > On Monday 14 March 2016 18:51:08 Peter Chen wrote: > > > On Wed,

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-19 Thread Russell King - ARM Linux
On Fri, Mar 18, 2016 at 09:54:14AM +0800, Peter Chen wrote: > Although I don't know what kinds of bugs it may have, it may be > met before, otherwise, why most of platform drivers need to call > dma_set_coherent_mask or dma_coerce_mask_and_coherent explicitly See Documentation/DMA-API.txt,

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-19 Thread Arnd Bergmann
On Monday 14 March 2016 18:51:08 Peter Chen wrote: > On Wed, Mar 09, 2016 at 05:16:50PM -0600, Li Yang wrote: > > On Tue, Mar 8, 2016 at 9:40 PM, Bjorn Andersson > > wrote: > > > On Tue, Mar 8, 2016 at 11:52 AM, Li Yang wrote: > > >> On Wed, Mar

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-19 Thread Peter Chen
On Thu, Mar 17, 2016 at 04:52:55PM +0100, Arnd Bergmann wrote: > On Monday 14 March 2016 18:51:08 Peter Chen wrote: > > On Wed, Mar 09, 2016 at 05:16:50PM -0600, Li Yang wrote: > > > On Tue, Mar 8, 2016 at 9:40 PM, Bjorn Andersson > > > wrote: > > > > On Tue, Mar 8,

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-19 Thread Arnd Bergmann
On Friday 18 March 2016 09:54:14 Peter Chen wrote: > > > > I don't think it's a very widespread problem, there are only very few > > developers that intentionally use this method, and some use the > > platform_device_register_full() call to create a device with a known > > mask, which is

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-14 Thread Peter Chen
On Wed, Mar 09, 2016 at 05:16:50PM -0600, Li Yang wrote: > On Tue, Mar 8, 2016 at 9:40 PM, Bjorn Andersson > wrote: > > On Tue, Mar 8, 2016 at 11:52 AM, Li Yang wrote: > >> On Wed, Mar 2, 2016 at 4:59 PM, Li Yang wrote: > >>>

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-09 Thread Li Yang
On Tue, Mar 8, 2016 at 9:40 PM, Bjorn Andersson wrote: > On Tue, Mar 8, 2016 at 11:52 AM, Li Yang wrote: >> On Wed, Mar 2, 2016 at 4:59 PM, Li Yang wrote: >>> On Mon, Feb 22, 2016 at 4:07 PM, Bjorn Andersson >>>

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-08 Thread Bjorn Andersson
On Tue, Mar 8, 2016 at 11:52 AM, Li Yang wrote: > On Wed, Mar 2, 2016 at 4:59 PM, Li Yang wrote: >> On Mon, Feb 22, 2016 at 4:07 PM, Bjorn Andersson >> wrote: >>> On Mon 22 Feb 02:03 PST 2016, Srinivas Kandagatla wrote: >>>

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-08 Thread Li Yang
On Wed, Mar 2, 2016 at 4:59 PM, Li Yang wrote: > On Mon, Feb 22, 2016 at 4:07 PM, Bjorn Andersson > wrote: >> On Mon 22 Feb 02:03 PST 2016, Srinivas Kandagatla wrote: >> >>> >>> >>> On 22/02/16 05:32, Bjorn Andersson wrote: >>> >On certain

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-03-02 Thread Li Yang
On Mon, Feb 22, 2016 at 4:07 PM, Bjorn Andersson wrote: > On Mon 22 Feb 02:03 PST 2016, Srinivas Kandagatla wrote: > >> >> >> On 22/02/16 05:32, Bjorn Andersson wrote: >> >On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set >> >to be able to do DMA

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-02-22 Thread Peter Chen
On Mon, Feb 22, 2016 at 02:07:50PM -0800, Bjorn Andersson wrote: > On Mon 22 Feb 02:03 PST 2016, Srinivas Kandagatla wrote: > > > I'm still puzzled to why the chipidea lives as a child device of the msm > device; but as this is a rather common structure I believe this still > needs to be figured

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-02-22 Thread Bjorn Andersson
On Mon 22 Feb 02:03 PST 2016, Srinivas Kandagatla wrote: > > > On 22/02/16 05:32, Bjorn Andersson wrote: > >On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set > >to be able to do DMA allocations, so use the of_dma_configure() helper > >to populate the dma properties and

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-02-22 Thread Srinivas Kandagatla
On 22/02/16 05:32, Bjorn Andersson wrote: On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set to be able to do DMA allocations, so use the of_dma_configure() helper to populate the dma properties and assign an appropriate dma_ops. Signed-off-by: Bjorn Andersson

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-02-21 Thread Bjorn Andersson
On Sun 21 Feb 22:02 PST 2016, Peter Chen wrote: > On Sun, Feb 21, 2016 at 09:32:13PM -0800, Bjorn Andersson wrote: > > On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set > > to be able to do DMA allocations, so use the of_dma_configure() helper > > to populate the dma

Re: [PATCH] usb: chipidea: Configure DMA properties and ops from DT

2016-02-21 Thread Peter Chen
On Sun, Feb 21, 2016 at 09:32:13PM -0800, Bjorn Andersson wrote: > On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set > to be able to do DMA allocations, so use the of_dma_configure() helper > to populate the dma properties and assign an appropriate dma_ops. > >