MTD: Correct misspelled Kconfig var CONFIG_MTD_DATAFLASH_VERIFY_WRITE

Submitted by Robert P. J. Day on Sept. 11, 2009, 9:44 a.m.

Details

Message ID alpine.LFD.2.00.0909110543410.11692@localhost
State New, archived
Headers show

Commit Message

Robert P. J. Day Sept. 11, 2009, 9:44 a.m.
Make the source file correction CONFIG_MTD_DATAFLASH_VERIFY_WRITE ->
CONFIG_MTD_DATAFLASH_WRITE_VERIFY to match the Kconfig variable in
drivers/mtd/devices/Kconfig.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

  whoops, sorry, previous commit msg didn't show the actual
correction, this one does.





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

        Linux Consulting, Training and Annoying Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

Comments

Peter Korsgaard Sept. 11, 2009, 6:50 p.m.
>>>>> "Robert" == Robert P J Day <rpjday@crashcourse.ca> writes:

 Robert> Make the source file correction CONFIG_MTD_DATAFLASH_VERIFY_WRITE ->
 Robert> CONFIG_MTD_DATAFLASH_WRITE_VERIFY to match the Kconfig variable in
 Robert> drivers/mtd/devices/Kconfig.

 Robert> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

Acked-by: Peter Korsgaard <jacmet@sunsite.dk>
Artem Bityutskiy Sept. 14, 2009, 3:07 p.m.
On Fri, 2009-09-11 at 05:44 -0400, Robert P. J. Day wrote:
> Make the source file correction CONFIG_MTD_DATAFLASH_VERIFY_WRITE ->
> CONFIG_MTD_DATAFLASH_WRITE_VERIFY to match the Kconfig variable in
> drivers/mtd/devices/Kconfig.
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

Actually, a patch you sent earlier is already in the mtd-2.6.git tree:
http://git.infradead.org/mtd-2.6.git/commit/05dd180709fca14fbae617c0dab1bed56be334fc
Robert P. J. Day Sept. 14, 2009, 5:07 p.m.
On Mon, 14 Sep 2009, Artem Bityutskiy wrote:

> On Fri, 2009-09-11 at 05:44 -0400, Robert P. J. Day wrote:
> > Make the source file correction CONFIG_MTD_DATAFLASH_VERIFY_WRITE ->
> > CONFIG_MTD_DATAFLASH_WRITE_VERIFY to match the Kconfig variable in
> > drivers/mtd/devices/Kconfig.
> >
> > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
>
> Actually, a patch you sent earlier is already in the mtd-2.6.git tree:
> http://git.infradead.org/mtd-2.6.git/commit/05dd180709fca14fbae617c0dab1bed56be334fc

  i submitted a different one based on the recommendation of what
should be changed for those two bits of text to match up.  either one
of those two patches will work just fine, they just resolve the issue
in opposite ways.  whatever people on this list prefer is fine with
me.

rday
--

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

        Linux Consulting, Training and Annoying Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

Patch hide | download patch | download mbox

diff --git a/drivers/mtd/devices/mtd_dataflash.c b/drivers/mtd/devices/mtd_dataflash.c
index 43976aa..70a7369 100644
--- a/drivers/mtd/devices/mtd_dataflash.c
+++ b/drivers/mtd/devices/mtd_dataflash.c
@@ -401,7 +401,7 @@  static int dataflash_write(struct mtd_info *mtd, loff_t to, size_t len,
 		(void) dataflash_waitready(priv->spi);


-#ifdef CONFIG_MTD_DATAFLASH_VERIFY_WRITE
+#ifdef CONFIG_MTD_DATAFLASH_WRITE_VERIFY

 		/* (3) Compare to Buffer1 */
 		addr = pageaddr << priv->page_offset;
@@ -430,7 +430,7 @@  static int dataflash_write(struct mtd_info *mtd, loff_t to, size_t len,
 		} else
 			status = 0;

-#endif	/* CONFIG_MTD_DATAFLASH_VERIFY_WRITE */
+#endif	/* CONFIG_MTD_DATAFLASH_WRITE_VERIFY */

 		remaining = remaining - writelen;
 		pageaddr++;