Patchwork Web site: A few grammatical fixes in Docs/General.

login
register
mail settings
Submitter Robert P. J. Day
Date Feb. 9, 2010, 11:34 a.m.
Message ID <alpine.LFD.2.00.1002090631540.6317@localhost>
Download mbox | patch
Permalink /patch/44904/
State New
Headers show

Comments

Robert P. J. Day - Feb. 9, 2010, 11:34 a.m.
Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

  in addition to a few picky fixes on that page, toward the bottom of
that page, one reads:

"This is wrong in most cases. The mtdblock In other words, please, do
                                          ^^^ ???
not use mtdblock unless you know exactly what you are doing.

  there seems to be something missing there, i'll leave it to people
higher up the food chain to decide what to do there.




rday
--

========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================
Artem Bityutskiy - Feb. 15, 2010, 8:27 p.m.
On Tue, 2010-02-09 at 06:34 -0500, Robert P. J. Day wrote:
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
Pushed, thanks.

> ---
> 
>   in addition to a few picky fixes on that page, toward the bottom of
> that page, one reads:
> 
> "This is wrong in most cases. The mtdblock In other words, please, do
>                                           ^^^ ???

Fixed, thanks a lot!

Patch

diff --git a/doc/general.xml b/doc/general.xml
index 8368b2e..14ba8ff 100644
--- a/doc/general.xml
+++ b/doc/general.xml
@@ -30,7 +30,7 @@  AG-AND, ECC'd NOR, etc.</p>
 <p>MTD subsystem does not deal with block devices like MMC, eMMC, SD,
 CompactFlash, etc. These devices are not raw flashes but they have a Flash
 Translation layer inside, which makes them look like block devices. These
-devices are subject of the Linux block subsystem, not MTD. Please, refer
+devices are the subject of the Linux block subsystem, not MTD. Please, refer to
 <a href="../faq/general.html#L_mtd_vs_hdd">this</a> FAQ section for a short
 list of the main differences between block and MTD devices. And the
 <a href="ubifs.html#L_raw_vs_ftl">raw flash vs. FTL devices</a> UBIFS section
@@ -46,15 +46,15 @@  discusses this in more details.</p>
 	them as bad or checking if an eraseblock is bad, getting information
 	about MTD devices, etc.</li>

-	<li>The <code>sysfs</code> interface is relatively and it provides full
-	information about each MTD device in the system. This interface is
+	<li>The <code>sysfs</code> interface is relatively newer and it provides
+	full information about each MTD device in the system. This interface is
 	easily extensible and developers are encouraged to use the
 	<code>sysfs</code> interface instead of older <code>ioctl</code> or
 	<code>/proc/mtd</code> interfaces, when possible.</li>

 	<li>The <code>/proc/mtd</code> proc file system file provides general
 	MTD information. This is a legacy interface and the sysfs interface
-	provide more information.</li>
+	provides more information.</li>
 </ul>

 <p>MTD subsystem supports bare <a href="nand.html">NAND</a> flashes with
@@ -77,7 +77,7 @@  starting from kernel version <code>2.6.29</code> and they live in the
 <code>drivers/mtd/tests</code> directory of the linux kernel source codes. You
 may compile the tests as kernel modules by enabling them in the kernel
 configuration menu by marking: "<b>Device Drivers</b>" ->
-"<b>Memory Technology Devices (MTD)</b>" -> "<b>MTD tests support</b>" (or
+"<b>Memory Technology Device (MTD) support</b>" -> "<b>MTD tests support</b>" (or
 the <code>MTD_TESTS</code> symbol in the <code>.config</code> file).</p>

 <p>If you have a pre-<code>2.6.29</code> kernel, you may find the tests