lkml.org 
[lkml]   [2017]   [Jun]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC PATCH 2/4] i2c: add docs to clarify DMA handling

> > +Therefore, it is *not* mandatory that the buffer of an i2c message is DMA safe.
> > +It does not seem reasonable to apply additional burdens when the feature is so
> > +rarely used. However, it is recommended to use a DMA-safe buffer, if your
> > +message size is likely applicable for DMA (FIXME: > 8 byte?).
>
> So you expect drivers to fall back to PIO automatically if the buffer is
> not DMA safe. Sounds good to me.

Yes, I strongly recommend that. Otherwise, drivers can always deal with
bounce buffers on their own.

> However, your check for a DMA-capable buffer is invoked only if
> CONFIG_DMA_API_DEBUG is enabled:

is *NOT* enabled!

>
> #if !defined(CONFIG_DMA_API_DEBUG)
> if (!virt_addr_valid(msg->buf) || object_is_on_stack(msg->buf)) {
> pr_debug("msg buffer to 0x%04x might not be DMA capable\n",
> msg->addr);
> return -EFAULT;
> }
> #endif
>

The #if block is there because DMA_API_DEBUG does a lot more, but if the
check in the helper is enabled, the core will fall back to PIO and you
won't get the additional info from DMA_API_DEBUG.

I think this needs a comment :)

Now OK?

Regards,

Wolfram
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2017-06-12 00:44    [W:0.055 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site