From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lists.s-osg.org ([54.187.51.154]:39882 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932293AbcFBPUv (ORCPT ); Thu, 2 Jun 2016 11:20:51 -0400 Subject: Re: [PATCH v4 5/7] ARM: Exynos: remove code for MFC custom reserved memory handling To: Krzysztof Kozlowski , Marek Szyprowski , linux-media@vger.kernel.org, linux-samsung-soc@vger.kernel.org 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> Cc: devicetree@vger.kernel.org, Sylwester Nawrocki , Kamil Debski , Kukjin Kim , Uli Middelberg , Bartlomiej Zolnierkiewicz From: Javier Martinez Canillas Message-ID: <5a12a8be-0402-dc0c-d242-5d9f3145e001@osg.samsung.com> Date: Thu, 2 Jun 2016 11:20:40 -0400 MIME-Version: 1.0 In-Reply-To: <574BEBB8.8040606@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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. > What is your final decision for DTS patches? Which approach for MFC > reserved memory node? > > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America