commit 588594710c78c07c2af7b1b9e92761c2fee324ee
author Tom Rini <trini@konsulko.com> Mon Apr 03 10:43:37 2023 -0400
committer Tom Rini <trini@konsulko.com> Mon Apr 03 10:43:37 2023 -0400
tree b9b178a88f4398c43f6cfd1b374d0ba56f86b5d0
parent 23eb89d57eaddbbe2a566b7dc2829b557a181aca

Revert "disk: Use a helper function to reduce duplication"

Per Takahiro Akashi this is not an equivalent rework, so revert it.

This reverts commit d87bdb82eae66512c222fd93280acaf4dd1cd4be.

Signed-off-by: Tom Rini <trini@konsulko.com>

disk/disk-uclass.c

diff --git a/disk/disk-uclass.c b/disk/disk-uclass.c
index 7f1fd80..d32747e 100644
--- a/disk/disk-uclass.c
+++ b/disk/disk-uclass.c
@@ -65,38 +65,26 @@
 	return 0;
 }
 
-static int blk_part_setup(struct udevice *dev, lbaint_t *startp,
-			  lbaint_t blkcnt)
-{
-	struct disk_part *part;
-
-	part = dev_get_uclass_plat(dev);
-	if (*startp >= part->gpt_part_info.size)
-		return -E2BIG;
-
-	if (*startp + blkcnt > part->gpt_part_info.size)
-		blkcnt = part->gpt_part_info.size - *startp;
-	*startp += part->gpt_part_info.start;
-
-	return 0;
-}
-
 static ulong part_blk_read(struct udevice *dev, lbaint_t start,
 			   lbaint_t blkcnt, void *buffer)
 {
 	struct udevice *parent;
+	struct disk_part *part;
 	const struct blk_ops *ops;
-	int ret;
 
 	parent = dev_get_parent(dev);
 	ops = blk_get_ops(parent);
 	if (!ops->read)
 		return -ENOSYS;
 
-	ret = blk_part_setup(dev, &start, blkcnt);
-	if (ret)
+	part = dev_get_uclass_plat(dev);
+	if (start >= part->gpt_part_info.size)
 		return 0;
 
+	if ((start + blkcnt) > part->gpt_part_info.size)
+		blkcnt = part->gpt_part_info.size - start;
+	start += part->gpt_part_info.start;
+
 	return ops->read(parent, start, blkcnt, buffer);
 }
 
@@ -104,18 +92,22 @@
 			    lbaint_t blkcnt, const void *buffer)
 {
 	struct udevice *parent;
+	struct disk_part *part;
 	const struct blk_ops *ops;
-	int ret;
 
 	parent = dev_get_parent(dev);
 	ops = blk_get_ops(parent);
 	if (!ops->write)
 		return -ENOSYS;
 
-	ret = blk_part_setup(dev, &start, blkcnt);
-	if (ret)
+	part = dev_get_uclass_plat(dev);
+	if (start >= part->gpt_part_info.size)
 		return 0;
 
+	if ((start + blkcnt) > part->gpt_part_info.size)
+		blkcnt = part->gpt_part_info.size - start;
+	start += part->gpt_part_info.start;
+
 	return ops->write(parent, start, blkcnt, buffer);
 }
 
@@ -123,18 +115,22 @@
 			    lbaint_t blkcnt)
 {
 	struct udevice *parent;
+	struct disk_part *part;
 	const struct blk_ops *ops;
-	int ret;
 
 	parent = dev_get_parent(dev);
 	ops = blk_get_ops(parent);
 	if (!ops->erase)
 		return -ENOSYS;
 
-	ret = blk_part_setup(dev, &start, blkcnt);
-	if (ret)
+	part = dev_get_uclass_plat(dev);
+	if (start >= part->gpt_part_info.size)
 		return 0;
 
+	if ((start + blkcnt) > part->gpt_part_info.size)
+		blkcnt = part->gpt_part_info.size - start;
+	start += part->gpt_part_info.start;
+
 	return ops->erase(parent, start, blkcnt);
 }