Skip to content

drivers: stepper: adi: trinamic tmc5041 #153198

drivers: stepper: adi: trinamic tmc5041

drivers: stepper: adi: trinamic tmc5041 #153198

Triggered via pull request October 7, 2024 11:02
Status Success
Total duration 2m 34s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
2m 28s
Run compliance checks on patch series (PR)
Fit to window
Zoom out
Zoom in

Annotations

1 error, 1 warning, and 10 notices
Run compliance checks on patch series (PR)
Process completed with exit code 1.
Run compliance checks on patch series (PR): ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details. You may want to run clang-format on this change: - &rampstat_value); + &rampstat_value); File:drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c Line:216 You may want to run clang-format on this change: - k_work_reschedule(&stepper_data->rampstat_callback_dwork, + k_work_reschedule( + &stepper_data->rampstat_callback_dwork, File:drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c Line:255 You may want to run clang-format on this change: - TMC5041_SWMODE(stepper_config->index), - BIT(10)); + TMC5041_SWMODE(stepper_config->index), BIT(10)); File:drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c Line:647 You may want to run clang-format on this change: - -#define TMC5041_SHAFT_CONFIG(child) \ - (DT_PROP(child, invert_direction) << TMC5041_GCONF_SHAFT_SHIFT(DT_REG_ADDR(child)))| - -#define TMC5041_STEPPER_CONFIG_DEFINE(child) \ +#define TMC5041_SHAFT_CONFIG(child) \ + (DT_PROP(child, invert_direction) << TMC5041_GCONF_SHAFT_SHIFT(DT_REG_ADDR(child))) | + +#define TMC5041_STEPPER_CONFIG_DEFINE(child) \ COND_CODE_1(DT_PROP(child, stallguard_threshold_velocity), \ BUILD_ASSERT(DT_PROP(child, stallguard_threshold_velocity), \ - "stallguard threshold velocity must be a positive value"), ()); \ - IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, (CHECK_RAMP_DATA(child))); \ - static const struct tmc5041_stepper_config tmc5041_stepper_config_##child = { \ - .controller = DEVICE_DT_GET(DT_PARENT(child)), \ - .default_micro_step_res = DT_PROP_OR(child, micro_step_res, \ - STEPPER_MICRO_STEP_256), \ - .index = DT_REG_ADDR(child), \ - .sg_threshold = DT_PROP_OR(child, stallguard2_threshold, 0), \ - .sg_threshold_velocity = DT_PROP_OR(child, stallguard_threshold_velocity, 0), \ - .sg_activation_delay_in_ms = DT_PROP_OR(child, \ - stallguard_activation_delay_ms, 100), \ - .is_sg_enabled = DT_PROP_OR(child, activate_stallguard2, false), \ + "stallguard threshold velocity must be a positive value"), ()); \ + IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, (CHECK_RAMP_DATA(child))); \ + static const struct tmc5041_stepper_config tmc5041_stepper_config_##child = { \ + .controller = DEVICE_DT_GET(DT_PARENT(child)), \ + .default_micro_step_res = \ + DT_PROP_OR(child, micro_step_res, STEPPER_MICRO_STEP_256), \ + .index = DT_REG_ADDR(child), \ + .sg_threshold = DT_PROP_OR(child, stallguard2_threshold, 0), \ + .sg_threshold_velocity = DT_PROP_OR(child, stallguard_threshold_velocity, 0), \ + .sg_activation_delay_in_ms = \ + DT_PROP_OR(child, stallguard_activation_delay_ms, 100), \ + .is_sg_enabled = DT_PROP_OR(child, activate_stallguard2, false), \ IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, \ (.default_ramp_config = TMC_RAMP_DT_SPEC_GET(child))) }; -#define TMC5041_STEPPER_DATA_DEFINE(child) \ - static struct tmc5041_stepper_data tmc5041_stepper_data_##child = { \ - .stepper = DEVICE_DT_GET(child),}; - -#define TMC5041_STEPPER_API_DEFINE(child) \ - static const struct stepper_driver_api tmc5041_stepper_api_##child = { \ - .enable = tmc5041_stepper_enable, \ - .is_moving = tmc5041_stepper_is_moving, \ - .move = tmc5041_stepper_move, \ - .set_max_velocity = tmc5041_stepper_set_max_velocity, \ - .set_micro_step_res = tmc5041_stepper_set_micro_step_res, \ - .get_micro_step_res = tmc5041_stepper_get_micro_step_res, \ - .set_actual_position = tmc5041_stepper_se
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c#L216
drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c:216 - &rampstat_value); + &rampstat_value);
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c#L255
drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c:255 - k_work_reschedule(&stepper_data->rampstat_callback_dwork, + k_work_reschedule( + &stepper_data->rampstat_callback_dwork,
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c#L647
drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c:647 - TMC5041_SWMODE(stepper_config->index), - BIT(10)); + TMC5041_SWMODE(stepper_config->index), BIT(10));
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c#L758
drivers/stepper/adi_tmc/adi_tmc5041_stepper_controller.c:758 - -#define TMC5041_SHAFT_CONFIG(child) \ - (DT_PROP(child, invert_direction) << TMC5041_GCONF_SHAFT_SHIFT(DT_REG_ADDR(child)))| - -#define TMC5041_STEPPER_CONFIG_DEFINE(child) \ +#define TMC5041_SHAFT_CONFIG(child) \ + (DT_PROP(child, invert_direction) << TMC5041_GCONF_SHAFT_SHIFT(DT_REG_ADDR(child))) | + +#define TMC5041_STEPPER_CONFIG_DEFINE(child) \ COND_CODE_1(DT_PROP(child, stallguard_threshold_velocity), \ BUILD_ASSERT(DT_PROP(child, stallguard_threshold_velocity), \ - "stallguard threshold velocity must be a positive value"), ()); \ - IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, (CHECK_RAMP_DATA(child))); \ - static const struct tmc5041_stepper_config tmc5041_stepper_config_##child = { \ - .controller = DEVICE_DT_GET(DT_PARENT(child)), \ - .default_micro_step_res = DT_PROP_OR(child, micro_step_res, \ - STEPPER_MICRO_STEP_256), \ - .index = DT_REG_ADDR(child), \ - .sg_threshold = DT_PROP_OR(child, stallguard2_threshold, 0), \ - .sg_threshold_velocity = DT_PROP_OR(child, stallguard_threshold_velocity, 0), \ - .sg_activation_delay_in_ms = DT_PROP_OR(child, \ - stallguard_activation_delay_ms, 100), \ - .is_sg_enabled = DT_PROP_OR(child, activate_stallguard2, false), \ + "stallguard threshold velocity must be a positive value"), ()); \ + IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, (CHECK_RAMP_DATA(child))); \ + static const struct tmc5041_stepper_config tmc5041_stepper_config_##child = { \ + .controller = DEVICE_DT_GET(DT_PARENT(child)), \ + .default_micro_step_res = \ + DT_PROP_OR(child, micro_step_res, STEPPER_MICRO_STEP_256), \ + .index = DT_REG_ADDR(child), \ + .sg_threshold = DT_PROP_OR(child, stallguard2_threshold, 0), \ + .sg_threshold_velocity = DT_PROP_OR(child, stallguard_threshold_velocity, 0), \ + .sg_activation_delay_in_ms = \ + DT_PROP_OR(child, stallguard_activation_delay_ms, 100), \ + .is_sg_enabled = DT_PROP_OR(child, activate_stallguard2, false), \ IF_ENABLED(CONFIG_STEPPER_ADI_TMC_RAMP_GEN, \ (.default_ramp_config = TMC_RAMP_DT_SPEC_GET(child))) }; -#define TMC5041_STEPPER_DATA_DEFINE(child) \ - static struct tmc5041_stepper_data tmc5041_stepper_data_##child = { \ - .stepper = DEVICE_DT_GET(child),}; - -#define TMC5041_STEPPER_API_DEFINE(child) \ - static const struct stepper_driver_api tmc5041_stepper_api_##child = { \ - .enable = tmc5041_stepper_enable, \ - .is_moving = tmc5041_stepper_is_moving, \ - .move = tmc5041_stepper_move, \ - .set_max_velocity = tmc5041_stepper_set_max_velocity, \ - .set_micro_step_res = tmc5041_stepper_set_micro_step_res, \ - .get_micro_step_res = tmc5041_stepper_get_micro_step_res, \ - .set_actual_position = tmc5041_stepper_set_actual_position, \ - .get_actual_position = tmc5041_stepper_get_actual_position, \ - .set_target_position = tmc5041_stepper_set_target_position, \ - .enable_constant_velocity_mode = tmc5041_stepper_enable_constant_velocity_mode, \ +#define TMC5041_STEPPER_DATA_DEFINE(child) \ + static struct tmc5041_stepper_data tmc5041_stepper_data_##child = { \ + .stepper = DEVICE_DT_GET(child), \ }; -#define TMC5041_STEPPER_DEFINE(child) \ - DEVICE_DT_DEFINE(child, tmc5041_stepper_init, NULL, &tmc5041_stepper_data_##child, \ - &tmc5041_stepper_config_##child, POST_KERNEL, \ +#define TMC5041_STEPPER_API_DEFINE(c
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc_reg.h#L141
drivers/stepper/adi_tmc/adi_tmc_reg.h:141 -#define TMC5041_WRITE_BIT 0x80U -#define TMC5041_ADDRESS_MASK 0x7FU +#define TMC5041_WRITE_BIT 0x80U +#define TMC5041_ADDRESS_MASK 0x7FU -#define TMC5041_GCONF_POSCMP_ENABLE_SHIFT 3 -#define TMC5041_GCONF_TEST_MODE_SHIFT 7 -#define TMC5041_GCONF_SHAFT_SHIFT(n) ((n)?8:9) -#define TMC5041_LOCK_GCONF_SHIFT 10 +#define TMC5041_GCONF_POSCMP_ENABLE_SHIFT 3 +#define TMC5041_GCONF_TEST_MODE_SHIFT 7 +#define TMC5041_GCONF_SHAFT_SHIFT(n) ((n) ? 8 : 9) +#define TMC5041_LOCK_GCONF_SHIFT 10 -#define TMC5041_GCONF 0x00 -#define TMC5041_GSTAT 0x01 -#define TMC5041_INPUT 0x04 -#define TMC5041_X_COMPARE 0x05 +#define TMC5041_GCONF 0x00 +#define TMC5041_GSTAT 0x01 +#define TMC5041_INPUT 0x04 +#define TMC5041_X_COMPARE 0x05 -#define TMC5041_PWMCONF(motor) (0x10 | TMC5041_MOTOR_ADDR_PWM(motor)) -#define TMC5041_PWM_STATUS(motor) (0x11 | TMC5041_MOTOR_ADDR_PWM(motor)) +#define TMC5041_PWMCONF(motor) (0x10 | TMC5041_MOTOR_ADDR_PWM(motor)) +#define TMC5041_PWM_STATUS(motor) (0x11 | TMC5041_MOTOR_ADDR_PWM(motor)) -#define TMC5041_RAMPMODE(motor) (0x00 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_XACTUAL(motor) (0x01 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VACTUAL(motor) (0x02 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VSTART(motor) (0x03 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_A1(motor) (0x04 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_V1(motor) (0x05 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_AMAX(motor) (0x06 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VMAX(motor) (0x07 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_DMAX(motor) (0x08 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_D1(motor) (0x0A | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VSTOP(motor) (0x0B | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_TZEROWAIT(motor) (0x0C | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_XTARGET(motor) (0x0D | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_IHOLD_IRUN(motor) (0x10 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VCOOLTHRS(motor) (0x11 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_VHIGH(motor) (0x12 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_SWMODE(motor) (0x14 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_RAMPSTAT(motor) (0x15 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_XLATCH(motor) (0x16 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_RAMPMODE(motor) (0x00 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_XACTUAL(motor) (0x01 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VACTUAL(motor) (0x02 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VSTART(motor) (0x03 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_A1(motor) (0x04 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_V1(motor) (0x05 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_AMAX(motor) (0x06 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VMAX(motor) (0x07 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_DMAX(motor) (0x08 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_D1(motor) (0x0A | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VSTOP(motor) (0x0B | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_TZEROWAIT(motor) (0x0C | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_XTARGET(motor) (0x0D | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_IHOLD_IRUN(motor) (0x10 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VCOOLTHRS(motor) (0x11 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_VHIGH(motor) (0x12 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_SWMODE(motor) (0x14 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_RAMPSTAT(motor) (0x15 | TMC5041_MOTOR_ADDR(motor)) +#define TMC5041_XLATCH(motor) (0x16 | TMC5041_MOTOR_ADDR(motor)) -#define TMC5041_MSLUT0(motor) (0x60 | TMC5041_MOTOR_ADDR_DRV(motor)) -#define TMC5041_MSLUT1(motor) (0x61 | TMC5041_MOTOR_ADDR_DRV(motor)) -#define TMC5041_MSLUT2(motor) (0x62 | TMC5041_MOTOR_ADDR_DRV(motor)) -#define TMC5041_MSLUT3(motor) (0x63 | TMC5041_MOTOR_ADDR_DRV(motor)) -#define TMC5041_MSLUT4(motor) (0x64
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc_spi.c#L11
drivers/stepper/adi_tmc/adi_tmc_spi.c:11 -#define BUFFER_SIZE 5U +#define BUFFER_SIZE 5U
You may want to run clang-format on this change: drivers/stepper/adi_tmc/adi_tmc_spi.h#L53
drivers/stepper/adi_tmc/adi_tmc_spi.h:53 -int tmc_spi_write_register(const struct spi_dt_spec *bus, const uint8_t write_bit, +int tmc_spi_write_register(const struct spi_dt_spec *bus, const uint8_t write_bit,
You may want to run clang-format on this change: include/zephyr/drivers/stepper/stepper_trinamic.h#L43
include/zephyr/drivers/stepper/stepper_trinamic.h:43 -#define TMC_RAMP_VSTART_MAX GENMASK(17, 0) -#define TMC_RAMP_VSTART_MIN 0 -#define TMC_RAMP_V1_MAX GENMASK(19, 0) -#define TMC_RAMP_V1_MIN 0 -#define TMC_RAMP_VMAX_MAX (GENMASK(22, 0) - 512) -#define TMC_RAMP_VMAX_MIN 0 -#define TMC_RAMP_A1_MAX GENMASK(15, 0) -#define TMC_RAMP_A1_MIN 0 -#define TMC_RAMP_AMAX_MAX GENMASK(15, 0) -#define TMC_RAMP_AMAX_MIN 0 -#define TMC_RAMP_D1_MAX GENMASK(15, 0) -#define TMC_RAMP_D1_MIN 1 -#define TMC_RAMP_DMAX_MAX GENMASK(15, 0) -#define TMC_RAMP_DMAX_MIN 0 -#define TMC_RAMP_VSTOP_MAX GENMASK(17, 0) -#define TMC_RAMP_VSTOP_MIN 1 -#define TMC_RAMP_TZEROWAIT_MAX (GENMASK(15, 0) - 512) -#define TMC_RAMP_TZEROWAIT_MIN 0 -#define TMC_RAMP_VCOOLTHRS_MAX GENMASK(22, 0) -#define TMC_RAMP_VCOOLTHRS_MIN 0 -#define TMC_RAMP_VHIGH_MAX GENMASK(22, 0) -#define TMC_RAMP_VHIGH_MIN 0 -#define TMC_RAMP_IHOLD_IRUN_MAX GENMASK(4, 0) -#define TMC_RAMP_IHOLD_IRUN_MIN 0 -#define TMC_RAMP_IHOLDDELAY_MAX GENMASK(3, 0) -#define TMC_RAMP_IHOLDDELAY_MIN 0 +#define TMC_RAMP_VSTART_MAX GENMASK(17, 0) +#define TMC_RAMP_VSTART_MIN 0 +#define TMC_RAMP_V1_MAX GENMASK(19, 0) +#define TMC_RAMP_V1_MIN 0 +#define TMC_RAMP_VMAX_MAX (GENMASK(22, 0) - 512) +#define TMC_RAMP_VMAX_MIN 0 +#define TMC_RAMP_A1_MAX GENMASK(15, 0) +#define TMC_RAMP_A1_MIN 0 +#define TMC_RAMP_AMAX_MAX GENMASK(15, 0) +#define TMC_RAMP_AMAX_MIN 0 +#define TMC_RAMP_D1_MAX GENMASK(15, 0) +#define TMC_RAMP_D1_MIN 1 +#define TMC_RAMP_DMAX_MAX GENMASK(15, 0) +#define TMC_RAMP_DMAX_MIN 0 +#define TMC_RAMP_VSTOP_MAX GENMASK(17, 0) +#define TMC_RAMP_VSTOP_MIN 1 +#define TMC_RAMP_TZEROWAIT_MAX (GENMASK(15, 0) - 512) +#define TMC_RAMP_TZEROWAIT_MIN 0 +#define TMC_RAMP_VCOOLTHRS_MAX GENMASK(22, 0) +#define TMC_RAMP_VCOOLTHRS_MIN 0 +#define TMC_RAMP_VHIGH_MAX GENMASK(22, 0) +#define TMC_RAMP_VHIGH_MIN 0 +#define TMC_RAMP_IHOLD_IRUN_MAX GENMASK(4, 0) +#define TMC_RAMP_IHOLD_IRUN_MIN 0 +#define TMC_RAMP_IHOLDDELAY_MAX GENMASK(3, 0) +#define TMC_RAMP_IHOLDDELAY_MIN 0
You may want to run clang-format on this change: include/zephyr/drivers/stepper/stepper_trinamic.h#L106
include/zephyr/drivers/stepper/stepper_trinamic.h:106 -#define CHECK_RAMP_DATA(node) \ +#define CHECK_RAMP_DATA(node) \ COND_CODE_1(DT_PROP(node, vstart), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, vstart), TMC_RAMP_VSTART_MIN, \ - TMC_RAMP_VSTART_MAX), "vstart out of range"), ()); \ + TMC_RAMP_VSTART_MAX), "vstart out of range"), ()); \ COND_CODE_1(DT_PROP(node, v1), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, v1), TMC_RAMP_V1_MIN, \ - TMC_RAMP_V1_MAX), "v1 out of range"), ()); \ + TMC_RAMP_V1_MAX), "v1 out of range"), ()); \ COND_CODE_1(DT_PROP(node, vmax), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, vmax), TMC_RAMP_VMAX_MIN, \ - TMC_RAMP_VMAX_MAX), "vmax out of range"), ()); \ + TMC_RAMP_VMAX_MAX), "vmax out of range"), ()); \ COND_CODE_1(DT_PROP(node, a1), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, a1), TMC_RAMP_A1_MIN, \ - TMC_RAMP_A1_MAX), "a1 out of range"), ()); \ + TMC_RAMP_A1_MAX), "a1 out of range"), ()); \ COND_CODE_1(DT_PROP(node, amax), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, amax), TMC_RAMP_AMAX_MIN, \ - TMC_RAMP_AMAX_MAX), "amax out of range"), ()); \ + TMC_RAMP_AMAX_MAX), "amax out of range"), ()); \ COND_CODE_1(DT_PROP(node, d1), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, d1), TMC_RAMP_D1_MIN, \ - TMC_RAMP_D1_MAX), "d1 out of range"), ()); \ + TMC_RAMP_D1_MAX), "d1 out of range"), ()); \ COND_CODE_1(DT_PROP(node, dmax), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, dmax), TMC_RAMP_DMAX_MIN, \ - TMC_RAMP_DMAX_MAX), "dmax out of range"), ()); \ + TMC_RAMP_DMAX_MAX), "dmax out of range"), ()); \ COND_CODE_1(DT_PROP(node, vstop), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, vstop), TMC_RAMP_VSTOP_MIN, \ - TMC_RAMP_VSTOP_MAX), "vstop out of range"), ()); \ + TMC_RAMP_VSTOP_MAX), "vstop out of range"), ()); \ COND_CODE_1(DT_PROP(node, tzerowait), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, tzerowait), TMC_RAMP_TZEROWAIT_MIN, \ - TMC_RAMP_TZEROWAIT_MAX), "tzerowait out of range"), ()); \ + TMC_RAMP_TZEROWAIT_MAX), "tzerowait out of range"), ()); \ COND_CODE_1(DT_PROP(node, vcoolthrs), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, vcoolthrs), TMC_RAMP_VCOOLTHRS_MIN, \ - TMC_RAMP_VCOOLTHRS_MAX), "vcoolthrs out of range"), ()); \ + TMC_RAMP_VCOOLTHRS_MAX), "vcoolthrs out of range"), ()); \ COND_CODE_1(DT_PROP(node, vhigh), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, vhigh), TMC_RAMP_VHIGH_MIN, \ - TMC_RAMP_VHIGH_MAX), "vhigh out of range"), ()); \ + TMC_RAMP_VHIGH_MAX), "vhigh out of range"), ()); \ COND_CODE_1(DT_PROP(node, ihold), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, ihold), TMC_RAMP_IHOLD_IRUN_MIN, \ - TMC_RAMP_IHOLD_IRUN_MAX), "ihold out of range"), ()); \ + TMC_RAMP_IHOLD_IRUN_MAX), "ihold out of range"), ()); \ COND_CODE_1(DT_PROP(node, irun), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, irun), TMC_RAMP_IHOLD_IRUN_MIN, \ - TMC_RAMP_IHOLD_IRUN_MAX), "irun out of range"), ()); \ + TMC_RAMP_IHOLD_IRUN_MAX), "irun out of range"), ()); \ COND_CODE_1(DT_PROP(node, iholddelay), \ BUILD_ASSERT(IN_RANGE(DT_PROP(node, iholddelay), TMC_RAMP_IHOLDDELAY_MIN, \ TMC_RAM
You may want to run clang-format on this change: include/zephyr/drivers/stepper/stepper_trinamic.h#L131
include/zephyr/drivers/stepper/stepper_trinamic.h:131 -#define TMC_RAMP_DT_SPEC_GET(node) \ - { \ - .vstart = DT_PROP_OR(node, vstart, TMC_RAMP_VSTART_MIN), \ - .v1 = DT_PROP_OR(node, v1, TMC_RAMP_V1_MIN), \ - .vmax = DT_PROP_OR(node, vmax, TMC_RAMP_VMAX_MIN), \ - .a1 = DT_PROP_OR(node, a1, TMC_RAMP_A1_MIN), \ - .amax = DT_PROP_OR(node, amax, TMC_RAMP_AMAX_MIN), \ - .d1 = DT_PROP_OR(node, d1, TMC_RAMP_D1_MIN), \ - .dmax = DT_PROP_OR(node, dmax, TMC_RAMP_DMAX_MIN), \ - .vstop = DT_PROP_OR(node, vstop, TMC_RAMP_VSTOP_MIN), \ - .tzerowait = DT_PROP_OR(node, tzerowait, TMC_RAMP_TZEROWAIT_MIN), \ - .vcoolthrs = DT_PROP_OR(node, vcoolthrs, TMC_RAMP_VCOOLTHRS_MIN), \ - .vhigh = DT_PROP_OR(node, vhigh, TMC_RAMP_VCOOLTHRS_MIN), \ - .iholdrun = (TMC5041_IRUN(DT_PROP_OR(node, irun, TMC_RAMP_IHOLD_IRUN_MIN)) | \ - TMC5041_IHOLD(DT_PROP_OR(node, ihold, TMC_RAMP_IHOLD_IRUN_MIN)) | \ - TMC5041_IHOLDDELAY(DT_PROP_OR(node, iholddelay, \ - TMC_RAMP_IHOLDDELAY_MIN))), \ +#define TMC_RAMP_DT_SPEC_GET(node) \ + { \ + .vstart = DT_PROP_OR(node, vstart, TMC_RAMP_VSTART_MIN), \ + .v1 = DT_PROP_OR(node, v1, TMC_RAMP_V1_MIN), \ + .vmax = DT_PROP_OR(node, vmax, TMC_RAMP_VMAX_MIN), \ + .a1 = DT_PROP_OR(node, a1, TMC_RAMP_A1_MIN), \ + .amax = DT_PROP_OR(node, amax, TMC_RAMP_AMAX_MIN), \ + .d1 = DT_PROP_OR(node, d1, TMC_RAMP_D1_MIN), \ + .dmax = DT_PROP_OR(node, dmax, TMC_RAMP_DMAX_MIN), \ + .vstop = DT_PROP_OR(node, vstop, TMC_RAMP_VSTOP_MIN), \ + .tzerowait = DT_PROP_OR(node, tzerowait, TMC_RAMP_TZEROWAIT_MIN), \ + .vcoolthrs = DT_PROP_OR(node, vcoolthrs, TMC_RAMP_VCOOLTHRS_MIN), \ + .vhigh = DT_PROP_OR(node, vhigh, TMC_RAMP_VCOOLTHRS_MIN), \ + .iholdrun = (TMC5041_IRUN(DT_PROP_OR(node, irun, TMC_RAMP_IHOLD_IRUN_MIN)) | \ + TMC5041_IHOLD(DT_PROP_OR(node, ihold, TMC_RAMP_IHOLD_IRUN_MIN)) | \ + TMC5041_IHOLDDELAY( \ + DT_PROP_OR(node, iholddelay, TMC_RAMP_IHOLDDELAY_MIN))), \

Artifacts

Produced during runtime
Name Size
compliance.xml
5.27 KB