Skip to content

Commit f8f6004

Browse files
faxe1008kartben
authored andcommitted
include: drivers: stepper: document stepper driver ops using Doxygen
Use doxygen driver_ops commands to properly document the required/optional stepper driver operations. Signed-off-by: Fabian Blatz <fabianblatz@gmail.com>
1 parent 7408959 commit f8f6004

2 files changed

Lines changed: 79 additions & 43 deletions

File tree

include/zephyr/drivers/stepper/stepper.h

Lines changed: 39 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -82,69 +82,77 @@ enum stepper_event {
8282
};
8383

8484
/**
85-
* @cond INTERNAL_HIDDEN
86-
*
87-
* Stepper hardware driver API definition and system call entry points.
88-
*
85+
* @brief Callback function for stepper driver events
8986
*/
87+
typedef void (*stepper_event_cb_t)(const struct device *dev, const enum stepper_event event,
88+
void *user_data);
9089

9190
/**
92-
* @brief Enable the stepper hardware driver
93-
*
94-
* @see stepper_enable() for details.
91+
* @def_driverbackendgroup{Stepper Hardware Driver,stepper_hw_driver}
92+
* @{
9593
*/
96-
typedef int (*stepper_enable_t)(const struct device *dev);
9794

9895
/**
99-
* @brief Disable the stepper hardware driver
100-
*
101-
* @see stepper_disable() for details.
96+
* @brief Callback API to enable the stepper hardware driver.
97+
* See stepper_enable() for argument description.
10298
*/
103-
typedef int (*stepper_disable_t)(const struct device *dev);
99+
typedef int (*stepper_enable_t)(const struct device *dev);
104100

105101
/**
106-
* @brief Set the stepper micro-step resolution
107-
*
108-
* @see stepper_set_micro_step_res() for details.
102+
* @brief Callback API to disable the stepper hardware driver.
103+
* See stepper_disable() for argument description.
109104
*/
110-
typedef int (*stepper_set_micro_step_res_t)(
111-
const struct device *dev, const enum stepper_micro_step_resolution resolution);
105+
typedef int (*stepper_disable_t)(const struct device *dev);
112106

113107
/**
114-
* @brief Get the stepper micro-step resolution
115-
*
116-
* @see stepper_get_micro_step_res() for details.
108+
* @brief Callback API to set the stepper micro-step resolution.
109+
* See stepper_set_micro_step_res() for argument description.
117110
*/
118-
typedef int (*stepper_get_micro_step_res_t)(const struct device *dev,
119-
enum stepper_micro_step_resolution *resolution);
111+
typedef int (*stepper_set_micro_step_res_t)(const struct device *dev,
112+
const enum stepper_micro_step_resolution resolution);
120113

121114
/**
122-
* @brief Callback function for stepper driver events
115+
* @brief Callback API to get the stepper micro-step resolution.
116+
* See stepper_get_micro_step_res() for argument description.
123117
*/
124-
typedef void (*stepper_event_cb_t)(const struct device *dev, const enum stepper_event event,
125-
void *user_data);
118+
typedef int (*stepper_get_micro_step_res_t)(const struct device *dev,
119+
enum stepper_micro_step_resolution *resolution);
126120

127121
/**
128-
* @brief Set the callback function to be called when a stepper_event occurs
129-
*
130-
* @see stepper_set_event_cb() for details.
122+
* @brief Callback API to set the event callback function.
123+
* See stepper_set_event_cb() for argument description.
131124
*/
132-
typedef int (*stepper_set_event_cb_t)(const struct device *dev,
133-
stepper_event_cb_t callback, void *user_data);
125+
typedef int (*stepper_set_event_cb_t)(const struct device *dev, stepper_event_cb_t callback,
126+
void *user_data);
134127

135128
/**
136129
* @driver_ops{Stepper Hardware Driver}
137130
*/
138131
__subsystem struct stepper_driver_api {
132+
/**
133+
* @driver_ops_mandatory @copybrief stepper_enable
134+
*/
139135
stepper_enable_t enable;
136+
/**
137+
* @driver_ops_mandatory @copybrief stepper_disable
138+
*/
140139
stepper_disable_t disable;
140+
/**
141+
* @driver_ops_mandatory @copybrief stepper_set_micro_step_res
142+
*/
141143
stepper_set_micro_step_res_t set_micro_step_res;
144+
/**
145+
* @driver_ops_mandatory @copybrief stepper_get_micro_step_res
146+
*/
142147
stepper_get_micro_step_res_t get_micro_step_res;
148+
/**
149+
* @driver_ops_optional @copybrief stepper_set_event_cb
150+
*/
143151
stepper_set_event_cb_t set_event_cb;
144152
};
145153

146154
/**
147-
* @endcond
155+
* @}
148156
*/
149157

150158
/**

include/zephyr/drivers/stepper/stepper_ctrl.h

Lines changed: 40 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,14 @@ struct stepper_ctrl_ramp {
7878
};
7979

8080
/**
81-
* @cond INTERNAL_HIDDEN
82-
*
83-
* Stepper Motion Controller driver API definition and entry points.
84-
*
81+
* @brief Callback function for stepper motion controller events
82+
*/
83+
typedef void (*stepper_ctrl_event_callback_t)(const struct device *dev,
84+
const enum stepper_ctrl_event event, void *user_data);
85+
86+
/**
87+
* @def_driverbackendgroup{Stepper Motion Controller,stepper_ctrl}
88+
* @{
8589
*/
8690

8791
/**
@@ -98,12 +102,6 @@ typedef int (*stepper_ctrl_set_reference_position_t)(const struct device *dev, c
98102
*/
99103
typedef int (*stepper_ctrl_get_actual_position_t)(const struct device *dev, int32_t *value);
100104

101-
/**
102-
* @brief Callback function for stepper motion controller events
103-
*/
104-
typedef void (*stepper_ctrl_event_callback_t)(const struct device *dev,
105-
const enum stepper_ctrl_event event, void *user_data);
106-
107105
/**
108106
* @brief Set the callback function to be called when a stepper motion controller event occurs
109107
*
@@ -165,23 +163,53 @@ typedef int (*stepper_ctrl_stop_t)(const struct device *dev);
165163
typedef int (*stepper_ctrl_is_moving_t)(const struct device *dev, bool *is_moving);
166164

167165
/**
168-
* @driver_ops{Stepper Motion Controller Driver API}
166+
* @driver_ops{Stepper Motion Controller}
169167
*/
170168
__subsystem struct stepper_ctrl_driver_api {
169+
/**
170+
* @driver_ops_optional @copybrief stepper_ctrl_set_reference_position
171+
*/
171172
stepper_ctrl_set_reference_position_t set_reference_position;
173+
/**
174+
* @driver_ops_optional @copybrief stepper_ctrl_get_actual_position
175+
*/
172176
stepper_ctrl_get_actual_position_t get_actual_position;
177+
/**
178+
* @driver_ops_optional @copybrief stepper_ctrl_set_event_cb
179+
*/
173180
stepper_ctrl_set_event_cb_t set_event_cb;
181+
/**
182+
* @driver_ops_optional @copybrief stepper_ctrl_set_microstep_interval
183+
*/
174184
stepper_ctrl_set_microstep_interval_t set_microstep_interval;
185+
/**
186+
* @driver_ops_optional @copybrief stepper_ctrl_configure_ramp
187+
*/
175188
stepper_ctrl_configure_ramp_t configure_ramp;
189+
/**
190+
* @driver_ops_mandatory @copybrief stepper_ctrl_move_by
191+
*/
176192
stepper_ctrl_move_by_t move_by;
193+
/**
194+
* @driver_ops_mandatory @copybrief stepper_ctrl_move_to
195+
*/
177196
stepper_ctrl_move_to_t move_to;
197+
/**
198+
* @driver_ops_optional @copybrief stepper_ctrl_run
199+
*/
178200
stepper_ctrl_run_t run;
201+
/**
202+
* @driver_ops_optional @copybrief stepper_ctrl_stop
203+
*/
179204
stepper_ctrl_stop_t stop;
205+
/**
206+
* @driver_ops_optional @copybrief stepper_ctrl_is_moving
207+
*/
180208
stepper_ctrl_is_moving_t is_moving;
181209
};
182210

183211
/**
184-
* @endcond
212+
* @}
185213
*/
186214

187215
/**

0 commit comments

Comments
 (0)