[PATCH v2] mtd: ofpart: support partitions of 4 GB and larger

[Aaron Sierra]

> From: "David Woodhouse" <dwmw2 at infradead.org>
> Sent: Friday, April 5, 2013 6:23:22 AM
> Subject: Re: [PATCH v2] mtd: ofpart: support partitions of 4 GB and larger
> 
> On Thu, 2013-02-21 at 16:29 -0600, Aaron Sierra wrote:
> > +       /* an 8 GB partition */
> > +       partition at 0 {
> > +               label = "filesystem #1";
> > +               reg = <0x0 0x00000000 0x2 0x00000000>;
> > +       };
> > +
> > +       /* a 4 GB partition */
> > +       partition at 200000000 {
> > +               label = "filesystem #2";
> > +               reg = <0x2 0x00000000 0x1 0x00000000>;
> > +       };
> > +};
> 
> The comments here don't match the actual definitions. I fixed it to
> read
> as follows:
> 
> flash at 2 {
> 	#address-cells = <2>;
> 	#size-cells = <2>;
> 
> 	/* an 8 GB partition */
> 	partition at 0 {
> 		label = "filesystem #1";
> 		reg = <0x0 0x00000000 0x1 0xdcd65000>;
> 	};
> 
> 	/* a 4 GB partition */
> 	partition at 200000000 {
> 		label = "filesystem #2";
> 		reg = <0x1 0xdcd65000 0x0 0xee6b2800>;
> 	};
> };

David,
Blame this on my vague use of GB. I assume things would have been
crystal clear if I'd used GiB. Is this already committed with your
change? Your version doesn't seem like a realistic partitioning
scheme based on likely device erase block sizes.

Is is possible to just update the comments?

- 	/* an 8 GB partition */
+ 	/* an 8 GiB partition */
...
- 	/* a 4 GB partition */
+ 	/* a 4 GiB partition */

-Aaron