Re: [PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-08-28 Thread Richard Zhao
On Tue, Aug 28, 2012 at 12:52:19PM +0200, Marc Kleine-Budde wrote: > On 08/28/2012 12:46 PM, Richard Zhao wrote: > > On Tue, Aug 28, 2012 at 12:31:25PM +0200, Marc Kleine-Budde wrote: > >> If udc_start() fails the qh_pool dma-pool cannot be closed because > >> it's still in use. This patch factors

Re: [PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-08-28 Thread Marc Kleine-Budde
On 08/28/2012 12:46 PM, Richard Zhao wrote: > On Tue, Aug 28, 2012 at 12:31:25PM +0200, Marc Kleine-Budde wrote: >> If udc_start() fails the qh_pool dma-pool cannot be closed because >> it's still in use. This patch factors out the dma_pool_free() loop >> into destroy_eps() and calls it in the erro

Re: [PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-08-28 Thread Richard Zhao
On Tue, Aug 28, 2012 at 12:31:25PM +0200, Marc Kleine-Budde wrote: > If udc_start() fails the qh_pool dma-pool cannot be closed because > it's still in use. This patch factors out the dma_pool_free() loop > into destroy_eps() and calls it in the error path of udc_start(), > too. > > Cc: Richard Zh

[PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-08-28 Thread Marc Kleine-Budde
If udc_start() fails the qh_pool dma-pool cannot be closed because it's still in use. This patch factors out the dma_pool_free() loop into destroy_eps() and calls it in the error path of udc_start(), too. Cc: Richard Zhao Signed-off-by: Marc Kleine-Budde --- Hello, Richard, can you please inclu

Re: [PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-07-16 Thread Richard Zhao
Hi Alex, Could you please also review & ack this patch? Thanks Richard -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-07-11 Thread Richard Zhao
From: Marc Kleine-Budde If udc_start() fails the qh_pool dma-pool cannot be closed because it's still in use. This patch factors out the dma_pool_free() loop into destroy_eps() and calls it in the error path of udc_start(), too. Signed-off-by: Marc Kleine-Budde Signed-off-by: Richard Zhao ---