lkml.org 
[lkml]   [2020]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v1 RFC 1/2] spi: introduce fallback to pio
    On Sun, Jun 14, 2020 at 01:04:57PM +0000, Robin Gong wrote:
    > On 2020/06/12 22:16 Mark Brown <broonie@kernel.org> wrote:
    > > On Fri, Jun 12, 2020 at 01:48:41PM +0000, Robin Gong wrote:
    > > > On 2020/06/12 18:14 Mark Brown <broonie@kernel.org> wrote:

    Please delete unneeded context from mails when replying. Doing this
    makes it much easier to find your reply in the message, helping ensure
    it won't be missed by people scrolling through the irrelevant quoted
    material.

    > > No, I mean that the reason the DMA transfer fails may be something that
    > > happens after we've started putting things on the bus - the bit about FIFOs is
    > > just a random example of an error that could happen.

    > Sorry Mark for that I can't get your point... The bus error such as data corrupt
    > seems not the spi core's business since it can only be caught in spi controller
    > driver or upper level such as mtd driver (spi-nor) which know what's the failure
    > happen at spi bus HW level or what's the correct data/message. In other words,
    > spi core can't detect such error by transfer_one().

    If we see an error in transfer_one() it could be from anything, we've no
    idea what happened on the bus - the controller may have got part way
    through the transfer before failing.

    > But despite of that case, do you think this patch is valid for transfer_one() failue
    > in dma and fallback to pio?

    No, not unless we know that nothing went out on the bus.
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2020-06-15 14:36    [W:2.216 / U:0.088 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site