From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Brugger Subject: Re: [PATCH] arm64: dts: mt8173: Add nor flash node Date: Wed, 9 Dec 2015 11:28:28 +0100 Message-ID: <5668024C.7020509@gmail.com> References: <1449460394-31601-1-git-send-email-bayi.cheng@mediatek.com> <1449462852.9118.25.camel@mhfsdcap03> <1449463619.9118.31.camel@mhfsdcap03> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Daniel Kurtz , bayi cheng Cc: "open list:OPEN FIRMWARE AND..." , srv_heupstream , "linux-kernel@vger.kernel.org" , linux-mediatek@lists.infradead.org, linux-mtd@lists.infradead.org, Brian Norris , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On 07/12/15 09:19, Daniel Kurtz wrote: > Bayi, > > On Mon, Dec 7, 2015 at 12:46 PM, bayi cheng wrote: >> On Mon, 2015-12-07 at 12:34 +0800, bayi cheng wrote: >>> On Mon, 2015-12-07 at 12:09 +0800, Daniel Kurtz wrote: >>>> Bayi, >>>> >>>> On Mon, Dec 7, 2015 at 11:53 AM, Bayi Cheng wrote: >>>>> Add Mediatek nor flash node >>>>> >>>>> Signed-off-by: Bayi Cheng >>>>> Acked-by: Brian Norris >>>>> --- >>>>> Previous version of this patch is >>>>> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-November/386005.html >>>>> Drop flash node from mt8173.dtsi accroding review comment >>>> >>>> Is there a flash on mt8173-evb that can be accessed by the nor_flash ? >>>> If so, can you include flash@0 node in the mt8173-evb.dts. >>>> >>>> -Dan >>>> >>> Hi Daniel, Thanks for your comments! you are right, there is a flash on >>> mt8173-evb which can be accessed by the nor_flash, and I will include >>> flash@0 node in mt8173-evb.dts (kernel v3.18) too. >>> >>> Bayi >> >> Hi Dan, I have reworked a EVB board for spi nor verify, we replaced the >> spi nand flash with spi nor flash on EVB board, >> and our shipping EVB board haven't support spi nor flash, So I will not >> include flash@0 node in the mt8173-evb.dts. I am sorry to made you >> confused! > > Thanks for confirming. > > Reviewed-by: Daniel Kurtz > Added to v4.5-next/dts Thanks a lot.