From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from host.buserror.net (host.buserror.net [209.198.135.123]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 467431A0599 for ; Fri, 19 Feb 2016 11:07:21 +1100 (AEDT) Message-ID: <1455840434.2463.97.camel@buserror.net> From: Scott Wood To: Leo Li , Raghav Dogra Cc: linux-mtd@lists.infradead.org, linuxppc-dev , Prabhakar Kushwaha , Brian Norris , Raghav Dogra , Jaiprakash Singh Date: Thu, 18 Feb 2016 18:07:14 -0600 In-Reply-To: References: <1455708258-3831-1-git-send-email-raghav.dogra@nxp.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Subject: Re: [PATCH][v3] mtd/ifc: Add support for IFC controller version 2.0 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2016-02-18 at 15:18 -0600, Leo Li wrote: > On Wed, Feb 17, 2016 at 5:24 AM, Raghav Dogra wrote: > > The new IFC controller version 2.0 has a different memory map page. > > Upto IFC 1.4 PAGE size is 4 KB and from IFC2.0 PAGE size is 64KB. > > This patch segregates the IFC global and runtime registers to appropriate > > PAGE sizes. > > > > Signed-off-by: Jaiprakash Singh > > Signed-off-by: Raghav Dogra > > Acked-by: Li Yang > > Signed-off-by: Raghav Dogra > > --- > > Changes for v3: not dependent on > > "drivers/memory: Add deep sleep support for IFC" patch > > > > Changes for v2: rebased to resolve conflicts > > Applicable to git://git.infradead.org/l2-mtd.git > > > > This patch is dependent on "drivers/memory: Add deep sleep support for > > IFC" > > https://patchwork.ozlabs.org/patch/582762/ > > which is also applicable to git://git.infradead.org/l2-mtd.git > > This patch seems to be in good shape, but the dependency is still > having quite some feedback to be addressed. Depending on it will > greatly delay the time that this critical patch for LS2 to be merged. > Could you remove the dependency like Scott already suggested? According to the changelog the dependency has been removed (it would have been clearer to remove this comment as well...). -Scott