Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1371852 > unrolled thread
| Started by | Vinod Koul <vinod.koul@intel.com> |
|---|---|
| First post | 2016-04-05 20:10 +0200 |
| Last post | 2016-04-05 20:10 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 3/3] spi: rockchip: check requesting dma channel with EPROBE_DEFER Vinod Koul <vinod.koul@intel.com> - 2016-04-05 20:10 +0200
| From | Vinod Koul <vinod.koul@intel.com> |
|---|---|
| Date | 2016-04-05 20:10 +0200 |
| Subject | Re: [PATCH 3/3] spi: rockchip: check requesting dma channel with EPROBE_DEFER |
| Message-ID | <rkDAm-864-23@gated-at.bofh.it> |
On Tue, Mar 22, 2016 at 10:53:21AM +0800, Shawn Lin wrote:
> >Strange, but on 4.4 there was some extra code in
> >dma_request_slave_channel() that wasn't in
> >dma_request_slave_channel_reason(). ...but looks like that all got
> >cleaned up in the same CL that added the new name.
>
> dma_request_chan already return ERR_PTR(-EPROBE_DEFER), but
> dma_request_slave_channel ignore this and rewrite it to be NULL.
> Strange behaviour looks to me. commit 0ad7c00057dc ("dma: add channel
> request API that supports deferred probe") did the right thing, but
> what happened then? It was drop for some reasons?
>
> Hello Vinod,
>
> Could you please elaborate some more infomation to commit 0ad7c00057dc
> ("dma: add channel request API that supports deferred probe") :) ?
The request API was cleaned up here and please see the Documentation on how
to do this.
See the commit a8135d0d7 - dmaengine: core: Introduce new, universal API to
request a channel
Thanks
--
~Vinod
Back to top | Article view | linux.kernel
csiph-web