From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from TX2EHSOBE006.bigfish.com (tx2ehsobe003.messaging.microsoft.com [65.55.88.13]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (Client CN "mail.global.frontbridge.com", Issuer "Cybertrust SureServer Standard Validation CA" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 104ECB6EF1 for ; Fri, 22 Oct 2010 17:01:20 +1100 (EST) Subject: Re: [PATCH v4 5/5] mtd: m25p80: add support to parse the partitions by OF node MIME-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset="us-ascii" From: Kumar Gala In-Reply-To: <1287301490.1951.0.camel@localhost> Date: Fri, 22 Oct 2010 01:01:12 -0500 Message-ID: <3AE1F37D-F328-40EF-8B35-F94CC074D384@freescale.com> References: <1286878714-13090-1-git-send-email-Mingkai.hu@freescale.com> <1286878714-13090-2-git-send-email-Mingkai.hu@freescale.com> <1286878714-13090-3-git-send-email-Mingkai.hu@freescale.com> <1286878714-13090-4-git-send-email-Mingkai.hu@freescale.com> <1286878714-13090-5-git-send-email-Mingkai.hu@freescale.com> <1286878714-13090-6-git-send-email-Mingkai.hu@freescale.com> <1287256647.1781.34.camel@brekeke> <1287301490.1951.0.camel@localhost> To: Grant Likely Cc: dedekind1@gmail.com, David Brownell , Linuxppc-dev list , linux-mtd@lists.infradead.org, spi-devel-general@lists.sourceforge.net, Mingkai Hu List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Oct 17, 2010, at 2:44 AM, Artem Bityutskiy wrote: > On Sat, 2010-10-16 at 19:05 -0600, Grant Likely wrote: >> On Sat, Oct 16, 2010 at 1:17 PM, Artem Bityutskiy = wrote: >>> On Tue, 2010-10-12 at 18:18 +0800, Mingkai Hu wrote: >>>> Signed-off-by: Mingkai Hu >>>> Acked-by: Grant Likely >>>> --- >>>> v4: >>>> - Updated to latest kernel base(Linux 2.6.36-rc7). >>>> - Made changes according to Grant's comments. >>>=20 >>> Looks good to me, pushed to l2-mtd-2.6.git, thanks. >>=20 >> Wait! It will break whenever CONFIG_OF is not set. I broke >> linux-next with this patch. It can be solved by wrapping the block >> with #ifdef CONFIG_OF, but I'd like to find a better solution. >=20 > OK, removed. Grant, poke. Would like to see this either resolved or have this patch = go in for .37 w/CONFIG_OF and fix properly for .38. - k=