All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Ravi Babu <ravibabu@ti.com>
Cc: devicetree@vger.kernel.org, nsekhar@list.ti.com,
	Sekhar Nori <nsekhar@ti.com>,
	bcousson@baylibre.com, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] ARM: dts: dra7-evm: increase QSPI SPL partition size
Date: Fri, 13 Jan 2017 10:12:40 -0800	[thread overview]
Message-ID: <20170113181239.GC2630@atomide.com> (raw)
In-Reply-To: <1484311234-21978-2-git-send-email-ravibabu@ti.com>

* Ravi Babu <ravibabu@ti.com> [170113 04:41]:
> The SPL size for DRA74x platform has increased and
> is now more than 64KB. Increase QSPI SPL partition
> size to 256KB for DRA74x EVM.
> 
> QSPI partition numbering changes because of this.

And this will break the existing partitions potentially..
See what was discussed on the list few days ago in thread
"[PATCH 1/6] ARM: dts: am335x-phycore-som: Update NAND
partition table".

It's best to have these left empty or as they originally
were and let u-boot configure the partitions.

Regards,

Tony



> Signed-off-by: Ravi Babu <ravibabu@ti.com>
> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
> ---
>  arch/arm/boot/dts/dra7-evm.dts | 24 ++++++------------------
>  1 file changed, 6 insertions(+), 18 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
> index 132f2be..2784241 100644
> --- a/arch/arm/boot/dts/dra7-evm.dts
> +++ b/arch/arm/boot/dts/dra7-evm.dts
> @@ -681,41 +681,29 @@
>  		 */
>  		partition@0 {
>  			label = "QSPI.SPL";
> -			reg = <0x00000000 0x000010000>;
> +			reg = <0x00000000 0x000040000>;
>  		};
>  		partition@1 {
> -			label = "QSPI.SPL.backup1";
> -			reg = <0x00010000 0x00010000>;
> -		};
> -		partition@2 {
> -			label = "QSPI.SPL.backup2";
> -			reg = <0x00020000 0x00010000>;
> -		};
> -		partition@3 {
> -			label = "QSPI.SPL.backup3";
> -			reg = <0x00030000 0x00010000>;
> -		};
> -		partition@4 {
>  			label = "QSPI.u-boot";
>  			reg = <0x00040000 0x00100000>;
>  		};
> -		partition@5 {
> +		partition@2 {
>  			label = "QSPI.u-boot-spl-os";
>  			reg = <0x00140000 0x00080000>;
>  		};
> -		partition@6 {
> +		partition@3 {
>  			label = "QSPI.u-boot-env";
>  			reg = <0x001c0000 0x00010000>;
>  		};
> -		partition@7 {
> +		partition@4 {
>  			label = "QSPI.u-boot-env.backup1";
>  			reg = <0x001d0000 0x0010000>;
>  		};
> -		partition@8 {
> +		partition@5 {
>  			label = "QSPI.kernel";
>  			reg = <0x001e0000 0x0800000>;
>  		};
> -		partition@9 {
> +		partition@6 {
>  			label = "QSPI.file-system";
>  			reg = <0x009e0000 0x01620000>;
>  		};
> -- 
> 1.9.1
> 

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: dts: dra7-evm: increase QSPI SPL partition size
Date: Fri, 13 Jan 2017 10:12:40 -0800	[thread overview]
Message-ID: <20170113181239.GC2630@atomide.com> (raw)
In-Reply-To: <1484311234-21978-2-git-send-email-ravibabu@ti.com>

* Ravi Babu <ravibabu@ti.com> [170113 04:41]:
> The SPL size for DRA74x platform has increased and
> is now more than 64KB. Increase QSPI SPL partition
> size to 256KB for DRA74x EVM.
> 
> QSPI partition numbering changes because of this.

And this will break the existing partitions potentially..
See what was discussed on the list few days ago in thread
"[PATCH 1/6] ARM: dts: am335x-phycore-som: Update NAND
partition table".

It's best to have these left empty or as they originally
were and let u-boot configure the partitions.

Regards,

Tony



> Signed-off-by: Ravi Babu <ravibabu@ti.com>
> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
> ---
>  arch/arm/boot/dts/dra7-evm.dts | 24 ++++++------------------
>  1 file changed, 6 insertions(+), 18 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
> index 132f2be..2784241 100644
> --- a/arch/arm/boot/dts/dra7-evm.dts
> +++ b/arch/arm/boot/dts/dra7-evm.dts
> @@ -681,41 +681,29 @@
>  		 */
>  		partition at 0 {
>  			label = "QSPI.SPL";
> -			reg = <0x00000000 0x000010000>;
> +			reg = <0x00000000 0x000040000>;
>  		};
>  		partition at 1 {
> -			label = "QSPI.SPL.backup1";
> -			reg = <0x00010000 0x00010000>;
> -		};
> -		partition at 2 {
> -			label = "QSPI.SPL.backup2";
> -			reg = <0x00020000 0x00010000>;
> -		};
> -		partition at 3 {
> -			label = "QSPI.SPL.backup3";
> -			reg = <0x00030000 0x00010000>;
> -		};
> -		partition at 4 {
>  			label = "QSPI.u-boot";
>  			reg = <0x00040000 0x00100000>;
>  		};
> -		partition at 5 {
> +		partition at 2 {
>  			label = "QSPI.u-boot-spl-os";
>  			reg = <0x00140000 0x00080000>;
>  		};
> -		partition at 6 {
> +		partition at 3 {
>  			label = "QSPI.u-boot-env";
>  			reg = <0x001c0000 0x00010000>;
>  		};
> -		partition at 7 {
> +		partition at 4 {
>  			label = "QSPI.u-boot-env.backup1";
>  			reg = <0x001d0000 0x0010000>;
>  		};
> -		partition at 8 {
> +		partition at 5 {
>  			label = "QSPI.kernel";
>  			reg = <0x001e0000 0x0800000>;
>  		};
> -		partition at 9 {
> +		partition at 6 {
>  			label = "QSPI.file-system";
>  			reg = <0x009e0000 0x01620000>;
>  		};
> -- 
> 1.9.1
> 

  reply	other threads:[~2017-01-13 18:12 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-13 12:40 [PATCH 0/2] dra7x: dts update for increase in QSPI SPL parttion size Ravi Babu
2017-01-13 12:40 ` Ravi Babu
     [not found] ` <1484311234-21978-1-git-send-email-ravibabu-l0cyMroinI0@public.gmane.org>
2017-01-13 12:40   ` [PATCH 1/2] ARM: dts: dra7-evm: increase QSPI SPL partition size Ravi Babu
2017-01-13 12:40     ` Ravi Babu
2017-01-13 18:12     ` Tony Lindgren [this message]
2017-01-13 18:12       ` Tony Lindgren
2017-01-17  8:14       ` B, Ravi
2017-01-17  8:14         ` B, Ravi
2017-01-17 23:27         ` Tony Lindgren
2017-01-17 23:27           ` Tony Lindgren
2017-01-18 11:58           ` Sekhar Nori
2017-01-18 11:58             ` Sekhar Nori
2017-01-18 17:39             ` Tony Lindgren
2017-01-18 17:39               ` Tony Lindgren
2017-01-13 12:40   ` [PATCH 2/2] ARM: dts: dra72x-evm: " Ravi Babu
2017-01-13 12:40     ` Ravi Babu

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=20170113181239.GC2630@atomide.com \
    --to=tony@atomide.com \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=nsekhar@list.ti.com \
    --cc=nsekhar@ti.com \
    --cc=ravibabu@ti.com \
    /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.