Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-29 Thread Scott Wood
On Tue, 2015-01-27 at 17:47 -0600, Peter Tyser wrote: Hi Scott, I waffled about removing it, but leaned towards leaving it in because: - I didn't want to change the existing U-Boot behavior for other users. A google of 'u-boot nand write' shows a lot of examples that don't

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-29 Thread Peter Tyser
On Thu, 2015-01-29 at 17:02 -0600, Scott Wood wrote: On Tue, 2015-01-27 at 17:47 -0600, Peter Tyser wrote: Hi Scott, I waffled about removing it, but leaned towards leaving it in because: - I didn't want to change the existing U-Boot behavior for other users. A google of

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-29 Thread Scott Wood
On Thu, 2015-01-29 at 17:37 -0600, Peter Tyser wrote: On Thu, 2015-01-29 at 17:02 -0600, Scott Wood wrote: On Tue, 2015-01-27 at 17:47 -0600, Peter Tyser wrote: Hi Scott, I waffled about removing it, but leaned towards leaving it in because: - I didn't want to change the

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-27 Thread Peter Tyser
Hi Scott, I waffled about removing it, but leaned towards leaving it in because: - I didn't want to change the existing U-Boot behavior for other users. A google of 'u-boot nand write' shows a lot of examples that don't include verification of writes, and they should if we remove

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-26 Thread Scott Wood
On Mon, 2015-01-26 at 17:17 -0600, Peter Tyser wrote: On Mon, 2015-01-26 at 16:33 -0600, Scott Wood wrote: On Mon, 2015-01-26 at 16:24 -0600, Peter Tyser wrote: The driver-specific verify_buf() function can be replaced with the standard read_page_raw() function to verify writes. This

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-26 Thread Scott Wood
On Mon, 2015-01-26 at 16:24 -0600, Peter Tyser wrote: The driver-specific verify_buf() function can be replaced with the standard read_page_raw() function to verify writes. This will allow verify_buf() to be removed from individual drivers. verify_buf() is no longer supported in mainline

Re: [U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-26 Thread Peter Tyser
On Mon, 2015-01-26 at 16:33 -0600, Scott Wood wrote: On Mon, 2015-01-26 at 16:24 -0600, Peter Tyser wrote: The driver-specific verify_buf() function can be replaced with the standard read_page_raw() function to verify writes. This will allow verify_buf() to be removed from individual

[U-Boot] [PATCH 1/5] nand: Use common read function instead of verify_buf()

2015-01-26 Thread Peter Tyser
The driver-specific verify_buf() function can be replaced with the standard read_page_raw() function to verify writes. This will allow verify_buf() to be removed from individual drivers. verify_buf() is no longer supported in mainline Linux, so it is a pain to continue supporting.