[PATCH 1/2] Updated XCEP partition sizes

[PATCH 1/2] Updated XCEP partition sizes

[Matej Kenda]

Increased partition for Linux kernel from 1 MB to 2 MB for
monolithically built kernel 2.6.37 for Iskratel XCEP SBC to fit in.

Signed-off-by: Matej Kenda <matejken@gmail.com>

---

diff --git a/arch/arm/mach-pxa/xcep.c b/arch/arm/mach-pxa/xcep.c
index 3260ce7..f7b1052 100644
--- a/arch/arm/mach-pxa/xcep.c
+++ b/arch/arm/mach-pxa/xcep.c
@@ -55,20 +55,20 @@ static struct mtd_partition xcep_partitions[] = {
 	}, {
 		.name =		"Bootloader ENV",
 		.size =		0x00040000,
-		.offset =	0x00040000,
+		.offset =	MTDPART_OFS_APPEND,
 		.mask_flags =	MTD_WRITEABLE
 	}, {
 		.name =		"Kernel",
-		.size =		0x00100000,
-		.offset =	0x00080000,
+		.size =		0x00200000,
+		.offset =	MTDPART_OFS_APPEND,
 	}, {
 		.name =		"Rescue fs",
 		.size =		0x00280000,
-		.offset =	0x00180000,
+		.offset =	MTDPART_OFS_APPEND,
 	}, {
 		.name =		"Filesystem",
 		.size =		MTDPART_SIZ_FULL,
-		.offset =	0x00400000
+		.offset =	MTDPART_OFS_APPEND
 	}
 };

[PATCH 1/2] Updated XCEP partition sizes

[Matej Kenda]

Hi Eric,

I've sent two simple patches for Iskratel XCEP last week.

Can they be included in the kernel?

TIA,

Matej

On 19 January 2011 13:01, Matej Kenda <matejken@gmail.com> wrote:
> Increased partition for Linux kernel from 1 MB to 2 MB for
> monolithically built kernel 2.6.37 for Iskratel XCEP SBC to fit in.
>
> Signed-off-by: Matej Kenda <matejken@gmail.com>
>
> ---
>
> diff --git a/arch/arm/mach-pxa/xcep.c b/arch/arm/mach-pxa/xcep.c
> index 3260ce7..f7b1052 100644
> --- a/arch/arm/mach-pxa/xcep.c
> +++ b/arch/arm/mach-pxa/xcep.c
> @@ -55,20 +55,20 @@ static struct mtd_partition xcep_partitions[] = {
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Bootloader ENV",
> ? ? ? ? ? ? ? ?.size = ? ? ? ? 0x00040000,
> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00040000,
> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
> ? ? ? ? ? ? ? ?.mask_flags = ? MTD_WRITEABLE
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Kernel",
> - ? ? ? ? ? ? ? .size = ? ? ? ? 0x00100000,
> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00080000,
> + ? ? ? ? ? ? ? .size = ? ? ? ? 0x00200000,
> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Rescue fs",
> ? ? ? ? ? ? ? ?.size = ? ? ? ? 0x00280000,
> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00180000,
> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
> ? ? ? ?}, {
> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Filesystem",
> ? ? ? ? ? ? ? ?.size = ? ? ? ? MTDPART_SIZ_FULL,
> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00400000
> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND
> ? ? ? ?}
> ?};
>

[PATCH 1/2] Updated XCEP partition sizes

[Eric Miao]

On Tue, Jan 25, 2011 at 2:54 AM, Matej Kenda <matejken@gmail.com> wrote:
> Hi Eric,
>
> I've sent two simple patches for Iskratel XCEP last week.
>
> Can they be included in the kernel?

I'm a bit tentative on this. Patch 1 is basically OK, but what if
someone needs a even larger partition in the future?  And some work
is on-going with cleanup of those defconfigs, guess I'll hold a
while those changes to the defconfig until that's clear enough.

BTW - did you run Uwe's script to make sure the changes to defconfig
is the smallest one?

>
> TIA,
>
> Matej
>
> On 19 January 2011 13:01, Matej Kenda <matejken@gmail.com> wrote:
>> Increased partition for Linux kernel from 1 MB to 2 MB for
>> monolithically built kernel 2.6.37 for Iskratel XCEP SBC to fit in.
>>
>> Signed-off-by: Matej Kenda <matejken@gmail.com>
>>
>> ---
>>
>> diff --git a/arch/arm/mach-pxa/xcep.c b/arch/arm/mach-pxa/xcep.c
>> index 3260ce7..f7b1052 100644
>> --- a/arch/arm/mach-pxa/xcep.c
>> +++ b/arch/arm/mach-pxa/xcep.c
>> @@ -55,20 +55,20 @@ static struct mtd_partition xcep_partitions[] = {
>> ? ? ? ?}, {
>> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Bootloader ENV",
>> ? ? ? ? ? ? ? ?.size = ? ? ? ? 0x00040000,
>> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00040000,
>> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
>> ? ? ? ? ? ? ? ?.mask_flags = ? MTD_WRITEABLE
>> ? ? ? ?}, {
>> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Kernel",
>> - ? ? ? ? ? ? ? .size = ? ? ? ? 0x00100000,
>> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00080000,
>> + ? ? ? ? ? ? ? .size = ? ? ? ? 0x00200000,
>> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
>> ? ? ? ?}, {
>> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Rescue fs",
>> ? ? ? ? ? ? ? ?.size = ? ? ? ? 0x00280000,
>> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00180000,
>> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND,
>> ? ? ? ?}, {
>> ? ? ? ? ? ? ? ?.name = ? ? ? ? "Filesystem",
>> ? ? ? ? ? ? ? ?.size = ? ? ? ? MTDPART_SIZ_FULL,
>> - ? ? ? ? ? ? ? .offset = ? ? ? 0x00400000
>> + ? ? ? ? ? ? ? .offset = ? ? ? MTDPART_OFS_APPEND
>> ? ? ? ?}
>> ?};
>>
>

[PATCH 1/2] Updated XCEP partition sizes

[Uwe Kleine-König]

On Wed, Jan 26, 2011 at 05:19:55AM +0800, Eric Miao wrote:
> On Tue, Jan 25, 2011 at 2:54 AM, Matej Kenda <matejken@gmail.com> wrote:
> > Hi Eric,
> >
> > I've sent two simple patches for Iskratel XCEP last week.
> >
> > Can they be included in the kernel?
> 
> I'm a bit tentative on this. Patch 1 is basically OK, but what if
> someone needs a even larger partition in the future?  And some work
> is on-going with cleanup of those defconfigs, guess I'll hold a
> while those changes to the defconfig until that's clear enough.
> 
> BTW - did you run Uwe's script to make sure the changes to defconfig
> is the smallest one?
there is no "Uwe's script" anymore.  Just run

	make ARCH=arm savedefconfig

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[PATCH 1/2] Updated XCEP partition sizes

[Matej Kenda]

2011/1/25 Eric Miao <eric.y.miao@gmail.com>:
> On Tue, Jan 25, 2011 at 2:54 AM, Matej Kenda <matejken@gmail.com> wrote:
>> I've sent two simple patches for Iskratel XCEP last week.
>>
>> Can they be included in the kernel?
>
> I'm a bit tentative on this. Patch 1 is basically OK, but what if
> someone needs a even larger partition in the future?

This small embedded computer has relatively small Flash (32 MB).

1MB partition for the kernel was created initially for kernel 2.4 a
few years ago. The size of that kernel was around 800 kB.

Later on, the size increased to 900 kB with kernel 2.6.20 (still
monolithic kernel). 2.6.32 was still small enough to be squeezed in by
moving some driver out to modules.

However, with 2.6.36 this was not possible any more: I was unable to
make the kernel smaller that 1.1 MB and therefore the partition needed
to be increased.

Assuming the growth rate of the kernel from 800 kB to 1.1 in four
years, 2 MB should be enough for next 8 years or so.

> And some work
> is on-going with cleanup of those defconfigs, guess I'll hold a
> while those changes to the defconfig until that's clear enough.
>
> BTW - did you run Uwe's script to make sure the changes to defconfig
> is the smallest one?

defconfig file was created with "make ARCH=arm savedefconfig"

Regards,

Matej