mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR
@ 2014-12-08  9:59 Teresa Gámez
  2014-12-08  9:59 ` [PATCH 2/3] ARM: dts: am335x-phytec-phycore-som: " Teresa Gámez
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Teresa Gámez @ 2014-12-08  9:59 UTC (permalink / raw)
  To: barebox

Increase kernel partition to end of SPI NOR.

Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
---
 arch/arm/dts/am335x-phytec-phyflex.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/am335x-phytec-phyflex.dts b/arch/arm/dts/am335x-phytec-phyflex.dts
index c4481f5..0de7e87 100644
--- a/arch/arm/dts/am335x-phytec-phyflex.dts
+++ b/arch/arm/dts/am335x-phytec-phyflex.dts
@@ -187,7 +187,7 @@
 
 		partition@4 {
 			label = "kernel";
-			reg = <0xe0000 0x400000>;
+			reg = <0xe0000 0x0>;
 		};
 	};
 };
-- 
1.9.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/3] ARM: dts: am335x-phytec-phycore-som: Increase kernel partition on SPI NOR
  2014-12-08  9:59 [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Teresa Gámez
@ 2014-12-08  9:59 ` Teresa Gámez
  2014-12-08  9:59 ` [PATCH 3/3] ARM: phytec: am335x: Update enviroment comment Teresa Gámez
  2014-12-09  9:13 ` [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Sascha Hauer
  2 siblings, 0 replies; 6+ messages in thread
From: Teresa Gámez @ 2014-12-08  9:59 UTC (permalink / raw)
  To: barebox

Increase kernel partition to end of SPI NOR.

Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
---
 arch/arm/dts/am335x-phytec-phycore-som.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/am335x-phytec-phycore-som.dtsi b/arch/arm/dts/am335x-phytec-phycore-som.dtsi
index f1bcb8b..98945b2 100644
--- a/arch/arm/dts/am335x-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/am335x-phytec-phycore-som.dtsi
@@ -162,7 +162,7 @@
 
 		partition@4 {
 			label = "kernel";
-			reg = <0xe0000 0x400000>;
+			reg = <0xe0000 0x0>;
 		};
 	};
 };
-- 
1.9.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 3/3] ARM: phytec: am335x: Update enviroment comment
  2014-12-08  9:59 [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Teresa Gámez
  2014-12-08  9:59 ` [PATCH 2/3] ARM: dts: am335x-phytec-phycore-som: " Teresa Gámez
@ 2014-12-08  9:59 ` Teresa Gámez
  2014-12-09  9:13 ` [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Sascha Hauer
  2 siblings, 0 replies; 6+ messages in thread
From: Teresa Gámez @ 2014-12-08  9:59 UTC (permalink / raw)
  To: barebox

We do not have a root partition in spi flash any more.
Adapt comment.

Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
---
 .../arm/boards/phytec-phycore-am335x/defaultenv-phycore-am335x/boot/spi | 2 +-
 .../arm/boards/phytec-phyflex-am335x/defaultenv-phyflex-am335x/boot/spi | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boards/phytec-phycore-am335x/defaultenv-phycore-am335x/boot/spi b/arch/arm/boards/phytec-phycore-am335x/defaultenv-phycore-am335x/boot/spi
index c87299a..2147a6c 100644
--- a/arch/arm/boards/phytec-phycore-am335x/defaultenv-phycore-am335x/boot/spi
+++ b/arch/arm/boards/phytec-phycore-am335x/defaultenv-phycore-am335x/boot/spi
@@ -4,5 +4,5 @@ global.bootm.image="/dev/m25p0.kernel"
 
 bootargs-ip
 
-# Use rootfs form NAND for now as rootfs partition < 4MB
+# Use rootfs form NAND
 global.linux.bootargs.dyn.root="root=ubi0:root ubi.mtd=nand0.root,2048 rw rootfstype=ubifs"
diff --git a/arch/arm/boards/phytec-phyflex-am335x/defaultenv-phyflex-am335x/boot/spi b/arch/arm/boards/phytec-phyflex-am335x/defaultenv-phyflex-am335x/boot/spi
index c87299a..2147a6c 100644
--- a/arch/arm/boards/phytec-phyflex-am335x/defaultenv-phyflex-am335x/boot/spi
+++ b/arch/arm/boards/phytec-phyflex-am335x/defaultenv-phyflex-am335x/boot/spi
@@ -4,5 +4,5 @@ global.bootm.image="/dev/m25p0.kernel"
 
 bootargs-ip
 
-# Use rootfs form NAND for now as rootfs partition < 4MB
+# Use rootfs form NAND
 global.linux.bootargs.dyn.root="root=ubi0:root ubi.mtd=nand0.root,2048 rw rootfstype=ubifs"
-- 
1.9.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR
  2014-12-08  9:59 [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Teresa Gámez
  2014-12-08  9:59 ` [PATCH 2/3] ARM: dts: am335x-phytec-phycore-som: " Teresa Gámez
  2014-12-08  9:59 ` [PATCH 3/3] ARM: phytec: am335x: Update enviroment comment Teresa Gámez
@ 2014-12-09  9:13 ` Sascha Hauer
  2014-12-09 10:16   ` Teresa Gámez
  2 siblings, 1 reply; 6+ messages in thread
From: Sascha Hauer @ 2014-12-09  9:13 UTC (permalink / raw)
  To: Teresa Gámez; +Cc: barebox

Hi Teresa,

On Mon, Dec 08, 2014 at 10:59:49AM +0100, Teresa Gámez wrote:
> Increase kernel partition to end of SPI NOR.
> 
> Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
> ---
>  arch/arm/dts/am335x-phytec-phyflex.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/dts/am335x-phytec-phyflex.dts b/arch/arm/dts/am335x-phytec-phyflex.dts
> index c4481f5..0de7e87 100644
> --- a/arch/arm/dts/am335x-phytec-phyflex.dts
> +++ b/arch/arm/dts/am335x-phytec-phyflex.dts
> @@ -187,7 +187,7 @@
>  
>  		partition@4 {
>  			label = "kernel";
> -			reg = <0xe0000 0x400000>;
> +			reg = <0xe0000 0x0>;
>  		};

I wasn't aware barebox handles this case like you intend here. What
about the kernel? Won't you end up with a zero sized partition there?

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR
  2014-12-09  9:13 ` [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Sascha Hauer
@ 2014-12-09 10:16   ` Teresa Gámez
  2014-12-09 10:52     ` Sascha Hauer
  0 siblings, 1 reply; 6+ messages in thread
From: Teresa Gámez @ 2014-12-09 10:16 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox

Hello Sascha,

Am Dienstag, den 09.12.2014, 10:13 +0100 schrieb Sascha Hauer:
> Hi Teresa,
> 
> On Mon, Dec 08, 2014 at 10:59:49AM +0100, Teresa Gámez wrote:
> > Increase kernel partition to end of SPI NOR.
> > 
> > Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
> > ---
> >  arch/arm/dts/am335x-phytec-phyflex.dts | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm/dts/am335x-phytec-phyflex.dts b/arch/arm/dts/am335x-phytec-phyflex.dts
> > index c4481f5..0de7e87 100644
> > --- a/arch/arm/dts/am335x-phytec-phyflex.dts
> > +++ b/arch/arm/dts/am335x-phytec-phyflex.dts
> > @@ -187,7 +187,7 @@
> >  
> >  		partition@4 {
> >  			label = "kernel";
> > -			reg = <0xe0000 0x400000>;
> > +			reg = <0xe0000 0x0>;
> >  		};
> 
> I wasn't aware barebox handles this case like you intend here. What
> about the kernel? Won't you end up with a zero sized partition there?

have a look at include/linux/mtd/partitions.h in the linux kernel.

There is a macro defined 
#define MTDPART_SIZ_FULL        (0)

And the comment above says: 
size: the partition size; if defined as MTDPART_SIZ_FULL, the partition
will extend to the end of the master MTD device.

This works fine for kernel and barebox. 

Teresa

> 
> Sascha
> 



_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR
  2014-12-09 10:16   ` Teresa Gámez
@ 2014-12-09 10:52     ` Sascha Hauer
  0 siblings, 0 replies; 6+ messages in thread
From: Sascha Hauer @ 2014-12-09 10:52 UTC (permalink / raw)
  To: Teresa Gámez; +Cc: barebox

On Tue, Dec 09, 2014 at 11:16:47AM +0100, Teresa Gámez wrote:
> Hello Sascha,
> 
> Am Dienstag, den 09.12.2014, 10:13 +0100 schrieb Sascha Hauer:
> > Hi Teresa,
> > 
> > On Mon, Dec 08, 2014 at 10:59:49AM +0100, Teresa Gámez wrote:
> > > Increase kernel partition to end of SPI NOR.
> > > 
> > > Signed-off-by: Teresa Gámez <t.gamez@phytec.de>
> > > ---
> > >  arch/arm/dts/am335x-phytec-phyflex.dts | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/arch/arm/dts/am335x-phytec-phyflex.dts b/arch/arm/dts/am335x-phytec-phyflex.dts
> > > index c4481f5..0de7e87 100644
> > > --- a/arch/arm/dts/am335x-phytec-phyflex.dts
> > > +++ b/arch/arm/dts/am335x-phytec-phyflex.dts
> > > @@ -187,7 +187,7 @@
> > >  
> > >  		partition@4 {
> > >  			label = "kernel";
> > > -			reg = <0xe0000 0x400000>;
> > > +			reg = <0xe0000 0x0>;
> > >  		};
> > 
> > I wasn't aware barebox handles this case like you intend here. What
> > about the kernel? Won't you end up with a zero sized partition there?
> 
> have a look at include/linux/mtd/partitions.h in the linux kernel.
> 
> There is a macro defined 
> #define MTDPART_SIZ_FULL        (0)
> 
> And the comment above says: 
> size: the partition size; if defined as MTDPART_SIZ_FULL, the partition
> will extend to the end of the master MTD device.
> 
> This works fine for kernel and barebox.

Oh nice, didn't know that. In this case consider your patches applied.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-12-09 10:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-08  9:59 [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Teresa Gámez
2014-12-08  9:59 ` [PATCH 2/3] ARM: dts: am335x-phytec-phycore-som: " Teresa Gámez
2014-12-08  9:59 ` [PATCH 3/3] ARM: phytec: am335x: Update enviroment comment Teresa Gámez
2014-12-09  9:13 ` [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR Sascha Hauer
2014-12-09 10:16   ` Teresa Gámez
2014-12-09 10:52     ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox