On 25.09.21 18:05, Pali Rohár wrote:
On Saturday 25 September 2021 02:49:18 Marek Behún wrote:
From: Marek Behún <[email protected]>

After getting MTD device via get_mtd_device_nm(), we need to put it with
put_mtd_device(), otherwise we get

   Removing MTD device #0 (mx25l6405d) with use count 1

before booting kernel.

Signed-off-by: Marek Behún <[email protected]>

Reviewed-by: Pali Rohár <[email protected]>
Tested-by: Pali Rohár <[email protected]>
Fixes: 92f36c8e74c1 ("arm: mvebu: turris_omnia: fixup MTD partitions in Linux' 
DTB")

Above commit was introduced in v2021.10-rc1, so I think this fixup
should go into v2021.10.

Stefan, what do you think?

Makes sense. I'll push this out today.

Thanks,
Stefan

---
  board/CZ.NIC/turris_omnia/turris_omnia.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c 
b/board/CZ.NIC/turris_omnia/turris_omnia.c
index bac78af04e..a48e1f5c30 100644
--- a/board/CZ.NIC/turris_omnia/turris_omnia.c
+++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
@@ -647,10 +647,13 @@ int ft_board_setup(void *blob, struct bd_info *bd)
        if (!fixup_mtd_partitions(blob, node, mtd))
                goto fail;
+ put_mtd_device(mtd);
        return 0;
fail:
        printf("Failed fixing SPI NOR partitions!\n");
+       if (!IS_ERR_OR_NULL(mtd))
+               put_mtd_device(mtd);
        return 0;
  }
  #endif
--
2.32.0



Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: [email protected]

Reply via email to