Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1246804

Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver

Show all headers | View raw

On Wed, Oct 14, 2015 at 03:07:16PM +0200, M'boumba Cedric Madianga wrote:
> >> +static inline uint32_t stm32_dma_read(struct stm32_dma_device *dmadev, u32 reg)
> >
> > this and few other could be made more readable
> 
> Ok, I could replace uint32_t by u32. Is it what you expect ?
> For others, I don't see how I could make it more readable.
> Could you please give me more details to help me ? Thanks in advance

Yes that is one and also aplitting this up so that it fits within 80chars
while not sacrficing readablity...

> >> +     } else if ((status & STM32_DMA_FEI) && (sfcr & STM32_DMA_SFCR_FEIE)) {
> >> +             dev_err(chan2dev(chan), "DMA error: received FEI event\n");
> >> +             stm32_dma_irq_clear(chan, STM32_DMA_FEI);
> >> +             chan->status = DMA_ERROR;
> >> +             spin_unlock(&chan->vchan.lock);
> >> +             return IRQ_HANDLED;
> >
> > this is repeat of above apart from err print!!
> 
> Not only for print as we also need that to define which bit to clear
> in the IRQ status.
> In that way we will be sure to handle each interrupt event.

You can print error type based on status, or print the whole status but
handle it same for all three cases

> >> +     enum dma_status status;
> >> +     unsigned long flags;
> >> +     unsigned int residue;
> >> +
> >> +     status = dma_cookie_status(c, cookie, state);
> >> +     if (status == DMA_COMPLETE)
> >> +             return status;
> >> +
> >> +     if (!state)
> >> +             return chan->status;
> > why channel status and not status from dma_cookie_status()?
> 
> If status is different than DMA_COMPLETE it seems better to use channel status.
> Indeed, in that way, we have the possibility to notify that there is a
> potential error in the bus via DMA_ERROR.
> But if I return status from dma_cookie_status(), I will always notify
> DMA_IN_PROGRESS.

No, you are supposed to return the descriptor status and not channel status!

-- 
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/4] Add support for STM32 DMA M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 2/4] dmaengine: Add STM32 DMA driver M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
    Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-13 16:40 +0200
      Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 10:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 11:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 11:00 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:20 +0200
          Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 15:30 +0200
            Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:50 +0200
              Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 16:30 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Daniel Thompson <daniel.thompson@linaro.org> - 2015-10-14 17:30 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 17:50 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-15 06:10 +0200
                Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 17:30 +0200
    Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-14 13:20 +0200
      Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 15:10 +0200
        Re: [PATCH v2 2/4] dmaengine: Add STM32 DMA driver Vinod Koul <vinod.koul@intel.com> - 2015-10-14 16:20 +0200
  [PATCH v2 4/4] ARM: configs: Add STM32 DMA support in STM32 defconfig M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 3/4] ARM: dts: Add STM32 DMA support for STM32F429 MCU M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
  [PATCH v2 1/4] dt-bindings: Document the STM32 DMA bindings M'boumba Cedric Madianga <cedric.madianga@gmail.com> - 2015-10-13 16:10 +0200
    Re: [PATCH v2 1/4] dt-bindings: Document the STM32 DMA bindings "M'boumba Cedric Madianga" <cedric.madianga@gmail.com> - 2015-10-14 11:00 +0200

csiph-web