[PATCH 1/7] ARM: Freescale i.MX23 evk: use cdev_open_by_name()

[Sascha Hauer]

cdev_by_name() returns a cdev without increasing its reference count. In
order to maintain a proper reference counting use cdev_open_by_name()
instead and make sure it's closed afterwards when no longer needed.

Signed-off-by: Sascha Hauer <s.ha...@pengutronix.de>
---
 arch/arm/boards/freescale-mx23-evk/mx23-evk.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boards/freescale-mx23-evk/mx23-evk.c 
b/arch/arm/boards/freescale-mx23-evk/mx23-evk.c
index d4de99eafb..b10a6e8dcd 100644
--- a/arch/arm/boards/freescale-mx23-evk/mx23-evk.c
+++ b/arch/arm/boards/freescale-mx23-evk/mx23-evk.c
@@ -52,22 +52,21 @@ static struct fsl_usb2_platform_data usb_pdata = {
  */
 static int register_persistent_environment(void)
 {
-       struct cdev *cdev;
+       struct cdev *cdev, *env;
 
        /*
         * The imx23-olinuxino only has one MCI card socket.
         * So, we expect its name as "disk0".
         */
-       cdev = cdev_by_name("disk0");
+       cdev = cdev_open_by_name("disk0", O_RDONLY);
        if (cdev == NULL) {
                pr_err("No MCI card preset\n");
                return -ENODEV;
        }
-
-
+       cdev_close(cdev);
 
        /* MCI card is present, also a useable partition on it? */
-       cdev = cdev_by_name("disk0.1");
+       cdev = cdev_open_by_name("disk0.1", O_RDONLY);
        if (cdev == NULL) {
                pr_err("No second partition available\n");
                pr_info("Please create at least a second partition with"
@@ -76,8 +75,10 @@ static int register_persistent_environment(void)
        }
 
        /* use the full partition as our persistent environment storage */
-       cdev = devfs_add_partition("disk0.1", 0, cdev->size,
+       env = devfs_add_partition("disk0.1", 0, cdev->size,
                                                DEVFS_PARTITION_FIXED, "env0");
+       cdev_close(cdev);
+
        return PTR_ERR_OR_ZERO(cdev);
 }
 
-- 
2.39.2