mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Roland Hieber <r.hieber@pengutronix.de>
To: "Ulrich Ölmann" <u.oelmann@pengutronix.de>
Subject: Re: [PATCH 2/3] Documentation: state: fix typos
Date: Tue, 10 Jul 2018 12:35:39 +0200	[thread overview]
Message-ID: <20180710103539.irp3zg7yys25e3ew@pengutronix.de> (raw)
In-Reply-To: <20180710095003.1256-2-u.oelmann@pengutronix.de>

In general: better than before, so:

Tested-by: Roland Hieber <r.hieber@pengutronix.de>

Some additional nitpicks:

On Tue, Jul 10, 2018 at 11:50:02AM +0200, Ulrich Ölmann wrote:
> Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
> ---
>  Documentation/user/state.rst | 62 ++++++++++++++++++------------------
>  1 file changed, 31 insertions(+), 31 deletions(-)
> 
> diff --git a/Documentation/user/state.rst b/Documentation/user/state.rst
> index 97e45d43b7cb..843705bdb6f3 100644
> --- a/Documentation/user/state.rst
> +++ b/Documentation/user/state.rst
> @@ -37,9 +37,9 @@ section.
>  Backends (e.g. Supported Memory Types)
>  --------------------------------------
>  
> -Some non-volatile memory is need for storing a *state* variable set:
> +Some non-volatile memory is needed for storing a *state* variable set:
>  
> -- Disk like devices: SD, (e)MMC, ATA
> +- disk like devices: SD, (e)MMC, ATA

Maybe add a dash ("disk-like") to ease sentence parsing.

>  - all kinds of NAND and NOR flash memories (mtd)
>  - MRAM
>  - EEPROM
> @@ -91,9 +91,9 @@ the binary data blob with the following content and layout:
>  
>  - 'magic value' is an unsigned value with native endianness, refer to
>    :ref:`'magic' property <barebox,state_magic>` about its value.
> -- 'byte count' is an unsigned value with native endianness
> -- 'binary data blob CRC32' is an unsigned value with native endianness
> -- 'header CRC32' is an unsigned value with native endianness
> +- 'byte count' is an unsigned value with native endianness.
> +- 'binary data blob CRC32' is an unsigned value with native endianness.
> +- 'header CRC32' is an unsigned value with native endianness.
>  
>  .. note:: the 32-bit CRC calculation uses the polynomial:
>  
> @@ -140,7 +140,7 @@ Currently two backend storage type implementations do exist, ``circular`` and
>  ``direct``.
>  
>  The state framework can select the correct backend storage type depending on the
> -backend medium. Media requiring erase operations (NAND, NOR flash) defaults to
> +backend medium. Media requiring erase operations (NAND, NOR flash) default to
>  the ``circular`` backend storage type automatically. In contrast EEPROMs and
>  RAMs are candidates for the ``direct`` backend storage type.
>  
> @@ -151,9 +151,9 @@ This kind of backend storage type is intended to be used with persistent RAMs or
>  EEPROMs.
>  These media are characterized by:
>  
> -- memory cells can be simply written at any time (no previous erase required)
> -- memory cells can be written as often as required (unlimted or very high endurance)
> -- can be written on a byte-by-byte manner
> +- memory cells can be simply written at any time (no previous erase required).
> +- memory cells can be written as often as required (unlimted or very high endurance).
> +- memory cells can be written on a byte-by-byte manner.
>  
>  Example: MRAM with 64 bytes at device's offset 0:
>  
> @@ -183,10 +183,10 @@ This kind of backend storage type is intended to be used with regular flash memo
>  
>  Flash memories are characterized by:
>  
> -- only erased memory cells can be written with new data
> -- written data cannot be written twice (at least not for modern flash devices)
> -- erase can happen on eraseblock sizes only (detectable, physical value)
> -- an eraseblock only supports a limited number of write-erase-cycles (as low as a few thousand cycles)
> +- only erased memory cells can be written with new data.
> +- written data cannot be written twice (at least not for modern flash devices).
> +- erase can happen on eraseblock sizes only (detectable, physical value).
> +- an eraseblock only supports a limited number of write-erase-cycles (as low as a few thousand cycles).
>  
>  The purpose of the ``circular`` backend storage type is to save erase cycles
>  which may wear out the flash's eraseblocks. This type instead incrementally fills
> @@ -196,7 +196,7 @@ eraseblock again.
>  
>  **NOR type flash memory is additionally characterized by**
>  
> -- can be written on a byte-by-byte manner
> +- memory cells can be written on a byte-by-byte manner.
>  
>  .. _state_framework,nor:
>  
> @@ -258,10 +258,10 @@ the eraseblock again. This reduces the need for a flash memory erase by factors.
>  
>  **NAND type flash memory is additionally characterized by**
>  
> -- organized in pages (size is a detectable, physical value)
> -- writes can only happen in multiples of the page size (which much less than the eraseblock size)
> +- it is organized in pages (size is a detectable, physical value).
> +- writes can only happen in multiples of the page size (which much less than the eraseblock size).
>  - partially writing a page can be limited in count or be entirely forbidden (in
> -  the case of *MLC* NANDs)
> +  the case of *MLC* NANDs).
>  
>  Example: NAND type flash memory with 128 kiB eraseblock size and 2 kiB page
>  size and a 2 kiB write size
> @@ -327,7 +327,7 @@ Redundant *state* Variable Set Copies
>  To avoid data loss when changing the *state* variable set, more than one
>  *state* variable set copy can be stored into the backend. Whenever the *state*
>  variable set changes, only one *state* variable set copy gets changed at a time.
> -In the case of an interruption and/or power loss resulting into an incomplete
> +In the case of an interruption and/or power loss resulting in an incomplete
>  write to the backend, the system can fall back to a different *state* variable
>  set copy (previous *state* variable set).
>  
> @@ -379,9 +379,9 @@ side-by-side location of the *state* variable set copies.
>  
>  *<X>* defines the stride size, *C#1*, *C#2* the *state* variable set copies.
>  
> -Since these kinds of MTD devices are partitioned, its a good practice to always
> -reserve multiple eraseblocks for the barebox's *state* feature. Keep in mind:
> -Even NOR type flash memories can be worn out.
> +Since these kinds of MTD devices are partitioned, it's a good practice to always
> +reserve multiple eraseblocks for the barebox' *state* feature. Keep in mind:

				    ^-- I would also drop the "the"
				    here, so it becomes "for barebox'
				    state feature".

> +even NOR type flash memories can be worn out.
>  
>  **NAND type flash memory**
>  
> @@ -397,8 +397,8 @@ eraseblocks and this size is automatically detected at run-time.
>      |<----------- eraseblock ---------->|<----------- eraseblock ---------->|<-
>      |<-------- redundant area --------->|<-------- redundant area --------->|<-
>  
> -Since these kinds of MTD devices are partitioned, its a good practice to always
> -reserve multiple eraseblocks for the barebox's *state* feature. Keep in mind:
> +Since these kinds of MTD devices are partitioned, it's a good practice to always
> +reserve multiple eraseblocks for the barebox' *state* feature. Keep in mind:

                                    ^-- and here too.

 - Roland

>  NAND type flash memories can be worn out, factory bad blocks can exist from the
>  beginning.
>  
> @@ -409,7 +409,7 @@ NAND type flash memory can have factory bad eraseblocks and more bad
>  eraseblocks can appear over the life time of the memory. They are detected by
>  the MTD layer, marked as bad and never used again.
>  
> -.. important:: If NAND type flash memory should be used as a backend, at least
> +.. important:: if NAND type flash memory should be used as a backend, at least
>     three eraseblocks are used to keep three redundant copies of the *state*
>     variable set. You should add some spare eraseblocks to the backend
>     partition by increasing the partition's size to a suitable value to handle
> @@ -418,7 +418,7 @@ the MTD layer, marked as bad and never used again.
>  Examples
>  --------
>  
> -The following examples intends to show how to setup and interconnect all
> +The following examples intend to show how to setup and interconnect all
>  required components for various non-volatile memories.
>  
>  All examples use just one *state* variable of type *uint8* named ``variable``
> @@ -456,7 +456,7 @@ a partition at a specific offset to be used as the backend for the
>  		};
>  	};
>  
> -With this 'backend' definition its possible to define the *state* variable set
> +With this 'backend' definition it's possible to define the *state* variable set
>  content, its backend-type and *state* variable set layout.
>  
>  .. code-block:: text
> @@ -506,7 +506,7 @@ a partition at a specific offset inside it to be used as the backend for the
>  		};
>  	};
>  
> -With this 'backend' definition its possible to define the *state* variable set
> +With this 'backend' definition it's possible to define the *state* variable set
>  content, its backend-type and *state* variable layout.
>  
>  .. code-block:: text
> @@ -537,7 +537,7 @@ SD/eMMC and ATA
>  The following devicetree node entry defines some kind of SD/eMMC memory and
>  a partition at a specific offset inside it to be used as the backend for the
>  *state* variable set. Note that currently there is no support for on-disk
> -partition tables. Instead, a ofpart partition description must be used. You
> +partition tables. Instead, an ofpart partition description must be used. You
>  have to make sure that this partition does not conflict with any other partition
>  in the partition table.
>  
> @@ -548,7 +548,7 @@ in the partition table.
>  		reg = <0x100000 0x20000>;
>  	};
>  
> -With this 'backend' definition its possible to define the *state* variable set
> +With this 'backend' definition it's possible to define the *state* variable set
>  content, its backend-type and *state* variable layout.
>  
>  .. code-block:: text
> @@ -595,7 +595,7 @@ a partition at a specific offset inside it to be used as the backend for the
>  		};
>  	};
>  
> -With this 'backend' definition its possible to define the *state* variable set
> +With this 'backend' definition it's possible to define the *state* variable set
>  content, its backend-type and *state* variable layout.
>  
>  .. code-block:: text
> @@ -649,7 +649,7 @@ within the EEPROM.
>  	};
>  };
>  
> -With this 'backend' definition its possible to define the *state* variable set
> +With this 'backend' definition it's possible to define the *state* variable set
>  content, its backend-type and *state* variable layout.
>  
>  .. code-block:: text
> -- 
> 2.18.0
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

-- 
Roland Hieber                     | r.hieber@pengutronix.de     |
Pengutronix e.K.                  | https://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |

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

  reply	other threads:[~2018-07-10 10:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-10  9:50 [PATCH 1/3] Documentation: state: harmonize capitalization in headings Ulrich Ölmann
2018-07-10  9:50 ` [PATCH 2/3] Documentation: state: fix typos Ulrich Ölmann
2018-07-10 10:35   ` Roland Hieber [this message]
2018-07-10  9:50 ` [PATCH 3/3] Documentation: state: add unit name to state variable in DT Ulrich Ölmann
2018-07-10 10:36   ` Roland Hieber
2018-07-13  6:19 ` [PATCH 1/3] Documentation: state: harmonize capitalization in headings Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180710103539.irp3zg7yys25e3ew@pengutronix.de \
    --to=r.hieber@pengutronix.de \
    --cc=u.oelmann@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox