> Instead of manually calculating the partition sizes, it's
> better to use defines already set in include/asm-arm/sizes.h
I don't know that I agree with this for a flash device. Knowing something
about the block geometry of the device with out having to dig up the data sheet
is pretty useful.
I'm not vouching for the original code's actual just what the intent might be.
You should have the 1st 4 blocks for a redundant x-loader, the next blocks for
a size sufficient to hold a u-boot, then generally 1 block for u-boot
environment, and then the definition from there as to how to have the kernel
and file system has a few ways to go.
Really using the boot command line is best as compared to a static array here.
> Compile test only.
>
It doesn't look like you did your math correctly. How can these 2 partitions
have the same .size in your patch given their original definition?
> .mask_flags = MTD_WRITEABLE /* force read-only */
> },
> {
> .name = "U-Boot",
> .offset = MTDPART_OFS_APPEND,
> - .size = 4*(64*2048),
> + .size = SZ_128K,
> .mask_flags = MTD_WRITEABLE /* force read-only */
> },
> {
> .name = "U-Boot Environment",
> .offset = MTDPART_OFS_APPEND,
> - .size = 2*(64*2048),
> + .size = SZ_128K,
> },
Regards,
Richard W.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
