diff mbox series

[1/1] omap: Omit fastboot.userdata_size related errors

Message ID 20170921200100.27886-1-semen.protsenko@linaro.org
State Accepted
Commit 19d141a0f90c640a8d5ed699745cbc8169918945
Headers show
Series [1/1] omap: Omit fastboot.userdata_size related errors | expand

Commit Message

Sam Protsenko Sept. 21, 2017, 8:01 p.m. UTC 
When using regular $partitions variable (for Linux boot), we can see
some unwanted messages:

> ERROR: cannot find partition: 'userdata'

> at arch/arm/mach-omap2/utils.c:96/omap_mmc_get_part_size()
> Warning: fastboot.userdata_size: unable to calc

Let's remove those, as missing 'userdata' partition is correct behavior
for Linux partition, and we don't want to see some Android-related
messages in this case.

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
---
 arch/arm/mach-omap2/utils.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

Comments

Tom Rini Oct. 7, 2017, 1:08 p.m. UTC | #1 
On Thu, Sep 21, 2017 at 11:01:00PM +0300, Sam Protsenko wrote:

> When using regular $partitions variable (for Linux boot), we can see

> some unwanted messages:

> 

> > ERROR: cannot find partition: 'userdata'

> 

> > at arch/arm/mach-omap2/utils.c:96/omap_mmc_get_part_size()

> > Warning: fastboot.userdata_size: unable to calc

> 

> Let's remove those, as missing 'userdata' partition is correct behavior

> for Linux partition, and we don't want to see some Android-related

> messages in this case.

> 

> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>


Applied to u-boot/master, thanks!

-- 
Tom
diff mbox series

Patch

diff --git a/arch/arm/mach-omap2/utils.c b/arch/arm/mach-omap2/utils.c
index 0b0bf1837c..3892853c7e 100644
--- a/arch/arm/mach-omap2/utils.c
+++ b/arch/arm/mach-omap2/utils.c
@@ -92,10 +92,8 @@  static u32 omap_mmc_get_part_size(const char *part)
 	}
 
 	res = part_get_info_by_name(dev_desc, part, &info);
-	if (res < 0) {
-		error("cannot find partition: '%s'\n", part);
+	if (res < 0)
 		return 0;
-	}
 
 	/* Calculate size in bytes */
 	sz = (info.size * (u64)info.blksz);
@@ -111,13 +109,10 @@  static void omap_set_fastboot_userdata_size(void)
 	u32 sz_kb;
 
 	sz_kb = omap_mmc_get_part_size("userdata");
-	if (sz_kb == 0) {
-		buf[0] = '\0';
-		printf("Warning: fastboot.userdata_size: unable to calc\n");
-	} else {
-		sprintf(buf, "%u", sz_kb);
-	}
+	if (sz_kb == 0)
+		return; /* probably it's not Android partition table */
 
+	sprintf(buf, "%u", sz_kb);
 	env_set("fastboot.userdata_size", buf);
 }
 #else