From ccb2e466874c12fa060f9741f491bffa897a3a0c Mon Sep 17 00:00:00 2001 From: Artem Bityutskiy Date: Tue, 15 Jun 2010 13:03:14 +0300 Subject: ubi-utils: harmonize mtd device node variables Consistently use 'mtd_dev_node' to name variables consining MTD device node path. Signed-off-by: Artem Bityutskiy --- ubi-utils/include/libubi.h | 20 ++++++++++---------- ubi-utils/src/libubi.c | 31 ++++++++++++++++--------------- ubi-utils/src/ubiattach.c | 2 +- 3 files changed, 27 insertions(+), 26 deletions(-) (limited to 'ubi-utils') diff --git a/ubi-utils/include/libubi.h b/ubi-utils/include/libubi.h index ffc4be9..03b16ad 100644 --- a/ubi-utils/include/libubi.h +++ b/ubi-utils/include/libubi.h @@ -43,8 +43,8 @@ typedef void * libubi_t; * @dev_num: number to assign to the newly created UBI device * (%UBI_DEV_NUM_AUTO should be used to automatically assign the * number) - * @mtd_num: MTD device number to attach (used if @dev is %NULL) - * @dev: path to MTD device node to attach + * @mtd_num: MTD device number to attach (used if @mtd_dev_node is %NULL) + * @mtd_dev_node: path to MTD device node to attach * @vid_hdr_offset: VID header offset (%0 means default offset and this is what * most of the users want) */ @@ -52,7 +52,7 @@ struct ubi_attach_request { int dev_num; int mtd_num; - const char *dev; + const char *mtd_dev_node; int vid_hdr_offset; }; @@ -226,9 +226,9 @@ int ubi_attach_mtd(libubi_t desc, const char *node, * @req: MTD attach request * * This function creates new UBI device by attaching an MTD device described by - * @req. If @req->dev is given it should contain path to the MTD device node. - * Otherwise functionality is similar than in function 'ubi_attach_mtd()' where - * @req->mtd_num is used. + * @req. If @req->mtd_dev_node is given it should contain path to the MTD + * device node. Otherwise functionality is similar than in function + * 'ubi_attach_mtd()' where @req->mtd_num is used. * * Returns %0 in case of success and %-1 in case of failure (errno is set). The * newly created UBI device number is returned in @req->dev_num. @@ -252,12 +252,12 @@ int ubi_detach_mtd(libubi_t desc, const char *node, int mtd_num); * ubi_detach - detach an MTD device by its node path. * @desc: UBI library descriptor * @node: name of the UBI control character device node - * @dev: path to an MTD device node + * @mtd_dev_node: path to an MTD device node * - * This function detaches an MTD device @dev from UBI. Returns zero in case of - * success and %-1 in case of failure. + * This function detaches an MTD device @mtd_dev_node from UBI. Returns zero in + * case of success and %-1 in case of failure. */ -int ubi_detach(libubi_t desc, const char *node, const char *dev); +int ubi_detach(libubi_t desc, const char *node, const char *mtd_dev_node); /** * ubi_remove_dev - remove an UBI device. diff --git a/ubi-utils/src/libubi.c b/ubi-utils/src/libubi.c index 3fe7956..2eab3ba 100644 --- a/ubi-utils/src/libubi.c +++ b/ubi-utils/src/libubi.c @@ -736,24 +736,25 @@ int ubi_attach_mtd(libubi_t desc, const char *node, #endif /** - * dev_to_mtdnum - converts device node to MTD number. - * @dev: path to device node to convert + * mtd_node_to_num - converts device node to MTD number. + * @mtd_dev_node: path to device node to convert * - * This function converts given @dev to MTD device number. @dev should contain - * path to the MTD device node. Returns MTD device number in case of success and - * %-1 in case of failure (errno is set). + * This function converts given @mtd_dev_node to MTD device number. + * @mtd_dev_node should contain path to the MTD device node. Returns MTD device + * number in case of success and %-1 in case of failure (errno is set). */ -static int dev_to_mtdnum(const char *dev) +static int mtd_node_to_num(const char *mtd_dev_node) { int major, minor; struct stat sb; - if (stat(dev, &sb) < 0) - return sys_errmsg("cannot stat \"%s\"", dev); + if (stat(mtd_dev_node, &sb) < 0) + return sys_errmsg("cannot stat \"%s\"", mtd_dev_node); if (!S_ISCHR(sb.st_mode)) { errno = EINVAL; - return sys_errmsg("\"%s\" is not a character device", dev); + return sys_errmsg("\"%s\" is not a character device", + mtd_dev_node); } major = major(sb.st_rdev); @@ -761,7 +762,7 @@ static int dev_to_mtdnum(const char *dev) if (major != MTD_CHAR_MAJOR) { errno = EINVAL; - return sys_errmsg("\"%s\" is not an MTD device", dev); + return sys_errmsg("\"%s\" is not an MTD device", mtd_dev_node); } return minor / 2; @@ -772,7 +773,7 @@ int ubi_attach(libubi_t desc, const char *node, struct ubi_attach_request *req) struct ubi_attach_req r; int ret; - if (!req->dev) + if (!req->mtd_dev_node) /* Fallback to opening by mtd_num */ return ubi_attach_mtd(desc, node, req); @@ -784,7 +785,7 @@ int ubi_attach(libubi_t desc, const char *node, struct ubi_attach_request *req) * User has passed path to device node. Lets find out MTD device number * of the device and pass it to the kernel. */ - r.mtd_num = dev_to_mtdnum(req->dev); + r.mtd_num = mtd_node_to_num(req->mtd_dev_node); if (r.mtd_num == -1) return -1; @@ -808,16 +809,16 @@ int ubi_detach_mtd(libubi_t desc, const char *node, int mtd_num) return ubi_remove_dev(desc, node, ubi_dev); } -int ubi_detach(libubi_t desc, const char *node, const char *dev) +int ubi_detach(libubi_t desc, const char *node, const char *mtd_dev_node) { int mtd_num; - if (!dev) { + if (!mtd_dev_node) { errno = EINVAL; return -1; } - mtd_num = dev_to_mtdnum(dev); + mtd_num = mtd_node_to_num(mtd_dev_node); if (mtd_num == -1) return -1; diff --git a/ubi-utils/src/ubiattach.c b/ubi-utils/src/ubiattach.c index 67f8c85..210624b 100644 --- a/ubi-utils/src/ubiattach.c +++ b/ubi-utils/src/ubiattach.c @@ -188,7 +188,7 @@ int main(int argc, char * const argv[]) req.dev_num = args.devn; req.mtd_num = args.mtdn; req.vid_hdr_offset = args.vidoffs; - req.dev = args.dev; + req.mtd_dev_node = args.dev; err = ubi_attach(libubi, args.node, &req); if (err) { -- cgit v1.2.3