Re: [PATCH] fs: implement cdev_mount()

[Sascha Hauer <sha@pengutronix.de>]

On Fri, Sep 30, 2022 at 05:40:17PM +0200, Ahmad Fatoum wrote:
> EFI loaders provide both block device and file system access to software
> running under it. For file system access, we will just want to get a
> mount if available and mount at a default location if not. Provide a
> helper that does just that.
> 
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
>  fs/fs.c      | 19 +++++++++++++++++++
>  include/fs.h |  1 +
>  2 files changed, 20 insertions(+)

Applied, thanks.

Also added the following.

Sascha

-----------------------------8<------------------------
>From 49c2e4ebe8f4b5f457201385f075fb80476a7977 Mon Sep 17 00:00:00 2001
From: Sascha Hauer <s.hauer@pengutronix.de>
Date: Tue, 4 Oct 2022 11:02:40 +0200
Subject: [PATCH] blspec: Use cdev_mount()

In blspec we want either get the path where the cdev is mounted or mount
it to the default path. That's what cdev_mount() does for us, so use it.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 common/blspec.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/common/blspec.c b/common/blspec.c
index d391f690ad..9ae7b49a77 100644
--- a/common/blspec.c
+++ b/common/blspec.c
@@ -677,9 +677,7 @@ static int blspec_scan_cdev(struct bootentries *bootentries, struct cdev *cdev)
 			found += ret;
 	}
 
-	rootpath = cdev_get_mount_path(cdev);
-	if (!rootpath)
-		rootpath = cdev_mount_default(cdev, NULL);
+	rootpath = cdev_mount(cdev);
 	if (!IS_ERR(rootpath)) {
 		ret = blspec_scan_directory(bootentries, rootpath);
 		if (ret > 0)
-- 
2.30.2

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |