Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751547AbbGLOib (ORCPT ); Sun, 12 Jul 2015 10:38:31 -0400 Received: from smtp03.smtpout.orange.fr ([80.12.242.125]:31367 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750865AbbGLOi3 (ORCPT ); Sun, 12 Jul 2015 10:38:29 -0400 X-ME-Helo: belgarion X-ME-Auth: amFyem1pay5yb2JlcnRAb3JhbmdlLmZy X-ME-Date: Sun, 12 Jul 2015 16:38:28 +0200 X-ME-IP: 86.199.64.162 From: Robert Jarzmik To: Guennadi Liakhovetski Cc: Mauro Carvalho Chehab , Jiri Kosina , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Robert Jarzmik Subject: Re: [PATCH v2 1/4] media: pxa_camera: fix the buffer free path References: <1436120872-24484-1-git-send-email-robert.jarzmik@free.fr> <1436120872-24484-2-git-send-email-robert.jarzmik@free.fr> X-URL: http://belgarath.falguerolles.org/ Date: Sun, 12 Jul 2015 16:35:42 +0200 In-Reply-To: (Guennadi Liakhovetski's message of "Sun, 12 Jul 2015 15:58:06 +0200 (CEST)") Message-ID: <87380tl9z5.fsf@belgarion.home> User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 906 Lines: 28 Guennadi Liakhovetski writes: > Hi Robert, > > On Sun, 5 Jul 2015, Robert Jarzmik wrote: > >> From: Robert Jarzmik >> >> Fix the error path where the video buffer wasn't allocated nor >> mapped. In this case, in the driver free path don't try to unmap memory >> which was not mapped in the first place. > > Have I missed your reply to my comments to v1 of this patch? This one > seems to be its exact copy? Yeah, that's because I don't have any ... reply from you on v1. At least I don't remember it and in [1] I don't see it. Cheers. -- Robert [1] http://www.spinics.net/lists/linux-media/msg88021.html -- 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/