Commit d5b813e4 authored by Michael Walle's avatar Michael Walle Committed by Vignesh Raghavendra
Browse files

mtd: spi-nor: otp: use more consistent wording 



Use the wording as used in the datasheet to describe the access methods
of the security registers (aka OTP storage). This will also match the
function names.

Signed-off-by: default avatarMichael Walle <michael@walle.cc>
Signed-off-by: default avatarVignesh Raghavendra <vigneshr@ti.com>
Reviewed-by: default avatarPratyush Yadav <p.yadav@ti.com>
Reviewed-by: default avatarTudor Ambarus <tudor.ambarus@microchip.com>
parent b97b1a76

drivers/mtd/spi-nor/otp.c

+19 −8
Original line number Diff line number Diff line
@@ -15,14 +15,21 @@ 
#define spi_nor_otp_n_regions(nor) ((nor)->params->otp.org->n_regions)



/**

 * spi_nor_otp_read_secr() - read OTP data

 * spi_nor_otp_read_secr() - read security register

 * @nor:	pointer to 'struct spi_nor'

 * @addr:       offset to read from

 * @len:        number of bytes to read

 * @buf:        pointer to dst buffer

 *

 * Read OTP data from one region by using the SPINOR_OP_RSECR commands. This

 * method is used on GigaDevice and Winbond flashes.

 * Read a security register by using the SPINOR_OP_RSECR commands.

 *

 * In Winbond/GigaDevice datasheets the term "security register" stands for

 * an one-time-programmable memory area, consisting of multiple bytes (usually

 * 256). Thus one "security register" maps to one OTP region.

 *

 * This method is used on GigaDevice and Winbond flashes.

 *

 * Please note, the read must not span multiple registers.

 *

 * Return: number of bytes read successfully, -errno otherwise

 */
@@ -56,16 +63,20 @@ int spi_nor_otp_read_secr(struct spi_nor *nor, loff_t addr, size_t len, u8 *buf) 
}



/**

 * spi_nor_otp_write_secr() - write OTP data

 * spi_nor_otp_write_secr() - write security register

 * @nor:        pointer to 'struct spi_nor'

 * @addr:       offset to write to

 * @len:        number of bytes to write

 * @buf:        pointer to src buffer

 *

 * Write OTP data to one region by using the SPINOR_OP_PSECR commands. This

 * method is used on GigaDevice and Winbond flashes.

 * Write a security register by using the SPINOR_OP_PSECR commands.

 *

 * For more information on the term "security register", see the documentation

 * of spi_nor_otp_read_secr().

 *

 * This method is used on GigaDevice and Winbond flashes.

 *

 * Please note, the write must not span multiple OTP regions.

 * Please note, the write must not span multiple registers.

 *

 * Return: number of bytes written successfully, -errno otherwise

 */
@@ -88,7 +99,7 @@ int spi_nor_otp_write_secr(struct spi_nor *nor, loff_t addr, size_t len, 


	/*

	 * We only support a write to one single page. For now all winbond

	 * flashes only have one page per OTP region.

	 * flashes only have one page per security register.

	 */

	ret = spi_nor_write_enable(nor);

	if (ret)