From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754703AbbLIK2l (ORCPT ); Wed, 9 Dec 2015 05:28:41 -0500 Received: from mail-wm0-f50.google.com ([74.125.82.50]:33846 "EHLO mail-wm0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754177AbbLIK2d (ORCPT ); Wed, 9 Dec 2015 05:28:33 -0500 Subject: Re: [PATCH] arm64: dts: mt8173: Add nor flash node To: Daniel Kurtz , bayi cheng References: <1449460394-31601-1-git-send-email-bayi.cheng@mediatek.com> <1449462852.9118.25.camel@mhfsdcap03> <1449463619.9118.31.camel@mhfsdcap03> 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" From: Matthias Brugger Message-ID: <5668024C.7020509@gmail.com> Date: Wed, 9 Dec 2015 11:28:28 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@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. From mboxrd@z Thu Jan 1 00:00:00 1970 From: matthias.bgg@gmail.com (Matthias Brugger) Date: Wed, 9 Dec 2015 11:28:28 +0100 Subject: [PATCH] arm64: dts: mt8173: Add nor flash node In-Reply-To: References: <1449460394-31601-1-git-send-email-bayi.cheng@mediatek.com> <1449462852.9118.25.camel@mhfsdcap03> <1449463619.9118.31.camel@mhfsdcap03> Message-ID: <5668024C.7020509@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.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 at 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 at 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 at 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.