[PATCH] ARM: pcm038: Fix partition size definitions

[PATCH] ARM: pcm038: Fix partition size definitions

[Alexander Shiyan]

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 arch/arm/boards/pcm038/pcm038.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/arch/arm/boards/pcm038/pcm038.c b/arch/arm/boards/pcm038/pcm038.c
index 372aea5..49677c2 100644
--- a/arch/arm/boards/pcm038/pcm038.c
+++ b/arch/arm/boards/pcm038/pcm038.c
@@ -332,22 +332,20 @@ static int pcm038_devices_init(void)
 
 	switch (bootsource_get()) {
 	case BOOTSOURCE_NAND:
-		devfs_add_partition("nand0", 0x00000, 0x80000,
-					DEVFS_PARTITION_FIXED, "self_raw");
+		devfs_add_partition("nand0", 0, SZ_512K,
+				    DEVFS_PARTITION_FIXED, "self_raw");
 		dev_add_bb_dev("self_raw", "self0");
-
-		devfs_add_partition("nand0", 0x80000, 0x100000,
-					DEVFS_PARTITION_FIXED, "env_raw");
+		devfs_add_partition("nand0", SZ_512K, SZ_128K,
+				    DEVFS_PARTITION_FIXED, "env_raw");
 		dev_add_bb_dev("env_raw", "env0");
 		envdev = "NAND";
 		break;
 	default:
-		devfs_add_partition("nor0", 0x00000, 0x80000,
-					DEVFS_PARTITION_FIXED, "self0");
-		devfs_add_partition("nor0", 0x80000, 0x100000,
-					DEVFS_PARTITION_FIXED, "env0");
+		devfs_add_partition("nor0", 0, SZ_512K,
+				    DEVFS_PARTITION_FIXED, "self0");
+		devfs_add_partition("nor0", SZ_512K, SZ_128K,
+				    DEVFS_PARTITION_FIXED, "env0");
 		protect_file("/dev/env0", 1);
-
 		envdev = "NOR";
 	}
 
-- 
1.8.1.5


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

Re: [PATCH] ARM: pcm038: Fix partition size definitions

[Sascha Hauer]

On Sun, Jun 09, 2013 at 12:40:08PM +0400, Alexander Shiyan wrote:
> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>

Applied, though I really had liked some hint what was wrong with the
partition sizes.

Sascha

> ---
>  arch/arm/boards/pcm038/pcm038.c | 18 ++++++++----------
>  1 file changed, 8 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/boards/pcm038/pcm038.c b/arch/arm/boards/pcm038/pcm038.c
> index 372aea5..49677c2 100644
> --- a/arch/arm/boards/pcm038/pcm038.c
> +++ b/arch/arm/boards/pcm038/pcm038.c
> @@ -332,22 +332,20 @@ static int pcm038_devices_init(void)
>  
>  	switch (bootsource_get()) {
>  	case BOOTSOURCE_NAND:
> -		devfs_add_partition("nand0", 0x00000, 0x80000,
> -					DEVFS_PARTITION_FIXED, "self_raw");
> +		devfs_add_partition("nand0", 0, SZ_512K,
> +				    DEVFS_PARTITION_FIXED, "self_raw");
>  		dev_add_bb_dev("self_raw", "self0");
> -
> -		devfs_add_partition("nand0", 0x80000, 0x100000,
> -					DEVFS_PARTITION_FIXED, "env_raw");
> +		devfs_add_partition("nand0", SZ_512K, SZ_128K,
> +				    DEVFS_PARTITION_FIXED, "env_raw");
>  		dev_add_bb_dev("env_raw", "env0");
>  		envdev = "NAND";
>  		break;
>  	default:
> -		devfs_add_partition("nor0", 0x00000, 0x80000,
> -					DEVFS_PARTITION_FIXED, "self0");
> -		devfs_add_partition("nor0", 0x80000, 0x100000,
> -					DEVFS_PARTITION_FIXED, "env0");
> +		devfs_add_partition("nor0", 0, SZ_512K,
> +				    DEVFS_PARTITION_FIXED, "self0");
> +		devfs_add_partition("nor0", SZ_512K, SZ_128K,
> +				    DEVFS_PARTITION_FIXED, "env0");
>  		protect_file("/dev/env0", 1);
> -
>  		envdev = "NOR";
>  	}
>  
> -- 
> 1.8.1.5
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

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