* [PATCH] arm64: dts: mt8173: Add nor flash node @ 2015-12-07 3:53 Bayi Cheng 2015-12-07 4:09 ` Daniel Kurtz 0 siblings, 1 reply; 6+ messages in thread From: Bayi Cheng @ 2015-12-07 3:53 UTC (permalink / raw) To: Matthias Brugger Cc: Brian Norris, Daniel Kurtz, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w, Bayi Cheng Add Mediatek nor flash node Signed-off-by: Bayi Cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> Acked-by: Brian Norris <computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> --- 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 --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi index 4dd5f93..af08e97 100644 --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi @@ -387,6 +387,17 @@ status = "disabled"; }; + nor_flash: spi@1100d000 { + compatible = "mediatek,mt8173-nor"; + reg = <0 0x1100d000 0 0xe0>; + clocks = <&pericfg CLK_PERI_SPI>, + <&topckgen CLK_TOP_SPINFI_IFR_SEL>; + clock-names = "spi", "sf"; + #address-cells = <1>; + #size-cells = <0>; + status = "disabled"; + }; + i2c3: i2c@11010000 { compatible = "mediatek,mt8173-i2c"; reg = <0 0x11010000 0 0x70>, -- 1.8.1.1.dirty -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: mt8173: Add nor flash node 2015-12-07 3:53 [PATCH] arm64: dts: mt8173: Add nor flash node Bayi Cheng @ 2015-12-07 4:09 ` Daniel Kurtz [not found] ` <CAGS+omCuB3VGcPk3fezakdVDnFB3GAmMD+wZ_6nGdv+uHoB40g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Daniel Kurtz @ 2015-12-07 4:09 UTC (permalink / raw) To: Bayi Cheng Cc: Matthias Brugger, Brian Norris, open list:OPEN FIRMWARE AND..., linux-arm-kernel@lists.infradead.org, linux-mediatek, linux-kernel@vger.kernel.org, linux-mtd, srv_heupstream Bayi, On Mon, Dec 7, 2015 at 11:53 AM, Bayi Cheng <bayi.cheng@mediatek.com> wrote: > Add Mediatek nor flash node > > Signed-off-by: Bayi Cheng <bayi.cheng@mediatek.com> > Acked-by: Brian Norris <computersforpeace@gmail.com> > --- > 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 > > --- > arch/arm64/boot/dts/mediatek/mt8173.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > index 4dd5f93..af08e97 100644 > --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi > +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > @@ -387,6 +387,17 @@ > status = "disabled"; > }; > > + nor_flash: spi@1100d000 { > + compatible = "mediatek,mt8173-nor"; > + reg = <0 0x1100d000 0 0xe0>; > + clocks = <&pericfg CLK_PERI_SPI>, > + <&topckgen CLK_TOP_SPINFI_IFR_SEL>; > + clock-names = "spi", "sf"; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > i2c3: i2c@11010000 { > compatible = "mediatek,mt8173-i2c"; > reg = <0 0x11010000 0 0x70>, > -- > 1.8.1.1.dirty > ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CAGS+omCuB3VGcPk3fezakdVDnFB3GAmMD+wZ_6nGdv+uHoB40g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH] arm64: dts: mt8173: Add nor flash node [not found] ` <CAGS+omCuB3VGcPk3fezakdVDnFB3GAmMD+wZ_6nGdv+uHoB40g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2015-12-07 4:34 ` bayi cheng 2015-12-07 4:46 ` bayi cheng 0 siblings, 1 reply; 6+ messages in thread From: bayi cheng @ 2015-12-07 4:34 UTC (permalink / raw) To: Daniel Kurtz Cc: Matthias Brugger, Brian Norris, open list:OPEN FIRMWARE AND..., linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, srv_heupstream On Mon, 2015-12-07 at 12:09 +0800, Daniel Kurtz wrote: > Bayi, > > On Mon, Dec 7, 2015 at 11:53 AM, Bayi Cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> wrote: > > Add Mediatek nor flash node > > > > Signed-off-by: Bayi Cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> > > Acked-by: Brian Norris <computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > > --- > > 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 > > > > --- > > arch/arm64/boot/dts/mediatek/mt8173.dtsi | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > index 4dd5f93..af08e97 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > @@ -387,6 +387,17 @@ > > status = "disabled"; > > }; > > > > + nor_flash: spi@1100d000 { > > + compatible = "mediatek,mt8173-nor"; > > + reg = <0 0x1100d000 0 0xe0>; > > + clocks = <&pericfg CLK_PERI_SPI>, > > + <&topckgen CLK_TOP_SPINFI_IFR_SEL>; > > + clock-names = "spi", "sf"; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + status = "disabled"; > > + }; > > + > > i2c3: i2c@11010000 { > > compatible = "mediatek,mt8173-i2c"; > > reg = <0 0x11010000 0 0x70>, > > -- > > 1.8.1.1.dirty > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: mt8173: Add nor flash node 2015-12-07 4:34 ` bayi cheng @ 2015-12-07 4:46 ` bayi cheng 2015-12-07 8:19 ` Daniel Kurtz 0 siblings, 1 reply; 6+ messages in thread From: bayi cheng @ 2015-12-07 4:46 UTC (permalink / raw) To: Daniel Kurtz Cc: open list:OPEN FIRMWARE AND..., srv_heupstream, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Matthias Brugger, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Brian Norris, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org 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 <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> wrote: > > > Add Mediatek nor flash node > > > > > > Signed-off-by: Bayi Cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> > > > Acked-by: Brian Norris <computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > > > --- > > > 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! Bayi > > > > > > --- > > > arch/arm64/boot/dts/mediatek/mt8173.dtsi | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > > index 4dd5f93..af08e97 100644 > > > --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > > +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > > @@ -387,6 +387,17 @@ > > > status = "disabled"; > > > }; > > > > > > + nor_flash: spi@1100d000 { > > > + compatible = "mediatek,mt8173-nor"; > > > + reg = <0 0x1100d000 0 0xe0>; > > > + clocks = <&pericfg CLK_PERI_SPI>, > > > + <&topckgen CLK_TOP_SPINFI_IFR_SEL>; > > > + clock-names = "spi", "sf"; > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + status = "disabled"; > > > + }; > > > + > > > i2c3: i2c@11010000 { > > > compatible = "mediatek,mt8173-i2c"; > > > reg = <0 0x11010000 0 0x70>, > > > -- > > > 1.8.1.1.dirty > > > > > > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: mt8173: Add nor flash node 2015-12-07 4:46 ` bayi cheng @ 2015-12-07 8:19 ` Daniel Kurtz 2015-12-09 10:28 ` Matthias Brugger 0 siblings, 1 reply; 6+ messages in thread From: Daniel Kurtz @ 2015-12-07 8:19 UTC (permalink / raw) To: bayi cheng Cc: open list:OPEN FIRMWARE AND..., srv_heupstream, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Matthias Brugger, linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Brian Norris, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org Bayi, On Mon, Dec 7, 2015 at 12:46 PM, bayi cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> 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 <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> wrote: >> > > Add Mediatek nor flash node >> > > >> > > Signed-off-by: Bayi Cheng <bayi.cheng-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> >> > > Acked-by: Brian Norris <computersforpeace-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> > > --- >> > > 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 <djkurtz@chromium. org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: mt8173: Add nor flash node 2015-12-07 8:19 ` Daniel Kurtz @ 2015-12-09 10:28 ` Matthias Brugger 0 siblings, 0 replies; 6+ messages in thread From: Matthias Brugger @ 2015-12-09 10:28 UTC (permalink / raw) To: Daniel Kurtz, bayi cheng Cc: open list:OPEN FIRMWARE AND..., srv_heupstream, linux-kernel@vger.kernel.org, linux-mediatek, linux-mtd, Brian Norris, 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 <bayi.cheng@mediatek.com> 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 <bayi.cheng@mediatek.com> wrote: >>>>> Add Mediatek nor flash node >>>>> >>>>> Signed-off-by: Bayi Cheng <bayi.cheng@mediatek.com> >>>>> Acked-by: Brian Norris <computersforpeace@gmail.com> >>>>> --- >>>>> 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 <djkurtz@chromium. org> > Added to v4.5-next/dts Thanks a lot. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-12-09 10:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-07 3:53 [PATCH] arm64: dts: mt8173: Add nor flash node Bayi Cheng
2015-12-07 4:09 ` Daniel Kurtz
[not found] ` <CAGS+omCuB3VGcPk3fezakdVDnFB3GAmMD+wZ_6nGdv+uHoB40g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-12-07 4:34 ` bayi cheng
2015-12-07 4:46 ` bayi cheng
2015-12-07 8:19 ` Daniel Kurtz
2015-12-09 10:28 ` Matthias Brugger
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).