Commit de3cac93 authored by Artem Bityutskiy's avatar Artem Bityutskiy Committed by David Woodhouse
Browse files

mtd: check for zero length in OTP functions 



This patch changes all the OTP functions like 'mtd_get_fact_prot_info()' and
makes them return zero immediately if the input 'len' parameter is 0. This is
not really needed currently, but most of the other functions do this, and it is
just consistent to do the same in the OTP functions.

This patch also moves the OTP functions from the header file to mtdcore.c
because they become a bit too big for being inlined.

Signed-off-by: default avatarArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: default avatarDavid Woodhouse <David.Woodhouse@intel.com>
parent bcb1d238

drivers/mtd/mtdcore.c

+73 −0
Original line number Diff line number Diff line
@@ -802,6 +802,79 @@ int mtd_panic_write(struct mtd_info *mtd, loff_t to, size_t len, size_t *retlen, 
}

EXPORT_SYMBOL_GPL(mtd_panic_write);



/*

 * Method to access the protection register area, present in some flash

 * devices. The user data is one time programmable but the factory data is read

 * only.

 */

int mtd_get_fact_prot_info(struct mtd_info *mtd, struct otp_info *buf,

			   size_t len)

{

	if (!mtd->_get_fact_prot_info)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_get_fact_prot_info(mtd, buf, len);

}

EXPORT_SYMBOL_GPL(mtd_get_fact_prot_info);



int mtd_read_fact_prot_reg(struct mtd_info *mtd, loff_t from, size_t len,

			   size_t *retlen, u_char *buf)

{

	*retlen = 0;

	if (!mtd->_read_fact_prot_reg)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_read_fact_prot_reg(mtd, from, len, retlen, buf);

}

EXPORT_SYMBOL_GPL(mtd_read_fact_prot_reg);



int mtd_get_user_prot_info(struct mtd_info *mtd, struct otp_info *buf,

			   size_t len)

{

	if (!mtd->_get_user_prot_info)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_get_user_prot_info(mtd, buf, len);

}

EXPORT_SYMBOL_GPL(mtd_get_user_prot_info);



int mtd_read_user_prot_reg(struct mtd_info *mtd, loff_t from, size_t len,

			   size_t *retlen, u_char *buf)

{

	*retlen = 0;

	if (!mtd->_read_user_prot_reg)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_read_user_prot_reg(mtd, from, len, retlen, buf);

}

EXPORT_SYMBOL_GPL(mtd_read_user_prot_reg);



int mtd_write_user_prot_reg(struct mtd_info *mtd, loff_t to, size_t len,

			    size_t *retlen, u_char *buf)

{

	*retlen = 0;

	if (!mtd->_write_user_prot_reg)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_write_user_prot_reg(mtd, to, len, retlen, buf);

}

EXPORT_SYMBOL_GPL(mtd_write_user_prot_reg);



int mtd_lock_user_prot_reg(struct mtd_info *mtd, loff_t from, size_t len)

{

	if (!mtd->_lock_user_prot_reg)

		return -EOPNOTSUPP;

	if (!len)

		return 0;

	return mtd->_lock_user_prot_reg(mtd, from, len);

}

EXPORT_SYMBOL_GPL(mtd_lock_user_prot_reg);



/* Chip-supported device locking */

int mtd_lock(struct mtd_info *mtd, loff_t ofs, uint64_t len)

{

include/linux/mtd/mtd.h

+11 −59
Original line number Diff line number Diff line
@@ -273,65 +273,17 @@ static inline int mtd_write_oob(struct mtd_info *mtd, loff_t to, 
	return mtd->_write_oob(mtd, to, ops);

}



/*

 * Method to access the protection register area, present in some flash

 * devices. The user data is one time programmable but the factory data is read

 * only.

 */

static inline int mtd_get_fact_prot_info(struct mtd_info *mtd,

					 struct otp_info *buf, size_t len)

{

	if (!mtd->_get_fact_prot_info)

		return -EOPNOTSUPP;

	return mtd->_get_fact_prot_info(mtd, buf, len);

}



static inline int mtd_read_fact_prot_reg(struct mtd_info *mtd, loff_t from,

					 size_t len, size_t *retlen,

					 u_char *buf)

{

	*retlen = 0;

	if (!mtd->_read_fact_prot_reg)

		return -EOPNOTSUPP;

	return mtd->_read_fact_prot_reg(mtd, from, len, retlen, buf);

}



static inline int mtd_get_user_prot_info(struct mtd_info *mtd,

					 struct otp_info *buf,

					 size_t len)

{

	if (!mtd->_get_user_prot_info)

		return -EOPNOTSUPP;

	return mtd->_get_user_prot_info(mtd, buf, len);

}



static inline int mtd_read_user_prot_reg(struct mtd_info *mtd, loff_t from,

					 size_t len, size_t *retlen,

					 u_char *buf)

{

	*retlen = 0;

	if (!mtd->_read_user_prot_reg)

		return -EOPNOTSUPP;

	return mtd->_read_user_prot_reg(mtd, from, len, retlen, buf);

}



static inline int mtd_write_user_prot_reg(struct mtd_info *mtd, loff_t to,

					  size_t len, size_t *retlen,

					  u_char *buf)

{

	*retlen = 0;

	if (!mtd->_write_user_prot_reg)

		return -EOPNOTSUPP;

	return mtd->_write_user_prot_reg(mtd, to, len, retlen, buf);

}



static inline int mtd_lock_user_prot_reg(struct mtd_info *mtd, loff_t from,

					 size_t len)

{

	if (!mtd->_lock_user_prot_reg)

		return -EOPNOTSUPP;

	return mtd->_lock_user_prot_reg(mtd, from, len);

}

int mtd_get_fact_prot_info(struct mtd_info *mtd, struct otp_info *buf,

			   size_t len);

int mtd_read_fact_prot_reg(struct mtd_info *mtd, loff_t from, size_t len,

			   size_t *retlen, u_char *buf);

int mtd_get_user_prot_info(struct mtd_info *mtd, struct otp_info *buf,

			   size_t len);

int mtd_read_user_prot_reg(struct mtd_info *mtd, loff_t from, size_t len,

			   size_t *retlen, u_char *buf);

int mtd_write_user_prot_reg(struct mtd_info *mtd, loff_t to, size_t len,

			    size_t *retlen, u_char *buf);

int mtd_lock_user_prot_reg(struct mtd_info *mtd, loff_t from, size_t len);



int mtd_writev(struct mtd_info *mtd, const struct kvec *vecs,

	       unsigned long count, loff_t to, size_t *retlen);

CRA Git | Maintained and supported by SUSTech CRA and CCSE