devices: use device_name_get

Since struct device name field is now optional, use device_name_get.

Signed-off-by: Gerard Marull-Paretas <[email protected]>

Author: gmarull

boards/arm/nrf9160dk_nrf52840/board.c

@@ -124,7 +124,7 @@ static int reset_pin_configure(void)
 	gpio_dt_flags_t flags = GET_FLAGS(reset_input, gpios, 0);
 
 	if (!device_is_ready(gpio)) {
-		LOG_ERR("%s is not ready", gpio->name);
+		LOG_ERR("%s is not ready", device_name_get(gpio));
 		return -ENODEV;
 	}
 
@@ -168,7 +168,7 @@ static int init(const struct device *dev)
 		gpio_flags_t flags = cfg->flags;
 
 		if (!device_is_ready(cfg->gpio)) {
-			LOG_ERR("%s is not ready", cfg->gpio->name);
+			LOG_ERR("%s is not ready", device_name_get(cfg->gpio));
 			return -ENODEV;
 		}
 

boards/arm/pinetime_devkit0/key_out.c

@@ -22,14 +22,14 @@ static int pinetime_key_out_init(const struct device *arg)
 
 	if (!device_is_ready(key_out.port)) {
 		LOG_ERR("key out gpio device %s is not ready",
-			key_out.port->name);
+			device_name_get(key_out.port));
 		return -ENODEV;
 	}
 
 	ret = gpio_pin_configure_dt(&key_out, GPIO_OUTPUT_ACTIVE);
 	if (ret != 0) {
 		LOG_ERR("failed to configure %s pin %d (err %d)",
-			key_out.port->name, key_out.pin, ret);
+			device_name_get(key_out.port), key_out.pin, ret);
 		return ret;
 	}
 

doc/kernel/services/threads/workqueue.rst

@@ -301,7 +301,7 @@ unchanged and the associated error message will not be printed.
     {
         struct device_info *the_device =
             CONTAINER_OF(item, struct device_info, work);
-        printk("Got error on device %s\n", the_device->name);
+        printk("Got error on device %s\n", device_name_get(the_device));
     }
 
     /* initialize name info for a device */

doc/services/smf/index.rst

@@ -422,22 +422,22 @@ Code::
 
 		if (!device_is_ready(button.port)) {
 			printk("Error: button device %s is not ready\n",
-				button.port->name);
+				device_name_get(button.port));
 			return;
 		}
 
 		ret = gpio_pin_configure_dt(&button, GPIO_INPUT);
 		if (ret != 0) {
 			printk("Error %d: failed to configure %s pin %d\n",
-				ret, button.port->name, button.pin);
+				ret, device_name_get(button.port), button.pin);
 			return;
 		}
 
 		ret = gpio_pin_interrupt_configure_dt(&button,
 			GPIO_INT_EDGE_TO_ACTIVE);
 		if (ret != 0) {
 			printk("Error %d: failed to configure interrupt on %s pin %d\n",
-				ret, button.port->name, button.pin);
+				ret, device_name_get(button.port), button.pin);
 			return;
 		}
 

drivers/adc/adc_ads1119.c

@@ -458,7 +458,7 @@ static int ads1119_init(const struct device *dev)
 
 	rc = ads1119_read_reg(dev, ADS1119_REG_STATUS, &status);
 	if (rc) {
-		LOG_ERR("Could not get %s status", dev->name);
+		LOG_ERR("Could not get %s status", device_name_get(dev));
 		return rc;
 	}
 

drivers/adc/adc_ads1x1x.c

@@ -554,7 +554,7 @@ static int ads1x1x_init(const struct device *dev)
 	k_sem_init(&data->acq_sem, 0, 1);
 
 	if (!device_is_ready(config->bus.bus)) {
-		LOG_ERR("I2C bus %s not ready", config->bus.bus->name);
+		LOG_ERR("I2C bus %s not ready", device_name_get(config->bus.bus));
 		return -ENODEV;
 	}
 

drivers/adc/adc_lmp90xxx.c

@@ -941,7 +941,7 @@ static int lmp90xxx_init(const struct device *dev)
 	data->ura = LMP90XXX_INVALID_URA;
 
 	if (!spi_is_ready(&config->bus)) {
-		LOG_ERR("SPI bus %s not ready", config->bus.bus->name);
+		LOG_ERR("SPI bus %s not ready", device_name_get(config->bus.bus));
 		return -ENODEV;
 	}
 

drivers/adc/adc_nrfx_adc.c

@@ -268,7 +268,7 @@ static int init_adc(const struct device *dev)
 
 	if (result != NRFX_SUCCESS) {
 		LOG_ERR("Failed to initialize device: %s",
-			    dev->name);
+			    device_name_get(dev));
 		return -EBUSY;
 	}
 

drivers/adc/adc_shell.c

@@ -112,7 +112,7 @@ static struct adc_hdl {
 static struct adc_hdl *get_adc(const char *device_label)
 {
 	for (int i = 0; i < ARRAY_SIZE(adc_list); i++) {
-		if (!strcmp(device_label, adc_list[i].dev->name)) {
+		if (!strcmp(device_label, device_name_get(adc_list[i].dev))) {
 			return &adc_list[i];
 		}
 	}
@@ -347,7 +347,7 @@ static int cmd_adc_print(const struct shell *shell, size_t argc, char **argv)
 			   "Acquisition Time: %u\n"
 			   "Channel ID: %u\n"
 			   "Resolution: %u",
-			   adc->dev->name,
+			   device_name_get(adc->dev),
 			   chosen_gain,
 			   chosen_reference,
 			   adc->channel_config.acquisition_time,
@@ -408,7 +408,7 @@ static void cmd_adc_dev_get(size_t idx, struct shell_static_entry *entry)
 {
 	/* -1 because the last element in the list is a "list terminator" */
 	if (idx < ARRAY_SIZE(adc_list) - 1) {
-		entry->syntax  = adc_list[idx].dev->name;
+		entry->syntax  = device_name_get(adc_list[idx].dev);
 		entry->handler = NULL;
 		entry->subcmd  = &sub_adc_cmds;
 		entry->help    = "Select subcommand for ADC property label.\n";

drivers/adc/adc_stm32.c

@@ -939,7 +939,7 @@ static void adc_stm32_isr(const struct device *dev)
 
 	adc_context_on_sampling_done(&data->ctx, dev);
 
-	LOG_DBG("%s ISR triggered.", dev->name);
+	LOG_DBG("%s ISR triggered.", device_name_get(dev));
 }
 
 static int adc_stm32_read(const struct device *dev,