All of lore.kernel.org
 help / color / mirror / Atom feed
From: wangzhou.bry@gmail.com (Zhou Wang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: hip04: NAND controller support
Date: Fri, 24 Oct 2014 14:24:14 +0800	[thread overview]
Message-ID: <5449F08E.6040808@gmail.com> (raw)
In-Reply-To: <CAN1soZyCojPe8HYQQqrPj7e84TrhTsQpupMqf5semZDn3NabCQ@mail.gmail.com>

On 2014?10?23? 17:32, Haojian Zhuang wrote:
> On Thu, Oct 23, 2014 at 4:24 PM, Zhou Wang <wangzhou.bry@gmail.com> wrote:
>> This patch adds dts support of NAND flash controller for Hisilicon Soc Hip04.
>> The patch is based on v3.18-rc1. I have tested that NAND flash controller
>> works fine in Hip04 D01 board.
>>
>> Signed-off-by: Zhou Wang <wangzhou.bry@gmail.com>
>> ---
>>   arch/arm/boot/dts/hip04.dtsi |   30 ++++++++++++++++++++++++++++++
>>   1 file changed, 30 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/hip04.dtsi b/arch/arm/boot/dts/hip04.dtsi
>> index 93b6c90..642a160 100644
>> --- a/arch/arm/boot/dts/hip04.dtsi
>> +++ b/arch/arm/boot/dts/hip04.dtsi
>> @@ -263,5 +263,35 @@
>>                          interrupts = <0 372 4>;
>>                  };
>>
>> +               nand: nand at 4020000 {
>> +                       compatible = "hisilicon,504-nfc";
>> +                       reg = <0x4020000 0x10000>, <0x5000000 0x1000>;
>> +                       interrupts = <0 379 4>;
>> +                       nand-bus-width = <8>;
>> +                       nand-ecc-mode = "none";
>> +                       hisi,nand-ecc-bits = <0>;
>> +                       #address-cells = <1>;
>> +                       #size-cells = <1>;
>> +
>> +                       partition at 0 {
>> +                               label = "nand_text";
>> +                               reg = <0x00000000 0x00400000>;
>> +                       };
>> +
>> +                       partition at 00400000 {
>> +                               label = "nand_monitor";
>> +                               reg = <0x00400000 0x00400000>;
>> +                       };
>> +
>> +                       partition at 00800000 {
>> +                               label = "nand_kernel";
>> +                               reg = <0x00800000 0x00800000>;
>> +                       };
>> +
>> +                       partition at 01000000 {
>> +                               label = "nand_fs";
>> +                               reg = <0x01000000 0x1f000000>;
>> +                       };
>
> Partitions are different on different boards. But nand controller is
> same on each chip. You
> should define nand controller in soc dtsi file, and define partition
> in board dts file.

My mistake, I will move the partition information to dts file.
Thanks a lot, Haojian.

Kind Regards
Zhou Wang
>
> Regards
> Haojian
>

WARNING: multiple messages have this Message-ID (diff)
From: Zhou Wang <wangzhou.bry-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Haojian Zhuang <haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
	Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>,
	Kevin Hilman
	<khilman-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	"xuwei (O)" <xuwei5-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>,
	wangzhou1-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] ARM: dts: hip04: NAND controller support
Date: Fri, 24 Oct 2014 14:24:14 +0800	[thread overview]
Message-ID: <5449F08E.6040808@gmail.com> (raw)
In-Reply-To: <CAN1soZyCojPe8HYQQqrPj7e84TrhTsQpupMqf5semZDn3NabCQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On 2014年10月23日 17:32, Haojian Zhuang wrote:
> On Thu, Oct 23, 2014 at 4:24 PM, Zhou Wang <wangzhou.bry-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> This patch adds dts support of NAND flash controller for Hisilicon Soc Hip04.
>> The patch is based on v3.18-rc1. I have tested that NAND flash controller
>> works fine in Hip04 D01 board.
>>
>> Signed-off-by: Zhou Wang <wangzhou.bry-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> ---
>>   arch/arm/boot/dts/hip04.dtsi |   30 ++++++++++++++++++++++++++++++
>>   1 file changed, 30 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/hip04.dtsi b/arch/arm/boot/dts/hip04.dtsi
>> index 93b6c90..642a160 100644
>> --- a/arch/arm/boot/dts/hip04.dtsi
>> +++ b/arch/arm/boot/dts/hip04.dtsi
>> @@ -263,5 +263,35 @@
>>                          interrupts = <0 372 4>;
>>                  };
>>
>> +               nand: nand@4020000 {
>> +                       compatible = "hisilicon,504-nfc";
>> +                       reg = <0x4020000 0x10000>, <0x5000000 0x1000>;
>> +                       interrupts = <0 379 4>;
>> +                       nand-bus-width = <8>;
>> +                       nand-ecc-mode = "none";
>> +                       hisi,nand-ecc-bits = <0>;
>> +                       #address-cells = <1>;
>> +                       #size-cells = <1>;
>> +
>> +                       partition@0 {
>> +                               label = "nand_text";
>> +                               reg = <0x00000000 0x00400000>;
>> +                       };
>> +
>> +                       partition@00400000 {
>> +                               label = "nand_monitor";
>> +                               reg = <0x00400000 0x00400000>;
>> +                       };
>> +
>> +                       partition@00800000 {
>> +                               label = "nand_kernel";
>> +                               reg = <0x00800000 0x00800000>;
>> +                       };
>> +
>> +                       partition@01000000 {
>> +                               label = "nand_fs";
>> +                               reg = <0x01000000 0x1f000000>;
>> +                       };
>
> Partitions are different on different boards. But nand controller is
> same on each chip. You
> should define nand controller in soc dtsi file, and define partition
> in board dts file.

My mistake, I will move the partition information to dts file.
Thanks a lot, Haojian.

Kind Regards
Zhou Wang
>
> Regards
> Haojian
>

--
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

  reply	other threads:[~2014-10-24  6:24 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-23  8:24 [PATCH] ARM: dts: hip04: NAND controller support Zhou Wang
2014-10-23  8:24 ` Zhou Wang
2014-10-23  9:32 ` Haojian Zhuang
2014-10-23  9:32   ` Haojian Zhuang
2014-10-24  6:24   ` Zhou Wang [this message]
2014-10-24  6:24     ` Zhou Wang
2014-10-24  6:54   ` Zhou Wang
2014-10-24  6:54     ` Zhou Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5449F08E.6040808@gmail.com \
    --to=wangzhou.bry@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.