From 4faf36203064dc7bf6a097d64a9e04270d260993 Mon Sep 17 00:00:00 2001 From: Kumar Gala Date: Thu, 7 Jul 2022 17:11:43 -0500 Subject: [PATCH] soc: apollo_lake: gpio: Drop use of DT_LABEL Change APL_GPIO_DEV_* defines to be DT_NODELABELs instead of "label" strings. This lets us change users of these defines to use DEVICE_DT_GET. We update samples/board/up_squared/gpio_counter to use DEVICE_DT_GET as part of this change. Signed-off-by: Kumar Gala --- .../boards/up_squared/gpio_counter/src/main.c | 17 +++++++--------- soc/x86/apollo_lake/soc_gpio.h | 20 +++++++++---------- 2 files changed, 17 insertions(+), 20 deletions(-) diff --git a/samples/boards/up_squared/gpio_counter/src/main.c b/samples/boards/up_squared/gpio_counter/src/main.c index 01c5fb39398..1c995f2f95e 100644 --- a/samples/boards/up_squared/gpio_counter/src/main.c +++ b/samples/boards/up_squared/gpio_counter/src/main.c @@ -40,7 +40,6 @@ struct _pin { uint32_t hat_num; uint32_t pin; - const char *gpio_dev_name; const struct device *gpio_dev; }; @@ -48,29 +47,29 @@ struct _pin counter_pins[] = { { .hat_num = 35, .pin = UP2_HAT_PIN_35, - .gpio_dev_name = UP2_HAT_PIN_35_DEV, + .gpio_dev = DEVICE_DT_GET(UP2_HAT_PIN_35_DEV), }, { .hat_num = 37, .pin = UP2_HAT_PIN_37, - .gpio_dev_name = UP2_HAT_PIN_37_DEV, + .gpio_dev = DEVICE_DT_GET(UP2_HAT_PIN_37_DEV), }, { .hat_num = 38, .pin = UP2_HAT_PIN_38, - .gpio_dev_name = UP2_HAT_PIN_38_DEV, + .gpio_dev = DEVICE_DT_GET(UP2_HAT_PIN_38_DEV), }, { .hat_num = 40, .pin = UP2_HAT_PIN_40, - .gpio_dev_name = UP2_HAT_PIN_40_DEV, + .gpio_dev = DEVICE_DT_GET(UP2_HAT_PIN_40_DEV), }, }; struct _pin intr_pin = { .hat_num = 16, .pin = UP2_HAT_PIN_16, - .gpio_dev_name = UP2_HAT_PIN_16_DEV, + .gpio_dev = DEVICE_DT_GET(UP2_HAT_PIN_16_DEV), }; static struct gpio_callback gpio_cb; @@ -91,10 +90,8 @@ void button_cb(const struct device *gpiodev, struct gpio_callback *cb, int get_gpio_dev(struct _pin *pin) { - pin->gpio_dev = device_get_binding(pin->gpio_dev_name); - if (!pin->gpio_dev) { - printk("ERROR: cannot get device binding for %s\n", - pin->gpio_dev_name); + if (!device_is_ready(pin->gpio_dev)) { + printk("ERROR: GPIO device is not ready for %s\n", pin->gpio_dev->name); return -1; } diff --git a/soc/x86/apollo_lake/soc_gpio.h b/soc/x86/apollo_lake/soc_gpio.h index cb278609dd3..3f98a405727 100644 --- a/soc/x86/apollo_lake/soc_gpio.h +++ b/soc/x86/apollo_lake/soc_gpio.h @@ -15,7 +15,7 @@ #ifndef __SOC_GPIO_H_ #define __SOC_GPIO_H_ -#define APL_GPIO_DEV_N_0 DT_LABEL(DT_NODELABEL(gpio_n_000_031)) +#define APL_GPIO_DEV_N_0 DT_NODELABEL(gpio_n_000_031) #define APL_GPIO_0 0 #define APL_GPIO_1 1 #define APL_GPIO_2 2 @@ -49,7 +49,7 @@ #define APL_GPIO_30 30 #define APL_GPIO_31 31 -#define APL_GPIO_DEV_N_1 DT_LABEL(DT_NODELABEL(gpio_n_032_063)) +#define APL_GPIO_DEV_N_1 DT_NODELABEL(gpio_n_032_063) #define APL_GPIO_32 0 #define APL_GPIO_33 1 #define APL_GPIO_34 2 @@ -83,7 +83,7 @@ #define APL_GPIO_TCK 30 #define APL_GPIO_TRST_B 31 -#define APL_GPIO_DEV_N_2 DT_LABEL(DT_NODELABEL(gpio_n_064_077)) +#define APL_GPIO_DEV_N_2 DT_NODELABEL(gpio_n_064_077) #define APL_GPIO_TMS 0 #define APL_GPIO_TDI 1 #define APL_GPIO_CX_PMODE 2 @@ -99,7 +99,7 @@ #define APL_GPIO_SVOD0_DATA 12 #define APL_GPIO_SVOD0_CLK 13 -#define APL_GPIO_DEV_NW_0 DT_LABEL(DT_NODELABEL(gpio_nw_000_031)) +#define APL_GPIO_DEV_NW_0 DT_NODELABEL(gpio_nw_000_031) #define APL_GPIO_187 0 #define APL_GPIO_188 1 #define APL_GPIO_189 2 @@ -133,7 +133,7 @@ #define APL_GPIO_PMIC_STDBY 30 #define APL_GPIO_PROCHOT_B 31 -#define APL_GPIO_DEV_NW_1 DT_LABEL(DT_NODELABEL(gpio_nw_032_063)) +#define APL_GPIO_DEV_NW_1 DT_NODELABEL(gpio_nw_032_063) #define APL_GPIO_PMIC_I2C_SCL 0 #define APL_GPIO_PMIC_I2C_SDA 1 #define APL_GPIO_74 2 @@ -167,7 +167,7 @@ #define APL_GPIO_105 30 #define APL_GPIO_106 31 -#define APL_GPIO_DEV_NW_2 DT_LABEL(DT_NODELABEL(gpio_nw_064_076)) +#define APL_GPIO_DEV_NW_2 DT_NODELABEL(gpio_nw_064_076) #define APL_GPIO_109 0 #define APL_GPIO_110 1 #define APL_GPIO_111 2 @@ -182,7 +182,7 @@ #define APL_GPIO_122 11 #define APL_GPIO_123 12 -#define APL_GPIO_DEV_W_0 DT_LABEL(DT_NODELABEL(gpio_w_000_031)) +#define APL_GPIO_DEV_W_0 DT_NODELABEL(gpio_w_000_031) #define APL_GPIO_124 0 #define APL_GPIO_125 1 #define APL_GPIO_126 2 @@ -216,7 +216,7 @@ #define APL_GPIO_OSC_CLK_OUT_0 30 #define APL_GPIO_OSC_CLK_OUT_1 31 -#define APL_GPIO_DEV_W_1 DT_LABEL(DT_NODELABEL(gpio_w_032_046)) +#define APL_GPIO_DEV_W_1 DT_NODELABEL(gpio_w_032_046) #define APL_GPIO_OSC_CLK_OUT_2 0 #define APL_GPIO_OSC_CLK_OUT_3 1 #define APL_GPIO_OSC_CLK_OUT_4 2 @@ -233,7 +233,7 @@ #define APL_GPIO_SUS_STAT_B 13 #define APL_GPIO_SUSPWRDNACK 14 -#define APL_GPIO_DEV_SW_0 DT_LABEL(DT_NODELABEL(gpio_sw_000_031)) +#define APL_GPIO_DEV_SW_0 DT_NODELABEL(gpio_sw_000_031) #define APL_GPIO_205 0 #define APL_GPIO_206 1 #define APL_GPIO_207 2 @@ -267,7 +267,7 @@ #define APL_GPIO_183 30 #define APL_GPIO_SMB_ALERTB 31 -#define APL_GPIO_DEV_SW_1 DT_LABEL(DT_NODELABEL(gpio_sw_032_042)) +#define APL_GPIO_DEV_SW_1 DT_NODELABEL(gpio_sw_032_042) #define APL_GPIO_SMB_CLK 0 #define APL_GPIO_SMB_DATA 1 #define APL_GPIO_LPC_ILB_SERIRQ 2