From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH v4 5/7] ARM: Exynos: remove code for MFC custom reserved memory handling Date: Thu, 02 Jun 2016 18:31:23 +0200 Message-ID: <57505F5B.90101@samsung.com> References: <1464096690-23605-1-git-send-email-m.szyprowski@samsung.com> <1464096690-23605-6-git-send-email-m.szyprowski@samsung.com> <574BEBB8.8040606@samsung.com> <5a12a8be-0402-dc0c-d242-5d9f3145e001@osg.samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <5a12a8be-0402-dc0c-d242-5d9f3145e001@osg.samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Javier Martinez Canillas , Marek Szyprowski , linux-media@vger.kernel.org, linux-samsung-soc@vger.kernel.org Cc: devicetree@vger.kernel.org, Sylwester Nawrocki , Kamil Debski , Kukjin Kim , Uli Middelberg , Bartlomiej Zolnierkiewicz List-Id: devicetree@vger.kernel.org On 06/02/2016 05:20 PM, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 05/30/2016 03:28 AM, Krzysztof Kozlowski wrote: >> On 05/24/2016 03:31 PM, Marek Szyprowski wrote: >>> Once MFC driver has been converted to generic reserved memory bindings, >>> there is no need for custom memory reservation code. >>> >>> Signed-off-by: Marek Szyprowski >>> --- >>> arch/arm/mach-exynos/Makefile | 2 - >>> arch/arm/mach-exynos/exynos.c | 19 -------- >>> arch/arm/mach-exynos/mfc.h | 16 ------- >>> arch/arm/mach-exynos/s5p-dev-mfc.c | 93 -------------------------------------- >>> 4 files changed, 130 deletions(-) >>> delete mode 100644 arch/arm/mach-exynos/mfc.h >>> delete mode 100644 arch/arm/mach-exynos/s5p-dev-mfc.c >> >> Thanks, applied. >> > > This patch can't be applied before patches 2/5 and 3/5, or the custom > memory regions reservation will break with the current s5p-mfc driver. Yes, I know. As I understood from talk with Marek, the driver is broken now so continuous work was not chosen. If it is not correct and full bisectability is required, then entire patchset requires special handling - I need a stable tag from media tree. Without this everything will be broken anyway. Best regards, Krzysztof