[PATCH] powerpc/rmu: Fix the error memory free parameters
Gang.Liu at freescale.com
Gang.Liu at freescale.com
Wed Apr 23 17:16:52 EST 2014
Hi Scott,
Do you have some comments about this patch,
or would you please help to apply it?
Thanks a lot!
Liu Gang
> -----Original Message-----
> From: Liu Gang [mailto:Gang.Liu at freescale.com]
> Sent: Wednesday, April 09, 2014 6:05 PM
> To: linuxppc-dev at lists.ozlabs.org; Wood Scott-B07421
> Cc: Zang Roy-R61911; Liu Gang-B34182
> Subject: [PATCH] powerpc/rmu: Fix the error memory free parameters
>
> There are error parameters should be corrected when calling
> dma_free_coherent to free rmu rx-ring buffers in fsl_open_inb_mbox()
> function.
>
> Signed-off-by: Liu Gang <Gang.Liu at freescale.com>
> Reported-by: Dan Carpenter <dan.carpenter at oracle.com>
> ---
> arch/powerpc/sysdev/fsl_rmu.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/powerpc/sysdev/fsl_rmu.c
> b/arch/powerpc/sysdev/fsl_rmu.c index 00e224a..b48197a 100644
> --- a/arch/powerpc/sysdev/fsl_rmu.c
> +++ b/arch/powerpc/sysdev/fsl_rmu.c
> @@ -881,9 +881,9 @@ fsl_open_inb_mbox(struct rio_mport *mport, void
> *dev_id, int mbox, int entries)
> rc = request_irq(IRQ_RIO_RX(mport), fsl_rio_rx_handler, 0,
> "msg_rx", (void *)mport);
> if (rc < 0) {
> - dma_free_coherent(priv->dev, RIO_MSG_BUFFER_SIZE,
> - rmu->msg_tx_ring.virt_buffer[i],
> - rmu->msg_tx_ring.phys_buffer[i]);
> + dma_free_coherent(priv->dev,
> + rmu->msg_rx_ring.size * RIO_MAX_MSG_SIZE,
> + rmu->msg_rx_ring.virt, rmu->msg_rx_ring.phys);
> goto out;
> }
>
> --
> 1.8.5
>
More information about the Linuxppc-dev
mailing list