On Fri, Apr 13, 2018 at 10:35:48AM +0200, Wolfram Sang wrote:
>
> > As the implementation stands it is global - only one DMA RX stream may
> > be in flight for the entire system. I am wondering if that is the right
> > granularity for the restriction. Perhaps it could be per-SDHI device,
> > allow
> As the implementation stands it is global - only one DMA RX stream may
> be in flight for the entire system. I am wondering if that is the right
> granularity for the restriction. Perhaps it could be per-SDHI device,
> allowing concurrent streams on different SDHI devices.
As we have only one D
On Thu, Apr 12, 2018 at 01:21:42PM +0200, Wolfram Sang wrote:
>
> > > +static unsigned long global_flags;
> >
> > Is the restriction on concurrent DMA RX streams global or per-device?
>
> ? Each device has only one DMA RX channel. Hey Simon, you upstreamed
> this driver :) Or did I get the quest
On Thu, Apr 12, 2018 at 01:34:41PM +0200, Geert Uytterhoeven wrote:
> Hi Wolfram,
>
> On Thu, Apr 12, 2018 at 1:31 PM, Wolfram Sang wrote:
> >> That should have been caught by the !soc check above, and have already
> >> returned with -ENODEV.
> >
> > Now I get it: You mean non-0 check, not non-NU
Hi Wolfram,
On Thu, Apr 12, 2018 at 1:31 PM, Wolfram Sang wrote:
>> That should have been caught by the !soc check above, and have already
>> returned with -ENODEV.
>
> Now I get it: You mean non-0 check, not non-NULL check...
soc->data _is_ a pointer. You only cast it to an integer on the next
> That should have been caught by the !soc check above, and have already
> returned with -ENODEV.
Now I get it: You mean non-0 check, not non-NULL check...
signature.asc
Description: PGP signature
Hi Wolfram,
On Thu, Apr 12, 2018 at 1:11 PM, Wolfram Sang wrote:
>> > + if (soc->data)
>>
>> This non-NULL check is not really needed.
>
> And if we match using the Gen3 generic compatible with a non-whitelisted
> SoC?
That should have been caught by the !soc check above, and have already
> > +static unsigned long global_flags;
>
> Is the restriction on concurrent DMA RX streams global or per-device?
? Each device has only one DMA RX channel. Hey Simon, you upstreamed
this driver :) Or did I get the question wrong?
> > + if (dir == DMA_FROM_DEVICE)
> > + clear_bit(SD
> > + if (soc->data)
>
> This non-NULL check is not really needed.
And if we match using the Gen3 generic compatible with a non-whitelisted
SoC?
signature.asc
Description: PGP signature
On Tue, Apr 10, 2018 at 11:38:27AM +0200, Wolfram Sang wrote:
> Early revisions of certain SoCs cannot do multiple DMA RX streams in
> parallel. To avoid data corruption, only allow one DMA RX channel and
> fall back to PIO, if needed.
>
> Signed-off-by: Wolfram Sang
> Reviewed-by: Yoshihiro Shim
Hi Wolfram,
On Tue, Apr 10, 2018 at 11:38 AM, Wolfram Sang
wrote:
> Early revisions of certain SoCs cannot do multiple DMA RX streams in
> parallel. To avoid data corruption, only allow one DMA RX channel and
> fall back to PIO, if needed.
>
> Signed-off-by: Wolfram Sang
> Reviewed-by: Yoshihiro
Early revisions of certain SoCs cannot do multiple DMA RX streams in
parallel. To avoid data corruption, only allow one DMA RX channel and
fall back to PIO, if needed.
Signed-off-by: Wolfram Sang
Reviewed-by: Yoshihiro Shimoda
Tested-by: Nguyen Viet Dung
---
drivers/mmc/host/renesas_sdhi_inter
12 matches
Mail list logo