Re: [REVIEWv7 PATCH 04/12] vb2: don't free alloc context if it is ERR_PTR

2014-11-26 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday 18 November 2014 13:51:00 Hans Verkuil wrote: > From: Hans Verkuil > > Don't try to free a pointer containing an ERR_PTR(). Wouldn't it be easier to return NULL from vb2_dma_contig_alloc_ctx() instead of an ERR_PTR ? > Signed-off-by: Hans Verkuil

Re: [REVIEWv7 PATCH 04/12] vb2: don't free alloc context if it is ERR_PTR

2014-11-23 Thread Pawel Osciak
On Tue, Nov 18, 2014 at 9:51 PM, Hans Verkuil wrote: > From: Hans Verkuil > > Don't try to free a pointer containing an ERR_PTR(). > > Signed-off-by: Hans Verkuil Acked-by: Pawel Osciak -- Best regards, Pawel Osciak -- To unsubscribe from this list: send the line "unsubscribe linux-media" in

[REVIEWv7 PATCH 04/12] vb2: don't free alloc context if it is ERR_PTR

2014-11-18 Thread Hans Verkuil
From: Hans Verkuil Don't try to free a pointer containing an ERR_PTR(). Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/videobuf2-dma-contig.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/videobuf2-dma-contig.c b/drivers/media/v4l2-core