From 77df397abfdedb063f9e72ec2c12b7993856c2ac Mon Sep 17 00:00:00 2001 From: Madhurima Paruchuri Date: Thu, 21 Jul 2022 11:59:08 +0530 Subject: zephyr: shim: leds: Use 'period' from PWM spec inplace of 'pwm-frequency' Update driver to use 'period' from PWM spec inplace of 'pwm-frequency' BUG=b:230093078 BRANCH=none TEST=zmake testall Signed-off-by: Madhurima Paruchuri Change-Id: I609619010d15a74bc3952d10788f5ef77ad877e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3777837 Reviewed-by: Fabio Baltieri --- zephyr/shim/src/led_driver/led.h | 4 +--- zephyr/shim/src/led_driver/led_pwm.c | 17 +++++------------ 2 files changed, 6 insertions(+), 15 deletions(-) diff --git a/zephyr/shim/src/led_driver/led.h b/zephyr/shim/src/led_driver/led.h index ddde50a347..155763ecca 100644 --- a/zephyr/shim/src/led_driver/led.h +++ b/zephyr/shim/src/led_driver/led.h @@ -61,9 +61,7 @@ struct gpio_pin_t { * Struct defining LED PWM pin and duty cycle to set. */ struct pwm_pin_t { - const struct device *pwm; - uint8_t channel; - pwm_flags_t flags; + struct pwm_dt_spec pwm; uint32_t pulse_ns; /* PWM Duty cycle ns */ }; diff --git a/zephyr/shim/src/led_driver/led_pwm.c b/zephyr/shim/src/led_driver/led_pwm.c index 37bd03d69a..60b7085e38 100644 --- a/zephyr/shim/src/led_driver/led_pwm.c +++ b/zephyr/shim/src/led_driver/led_pwm.c @@ -27,18 +27,13 @@ LOG_MODULE_REGISTER(pwm_led, LOG_LEVEL_ERR); * e.g. freq = 500 Hz, period_ns = 1000000000/500 = 2000000ns * duty_cycle = 50 %, pulse_ns = (2000000*50)/100 = 1000000ns */ -const uint32_t period_ns = - (NSEC_PER_SEC / DT_PROP(PWM_LED_PINS_NODE, pwm_frequency)); #define SET_PIN(node_id, prop, i) \ { \ - .pwm = DEVICE_DT_GET( \ - DT_PWMS_CTLR(DT_PHANDLE_BY_IDX(node_id, prop, i))), \ - .channel = \ - DT_PWMS_CHANNEL(DT_PHANDLE_BY_IDX(node_id, prop, i)), \ - .flags = DT_PWMS_FLAGS(DT_PHANDLE_BY_IDX(node_id, prop, i)), \ + .pwm = PWM_DT_SPEC_GET(DT_PHANDLE_BY_IDX(node_id, prop, i)), \ .pulse_ns = DIV_ROUND_NEAREST( \ - period_ns * DT_PHA_BY_IDX(node_id, prop, i, value), \ + DT_PWMS_PERIOD(DT_PHANDLE_BY_IDX(node_id, prop, i)) * \ + DT_PHA_BY_IDX(node_id, prop, i, value), \ 100), \ }, @@ -78,11 +73,9 @@ const struct led_pins_node_t *pins_node[] = { DT_FOREACH_CHILD( */ void led_set_color_with_node(const struct led_pins_node_t *pins_node) { + struct pwm_pin_t *pwm_pins = pins_node->pwm_pins; for (int j = 0; j < pins_node->pins_count; j++) { - pwm_set(pins_node->pwm_pins[j].pwm, - pins_node->pwm_pins[j].channel, period_ns, - pins_node->pwm_pins[j].pulse_ns, - pins_node->pwm_pins[j].flags); + pwm_set_pulse_dt(&pwm_pins[j].pwm, pwm_pins[j].pulse_ns); } } -- cgit v1.2.1