[U-Boot] [PATCH] M28: Document that units has to be set to sectors on SD bootcard

[U-Boot] [PATCH] M28: Document that units has to be set to sectors on SD bootcard

[Marek Vasut]

Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
---
 doc/README.m28 |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/doc/README.m28 b/doc/README.m28
index b749ce0..242230b 100644
--- a/doc/README.m28
+++ b/doc/README.m28
@@ -116,6 +116,12 @@ must have the following parameters:
 For example in Linux fdisk, the sequence for a clear card is the following:
 
 	* o ..................... create a clear partition table
+	* p ..................... here you need to verify that units is set to
+	                          sectors. In the output, earch for a line
+				  containing:
+				      Units = sectors of 1 * 512 = 512 bytes
+				  If no such line is present, press "u" to
+				  switch units to sectors.
 	* n ..................... create new partition
 		* p ............. primary partition
 		* 1 ............. first partition
-- 
1.7.7.1

[U-Boot] [PATCH] M28: Document that units has to be set to sectors on SD bootcard

[Marek Vasut]

> On Thu, Nov 24, 2011 at 2:57 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> >  For example in Linux fdisk, the sequence for a clear card is the
> > following:
> > 
> >        * o ..................... create a clear partition table
> > +       * p ..................... here you need to verify that units is
> > set to +                                 sectors. In the output, earch
> > for a line
> 
> s/earch/search

Thanks!
> 
> > +                                 containing:
> > +                                     Units = sectors of 1 * 512 = 512
> > bytes +                                 If no such line is present,
> > press "u" to +                                 switch units to sectors.
> 
> Or maybe an easier way is to call:
> sudo fdisk /dev/mmcblk0 -u

-u=sectors is even better. Of course, this doesn't work on *BSD. Other 
OSes/game-loaders/etc. I don't think we even consider ;-)
> 
> Regards,
> 
> Fabio Estevam

[U-Boot] [PATCH V2] M28: Document that units has to be set to sectors on SD bootcard

[Marek Vasut]

Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
---
 doc/README.m28 |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

V2: Make this all simpler for user to follow

diff --git a/doc/README.m28 b/doc/README.m28
index b749ce0..e4929ad 100644
--- a/doc/README.m28
+++ b/doc/README.m28
@@ -113,7 +113,8 @@ must have the following parameters:
 	* Partition size ........ at least 1024 kb
 	* Partition type ........ 0x53 (sometimes "OnTrack DM6 Aux3")
 
-For example in Linux fdisk, the sequence for a clear card is the following:
+For example in Linux fdisk, the sequence for a clear card is the follows. Be
+sure to run fdisk with the option "-u=sectors" to set units to sectors:
 
 	* o ..................... create a clear partition table
 	* n ..................... create new partition
-- 
1.7.7.1

[U-Boot] [PATCH V2] M28: Document that units has to be set to sectors on SD bootcard

[Fabio Estevam]

On Thu, Nov 24, 2011 at 3:39 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> ---
> ?doc/README.m28 | ? ?3 ++-
> ?1 files changed, 2 insertions(+), 1 deletions(-)
>
> V2: Make this all simpler for user to follow
>
> diff --git a/doc/README.m28 b/doc/README.m28
> index b749ce0..e4929ad 100644
> --- a/doc/README.m28
> +++ b/doc/README.m28
> @@ -113,7 +113,8 @@ must have the following parameters:
> ? ? ? ?* Partition size ........ at least 1024 kb
> ? ? ? ?* Partition type ........ 0x53 (sometimes "OnTrack DM6 Aux3")
>
> -For example in Linux fdisk, the sequence for a clear card is the following:
> +For example in Linux fdisk, the sequence for a clear card is the follows. Be

s/follows/following

Other than that:

Acked-by: Fabio Estevam <fabio.estevam@freescale.com>

[U-Boot] [PATCH V3] M28: Document that units has to be set to sectors on SD bootcard

[Marek Vasut]

Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
---
 doc/README.m28 |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

V2: Make this all simpler for user to follow
V3: Fix grammar

diff --git a/doc/README.m28 b/doc/README.m28
index b749ce0..548982f 100644
--- a/doc/README.m28
+++ b/doc/README.m28
@@ -113,7 +113,8 @@ must have the following parameters:
 	* Partition size ........ at least 1024 kb
 	* Partition type ........ 0x53 (sometimes "OnTrack DM6 Aux3")
 
-For example in Linux fdisk, the sequence for a clear card is the following:
+For example in Linux fdisk, the sequence for a clear card follows. Be sure to
+run fdisk with the option "-u=sectors" to set units to sectors:
 
 	* o ..................... create a clear partition table
 	* n ..................... create new partition
-- 
1.7.7.1

[U-Boot] [PATCH] M28: Document that units has to be set to sectors on SD bootcard

[Wolfgang Denk]

Dear Marek Vasut,

In message <1322153845-19369-1-git-send-email-marek.vasut@gmail.com> you wrote:
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> ---
>  doc/README.m28 |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/doc/README.m28 b/doc/README.m28
> index b749ce0..242230b 100644
> --- a/doc/README.m28
> +++ b/doc/README.m28
> @@ -116,6 +116,12 @@ must have the following parameters:
>  For example in Linux fdisk, the sequence for a clear card is the following:
>  
>  	* o ..................... create a clear partition table
> +	* p ..................... here you need to verify that units is set to
> +	                          sectors. In the output, earch for a line
> +				  containing:
> +				      Units = sectors of 1 * 512 = 512 bytes
> +				  If no such line is present, press "u" to
> +				  switch units to sectors.
>  	* n ..................... create new partition
>  		* p ............. primary partition
>  		* 1 ............. first partition

Would it not make sense to actually make sure this is the case, like
by passing the "-u" option to fdisk ?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A person with one watch knows what time it  is;  a  person  with  two
watches is never sure.                                       Proverb

[U-Boot] [PATCH] M28: Document that units has to be set to sectors on SD bootcard

[Marek Vasut]

> Dear Marek Vasut,
> 
> In message <1322153845-19369-1-git-send-email-marek.vasut@gmail.com> you 
wrote:
> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> > Cc: Stefano Babic <sbabic@denx.de>
> > Cc: Wolfgang Denk <wd@denx.de>
> > ---
> > 
> >  doc/README.m28 |    6 ++++++
> >  1 files changed, 6 insertions(+), 0 deletions(-)
> > 
> > diff --git a/doc/README.m28 b/doc/README.m28
> > index b749ce0..242230b 100644
> > --- a/doc/README.m28
> > +++ b/doc/README.m28
> > 
> > @@ -116,6 +116,12 @@ must have the following parameters:
> >  For example in Linux fdisk, the sequence for a clear card is the following:
> >  	* o ..................... create a clear partition table
> > 
> > +	* p ..................... here you need to verify that units is set to
> > +	                          sectors. In the output, earch for a line
> > +				  containing:
> > +				      Units = sectors of 1 * 512 = 512 bytes
> > +				  If no such line is present, press "u" to
> > +				  switch units to sectors.
> > 
> >  	* n ..................... create new partition
> >  	
> >  		* p ............. primary partition
> >  		* 1 ............. first partition
> 
> Would it not make sense to actually make sure this is the case, like
> by passing the "-u" option to fdisk ?

Definitelly, please check v3 of the patch. Fabio was so kind to check my grammar 
issues etc ;-)

M