drivers: disk: mmc_subsys: remove CONFIG_MMC_VOLUME_NAME

Remove CONFIG_MMC_VOLUME_NAME, and set the disk name based on the
``disk-name`` property. This aligns with other disk drivers, and allows
for multiple instances of the mmc_subsys disk driver to be registered.

Add disk-name properties for all in tree definitions for the
mmc-subsys disk driver, and change all in tree usage of the disk name

Fixes #75004

Signed-off-by: Daniel DeGrasse <[email protected]>

Author: danieldegrasse

boards/nxp/mimxrt595_evk/mimxrt595_evk_mimxrt595s_cm33.dts

@@ -371,6 +371,7 @@ zephyr_udc0: &usbhs {
 	cd-gpios = <&gpio2 9 GPIO_ACTIVE_LOW>;
 	mmc {
 		compatible = "zephyr,mmc-disk";
+		disk-name = "SD2";
 		status = "okay";
 	};
 	pinctrl-0 = <&pinmux_usdhc>;

boards/renesas/rcar_h3ulcb/rcar_h3ulcb_r8a77951_a57.dts

@@ -94,6 +94,7 @@
 	pinctrl-names = "default", "uhs";
 	disk {
 		compatible = "zephyr,mmc-disk";
+		disk-name = "SD2";
 		status = "disabled";
 	};
 	bus-width = <8>;

boards/renesas/rcar_salvator_xs/rcar_salvator_xs.dts

@@ -46,6 +46,7 @@
 	pinctrl-names = "default", "uhs";
 	disk {
 		compatible = "zephyr,mmc-disk";
+		disk-name = "SD2";
 		status = "disabled";
 	};
 	bus-width = <8>;

boards/renesas/rcar_spider_s4/rcar_spider_s4_r8a779f0_a55.dts

@@ -41,6 +41,7 @@
 	pinctrl-names = "default", "uhs";
 	disk {
 		compatible = "zephyr,mmc-disk";
+		disk-name = "SD2";
 		status = "okay";
 	};
 	bus-width = <8>;

doc/connectivity/usb/device/usb_device.rst

@@ -323,9 +323,8 @@ or SD Card to the host. Only one disk instance can be exported at a time.
 The disc to be used by the implementation is set by the
 :kconfig:option:`CONFIG_MASS_STORAGE_DISK_NAME` and should be the same as the
 name used by the disc access driver that the application wants to expose to the
-host. MMC disk drivers use option :kconfig:option:`CONFIG_MMC_VOLUME_NAME`, or
-flash, RAM, and SDMMC disk drivers use node property ``disk-name`` to set
-the disk name.
+host. Flash, RAM, and SDMMC/MMC disk drivers use node property ``disk-name`` to
+set the disk name.
 
 For the emulated block device on a flash partition, the flash partition and
 flash disk to be used must be described in the devicetree. If a storage partition

drivers/disk/Kconfig.mmc

@@ -16,13 +16,6 @@ config SD_INIT_PRIORITY
 	help
 	  MMC controller driver initialization priority.
 
-config MMC_VOLUME_NAME
-	string "MMC Disk mount point or drive name"
-	default "SD" if FAT_FILESYSTEM_ELM
-	default "MMC"
-	help
-	  Disk name as per file system naming guidelines.
-
 config MMC_SUBSYS
 	bool "MMC access via SD subsystem"
 	select MMC_STACK

drivers/disk/mmc_subsys.c

@@ -27,7 +27,7 @@ struct mmc_config {
 struct mmc_data {
 	struct sd_card card;
 	enum sd_status status;
-	char *name;
+	struct disk_info *disk_info;
 };
 
 
@@ -107,21 +107,15 @@ static const struct disk_operations mmc_disk_ops = {
 	.ioctl = disk_mmc_access_ioctl,
 };
 
-static struct disk_info mmc_disk = {
-	.ops = &mmc_disk_ops,
-};
-
 static int disk_mmc_init(const struct device *dev)
 {
 	struct mmc_data *data = dev->data;
 	const struct mmc_config *config = dev->config;
 
 	data->status = SD_UNINIT;
 	data->card.bus_width = config->bus_width;
-	mmc_disk.dev = dev;
-	mmc_disk.name = data->name;
 
-	return disk_access_register(&mmc_disk);
+	return disk_access_register(data->disk_info);
 }
 
 #define DISK_ACCESS_MMC_INIT(n)						\
@@ -130,8 +124,14 @@ static int disk_mmc_init(const struct device *dev)
 		.bus_width = DT_INST_PROP(n, bus_width),			\
 	};									\
 										\
+	static struct disk_info mmc_disk_##n = {				\
+		.name = DT_INST_PROP(n, disk_name),				\
+		.ops = &mmc_disk_ops,						\
+		.dev = DEVICE_DT_INST_GET(n),					\
+	};									\
+										\
 	static struct mmc_data mmc_data_##n = {				\
-		.name = CONFIG_MMC_VOLUME_NAME,				\
+		.disk_info = &mmc_disk_##n,					\
 	};									\
 										\
 	DEVICE_DT_INST_DEFINE(n,						\

dts/bindings/sd/zephyr,mmc-disk.yaml

@@ -19,3 +19,9 @@ properties:
       - 1
       - 4
       - 8
+
+  disk-name:
+    type: string
+    required: true
+    description: |
+      Disk name.

dts/x86/intel/alder_lake.dtsi

@@ -348,6 +348,7 @@
 
 			mmc {
 				compatible = "zephyr,mmc-disk";
+				disk-name = "SD2";
 				bus-width = <8>;
 				status = "okay";
 			};

samples/subsys/fs/fs_sample/src/main.c

@@ -22,7 +22,12 @@
  *  Note the fatfs library is able to mount only strings inside _VOLUME_STRS
  *  in ffconf.h
  */
+#if defined(CONFIG_DISK_DRIVER_MMC)
+#define DISK_DRIVE_NAME "SD2"
+#else
 #define DISK_DRIVE_NAME "SD"
+#endif
+
 #define DISK_MOUNT_PT "/"DISK_DRIVE_NAME":"
 
 static FATFS fat_fs;