* [PATCH 0/6] pwm: New abstraction and userspace API
@ 2024-07-08 10:52 Uwe Kleine-König
2024-07-08 10:52 ` [PATCH 4/6] pwm: Add tracing for waveform callbacks Uwe Kleine-König
0 siblings, 1 reply; 4+ messages in thread
From: Uwe Kleine-König @ 2024-07-08 10:52 UTC (permalink / raw)
To: linux-pwm
Cc: Steven Rostedt, Masami Hiramatsu, Mathieu Desnoyers,
linux-trace-kernel, Michael Hennerich, Nuno Sá,
Fabrice Gasnier, Maxime Coquelin, Alexandre Torgue, linux-stm32,
linux-arm-kernel
Hello,
this series implements a new abstraction to model the output waveform of
a PWM. The main improvement is that it defines a duty_offset instead of
a polarity and so allows to model more wave forms. The motivation for
this is that we need a PWM channel to have an offset compared to another
channel from the same chip, that is something like that:
__ __ __
/ \_______________/ \_______________/ \_________
^ __ ^ __ ^ __
____/ \_______________/ \_______________/ \_____
^ ^ ^
The kernel API for that is still missing (so it cannot be used yet from
the iio driver we intend to use it), but there is a userspace API that
makes use of it.
This is actually the 2nd series that implements a userspace API using a
chardev, the userspace lib from the last iteration is updated
accordingly. See
https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/libpwm.git
The series bases on top of
https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next
plus the patch
pwm: Make info in traces about affected pwm more useful
available at
https://lore.kernel.org/linux-pwm/20240705211452.1157967-2-u.kleine-koenig@baylibre.com/
.
Unfortunately this requires a new set of callbacks for lowlevel drivers.
In this series axi-pwmgen and stm32 are converted accordingly.
The documentation situation is not optimal yet, and I expect that the
locking patch triggers a lockdep warning for the meson driver. This is
however a false positive and a problem that needs addressing in the clk
subsystem.
Looking forward to your feedback,
Uwe Kleine-König
Uwe Kleine-König (6):
pwm: Add more locking
pwm: New abstraction for PWM waveforms
pwm: Add support for pwmchip devices for faster and easier userspace
access
pwm: Add tracing for waveform callbacks
pwm: axi-pwmgen: Implementation of the waveform callbacks
pwm: stm32: Implementation of the waveform callbacks
drivers/pwm/core.c | 678 +++++++++++++++++++++++++++++++++--
drivers/pwm/pwm-axi-pwmgen.c | 148 +++++---
drivers/pwm/pwm-stm32.c | 605 +++++++++++++++++++------------
include/linux/pwm.h | 51 +++
include/trace/events/pwm.h | 134 ++++++-
include/uapi/linux/pwm.h | 24 ++
6 files changed, 1334 insertions(+), 306 deletions(-)
create mode 100644 include/uapi/linux/pwm.h
base-commit: 120a528213b6693214e3cbc24a9c3052a4b1024b
prerequisite-patch-id: 0e21153cd012f41ba9db52357fd08219af53e26c
--
2.43.0
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH 4/6] pwm: Add tracing for waveform callbacks 2024-07-08 10:52 [PATCH 0/6] pwm: New abstraction and userspace API Uwe Kleine-König @ 2024-07-08 10:52 ` Uwe Kleine-König 2024-07-08 18:14 ` Trevor Gamblin 0 siblings, 1 reply; 4+ messages in thread From: Uwe Kleine-König @ 2024-07-08 10:52 UTC (permalink / raw) To: linux-pwm Cc: Steven Rostedt, Masami Hiramatsu, Mathieu Desnoyers, linux-trace-kernel --- drivers/pwm/core.c | 68 ++++++++++++++++--- include/trace/events/pwm.h | 134 ++++++++++++++++++++++++++++++++++--- 2 files changed, 183 insertions(+), 19 deletions(-) diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c index d64c033c4cb2..a2320ae77220 100644 --- a/drivers/pwm/core.c +++ b/drivers/pwm/core.c @@ -138,6 +138,52 @@ static int pwm_check_rounding(const struct pwm_waveform *wf, return 0; } +static int pwm_round_waveform_tohw(struct pwm_chip *chip, struct pwm_device *pwm, + const struct pwm_waveform *wf, void *wfhw) +{ + const struct pwm_ops *ops = chip->ops; + int ret; + + ret = ops->round_waveform_tohw(chip, pwm, wf, wfhw); + trace_pwm_round_waveform_tohw(pwm, wf, wfhw, ret); + + return ret; +} + +static int pwm_round_waveform_fromhw(struct pwm_chip *chip, struct pwm_device *pwm, + const void *wfhw, struct pwm_waveform *wf) +{ + const struct pwm_ops *ops = chip->ops; + int ret; + + ret = ops->round_waveform_fromhw(chip, pwm, wfhw, wf); + trace_pwm_round_waveform_fromhw(pwm, wfhw, wf, ret); + + return ret; +} + +static int pwm_read_waveform(struct pwm_chip *chip, struct pwm_device *pwm, void *wfhw) +{ + const struct pwm_ops *ops = chip->ops; + int ret; + + ret = ops->read_waveform(chip, pwm, wfhw); + trace_pwm_read_waveform(pwm, wfhw, ret); + + return ret; +} + +static int pwm_write_waveform(struct pwm_chip *chip, struct pwm_device *pwm, const void *wfhw) +{ + const struct pwm_ops *ops = chip->ops; + int ret; + + ret = ops->write_waveform(chip, pwm, wfhw); + trace_pwm_write_waveform(pwm, wfhw, ret); + + return ret; +} + #define WFHWSIZE 20 static int pwm_get_waveform(struct pwm_device *pwm, @@ -155,11 +201,11 @@ static int pwm_get_waveform(struct pwm_device *pwm, if (!chip->operational) return -ENODEV; - err = ops->read_waveform(chip, pwm, &wfhw); + err = pwm_read_waveform(chip, pwm, &wfhw); if (err) return err; - return ops->round_waveform_fromhw(chip, pwm, &wfhw, wf); + return pwm_round_waveform_fromhw(chip, pwm, &wfhw, wf); } /* Called with the pwmchip lock held */ @@ -180,12 +226,12 @@ static int __pwm_set_waveform(struct pwm_device *pwm, wf->duty_offset >= wf->period_length)) return -EINVAL; - err = ops->round_waveform_tohw(chip, pwm, wf, &wfhw); + err = pwm_round_waveform_tohw(chip, pwm, wf, &wfhw); if (err) return err; if ((IS_ENABLED(CONFIG_PWM_DEBUG) || exact) && wf->period_length) { - err = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf_rounded); + err = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf_rounded); if (err) return err; @@ -203,7 +249,7 @@ static int __pwm_set_waveform(struct pwm_device *pwm, } } - err = ops->write_waveform(chip, pwm, &wfhw); + err = pwm_write_waveform(chip, pwm, &wfhw); if (err) return err; @@ -213,12 +259,12 @@ static int __pwm_set_waveform(struct pwm_device *pwm, if (IS_ENABLED(CONFIG_PWM_DEBUG) && ops->read_waveform && wf->period_length) { struct pwm_waveform wf_set; - err = ops->read_waveform(chip, pwm, &wfhw); + err = pwm_read_waveform(chip, pwm, &wfhw); if (err) /* maybe ignore? */ return err; - err = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf_set); + err = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf_set); if (err) /* maybe ignore? */ return err; @@ -443,11 +489,11 @@ static int __pwm_apply(struct pwm_device *pwm, const struct pwm_state *state) * high. */ - err = ops->round_waveform_tohw(chip, pwm, &wf, &wfhw); + err = pwm_round_waveform_tohw(chip, pwm, &wf, &wfhw); if (err) return err; - err = ops->write_waveform(chip, pwm, &wfhw); + err = pwm_write_waveform(chip, pwm, &wfhw); if (err) return err; @@ -548,11 +594,11 @@ static int pwm_get_state_hw(struct pwm_device *pwm, struct pwm_state *state) scoped_guard(pwmchip, chip) { - ret = ops->read_waveform(chip, pwm, &wfhw); + ret = pwm_read_waveform(chip, pwm, &wfhw); if (ret) return ret; - ret = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf); + ret = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf); if (ret) return ret; } diff --git a/include/trace/events/pwm.h b/include/trace/events/pwm.h index 8022701c446d..3b5b20d2aff0 100644 --- a/include/trace/events/pwm.h +++ b/include/trace/events/pwm.h @@ -8,15 +8,135 @@ #include <linux/pwm.h> #include <linux/tracepoint.h> +#define TP_PROTO_pwm(args...) \ + TP_PROTO(struct pwm_device *pwm, args) + +#define TP_ARGS_pwm(args...) \ + TP_ARGS(pwm, args) + +#define TP_STRUCT__entry_pwm(args...) \ + TP_STRUCT__entry( \ + __field(unsigned int, chipid) \ + __field(unsigned int, hwpwm) \ + args) + +#define TP_fast_assign_pwm(args...) \ + TP_fast_assign( \ + __entry->chipid = pwm->chip->id; \ + __entry->hwpwm = pwm->hwpwm; \ + args) + +#define TP_printk_pwm(fmt, args...) \ + TP_printk("pwmchip%u.%u: " fmt, __entry->chipid, __entry->hwpwm, args) + +#define __field_pwmwf(wf) \ + __field(u64, wf ## _period_length) \ + __field(u64, wf ## _duty_length) \ + __field(u64, wf ## _duty_offset) \ + +#define fast_assign_pwmwf(wf) \ + __entry->wf ## _period_length = wf->period_length; \ + __entry->wf ## _duty_length = wf->duty_length; \ + __entry->wf ## _duty_offset = wf->duty_offset + +#define printk_pwmwf_format(wf) \ + "%lld/%lld [+%lld]" + +#define printk_pwmwf_formatargs(wf) \ + __entry->wf ## _duty_length, __entry->wf ## _period_length, __entry->wf ## _duty_offset + +TRACE_EVENT(pwm_round_waveform_tohw, + + TP_PROTO_pwm(const struct pwm_waveform *wf, void *wfhw, int err), + + TP_ARGS_pwm(wf, wfhw, err), + + TP_STRUCT__entry_pwm( + __field_pwmwf(wf) + __field(void *, wfhw) + __field(int, err) + ), + + TP_fast_assign_pwm( + fast_assign_pwmwf(wf); + __entry->wfhw = wfhw; + __entry->err = err; + ), + + TP_printk_pwm(printk_pwmwf_format(wf) " > %p err=%d", + printk_pwmwf_formatargs(wf), __entry->wfhw, __entry->err) +); + +TRACE_EVENT(pwm_round_waveform_fromhw, + + TP_PROTO_pwm(const void *wfhw, struct pwm_waveform *wf, int err), + + TP_ARGS_pwm(wfhw, wf, err), + + TP_STRUCT__entry_pwm( + __field(const void *, wfhw) + __field_pwmwf(wf) + __field(int, err) + ), + + TP_fast_assign_pwm( + __entry->wfhw = wfhw; + fast_assign_pwmwf(wf); + __entry->err = err; + ), + + TP_printk_pwm("%p > " printk_pwmwf_format(wf) " err=%d", + __entry->wfhw, printk_pwmwf_formatargs(wf), __entry->err) +); + +TRACE_EVENT(pwm_read_waveform, + + TP_PROTO_pwm(void *wfhw, int err), + + TP_ARGS_pwm(wfhw, err), + + TP_STRUCT__entry_pwm( + __field(void *, wfhw) + __field(int, err) + ), + + TP_fast_assign_pwm( + __entry->wfhw = wfhw; + __entry->err = err; + ), + + TP_printk_pwm("%p err=%d", + __entry->wfhw, __entry->err) +); + +TRACE_EVENT(pwm_write_waveform, + + TP_PROTO_pwm(const void *wfhw, int err), + + TP_ARGS_pwm(wfhw, err), + + TP_STRUCT__entry_pwm( + __field(const void *, wfhw) + __field(int, err) + ), + + TP_fast_assign_pwm( + __entry->wfhw = wfhw; + __entry->err = err; + ), + + TP_printk_pwm("%p err=%d", + __entry->wfhw, __entry->err) +); + + DECLARE_EVENT_CLASS(pwm, TP_PROTO(struct pwm_device *pwm, const struct pwm_state *state, int err), TP_ARGS(pwm, state, err), - TP_STRUCT__entry( - __field(unsigned int, chipid) - __field(unsigned int, hwpwm) + TP_STRUCT__entry_pwm( __field(u64, period) __field(u64, duty_cycle) __field(enum pwm_polarity, polarity) @@ -24,9 +144,7 @@ DECLARE_EVENT_CLASS(pwm, __field(int, err) ), - TP_fast_assign( - __entry->chipid = pwm->chip->id; - __entry->hwpwm = pwm->hwpwm; + TP_fast_assign_pwm( __entry->period = state->period; __entry->duty_cycle = state->duty_cycle; __entry->polarity = state->polarity; @@ -34,8 +152,8 @@ DECLARE_EVENT_CLASS(pwm, __entry->err = err; ), - TP_printk("pwmchip%u.%u: period=%llu duty_cycle=%llu polarity=%d enabled=%d err=%d", - __entry->chipid, __entry->hwpwm, __entry->period, __entry->duty_cycle, + TP_printk_pwm("period=%llu duty_cycle=%llu polarity=%d enabled=%d err=%d", + __entry->period, __entry->duty_cycle, __entry->polarity, __entry->enabled, __entry->err) ); -- 2.43.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 4/6] pwm: Add tracing for waveform callbacks 2024-07-08 10:52 ` [PATCH 4/6] pwm: Add tracing for waveform callbacks Uwe Kleine-König @ 2024-07-08 18:14 ` Trevor Gamblin 2024-07-09 6:54 ` Uwe Kleine-König 0 siblings, 1 reply; 4+ messages in thread From: Trevor Gamblin @ 2024-07-08 18:14 UTC (permalink / raw) To: Uwe Kleine-König, linux-pwm Cc: Steven Rostedt, Masami Hiramatsu, Mathieu Desnoyers, linux-trace-kernel Missing a sign-off? On 2024-07-08 6:52 a.m., Uwe Kleine-König wrote: > --- > drivers/pwm/core.c | 68 ++++++++++++++++--- > include/trace/events/pwm.h | 134 ++++++++++++++++++++++++++++++++++--- > 2 files changed, 183 insertions(+), 19 deletions(-) > > diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c > index d64c033c4cb2..a2320ae77220 100644 > --- a/drivers/pwm/core.c > +++ b/drivers/pwm/core.c > @@ -138,6 +138,52 @@ static int pwm_check_rounding(const struct pwm_waveform *wf, > return 0; > } > > +static int pwm_round_waveform_tohw(struct pwm_chip *chip, struct pwm_device *pwm, > + const struct pwm_waveform *wf, void *wfhw) > +{ > + const struct pwm_ops *ops = chip->ops; > + int ret; > + > + ret = ops->round_waveform_tohw(chip, pwm, wf, wfhw); > + trace_pwm_round_waveform_tohw(pwm, wf, wfhw, ret); > + > + return ret; > +} > + > +static int pwm_round_waveform_fromhw(struct pwm_chip *chip, struct pwm_device *pwm, > + const void *wfhw, struct pwm_waveform *wf) > +{ > + const struct pwm_ops *ops = chip->ops; > + int ret; > + > + ret = ops->round_waveform_fromhw(chip, pwm, wfhw, wf); > + trace_pwm_round_waveform_fromhw(pwm, wfhw, wf, ret); > + > + return ret; > +} > + > +static int pwm_read_waveform(struct pwm_chip *chip, struct pwm_device *pwm, void *wfhw) > +{ > + const struct pwm_ops *ops = chip->ops; > + int ret; > + > + ret = ops->read_waveform(chip, pwm, wfhw); > + trace_pwm_read_waveform(pwm, wfhw, ret); > + > + return ret; > +} > + > +static int pwm_write_waveform(struct pwm_chip *chip, struct pwm_device *pwm, const void *wfhw) > +{ > + const struct pwm_ops *ops = chip->ops; > + int ret; > + > + ret = ops->write_waveform(chip, pwm, wfhw); > + trace_pwm_write_waveform(pwm, wfhw, ret); > + > + return ret; > +} > + > #define WFHWSIZE 20 > > static int pwm_get_waveform(struct pwm_device *pwm, > @@ -155,11 +201,11 @@ static int pwm_get_waveform(struct pwm_device *pwm, > if (!chip->operational) > return -ENODEV; > > - err = ops->read_waveform(chip, pwm, &wfhw); > + err = pwm_read_waveform(chip, pwm, &wfhw); > if (err) > return err; > > - return ops->round_waveform_fromhw(chip, pwm, &wfhw, wf); > + return pwm_round_waveform_fromhw(chip, pwm, &wfhw, wf); > } > > /* Called with the pwmchip lock held */ > @@ -180,12 +226,12 @@ static int __pwm_set_waveform(struct pwm_device *pwm, > wf->duty_offset >= wf->period_length)) > return -EINVAL; > > - err = ops->round_waveform_tohw(chip, pwm, wf, &wfhw); > + err = pwm_round_waveform_tohw(chip, pwm, wf, &wfhw); > if (err) > return err; > > if ((IS_ENABLED(CONFIG_PWM_DEBUG) || exact) && wf->period_length) { > - err = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf_rounded); > + err = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf_rounded); > if (err) > return err; > > @@ -203,7 +249,7 @@ static int __pwm_set_waveform(struct pwm_device *pwm, > } > } > > - err = ops->write_waveform(chip, pwm, &wfhw); > + err = pwm_write_waveform(chip, pwm, &wfhw); > if (err) > return err; > > @@ -213,12 +259,12 @@ static int __pwm_set_waveform(struct pwm_device *pwm, > if (IS_ENABLED(CONFIG_PWM_DEBUG) && ops->read_waveform && wf->period_length) { > struct pwm_waveform wf_set; > > - err = ops->read_waveform(chip, pwm, &wfhw); > + err = pwm_read_waveform(chip, pwm, &wfhw); > if (err) > /* maybe ignore? */ > return err; > > - err = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf_set); > + err = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf_set); > if (err) > /* maybe ignore? */ > return err; > @@ -443,11 +489,11 @@ static int __pwm_apply(struct pwm_device *pwm, const struct pwm_state *state) > * high. > */ > > - err = ops->round_waveform_tohw(chip, pwm, &wf, &wfhw); > + err = pwm_round_waveform_tohw(chip, pwm, &wf, &wfhw); > if (err) > return err; > > - err = ops->write_waveform(chip, pwm, &wfhw); > + err = pwm_write_waveform(chip, pwm, &wfhw); > if (err) > return err; > > @@ -548,11 +594,11 @@ static int pwm_get_state_hw(struct pwm_device *pwm, struct pwm_state *state) > > scoped_guard(pwmchip, chip) { > > - ret = ops->read_waveform(chip, pwm, &wfhw); > + ret = pwm_read_waveform(chip, pwm, &wfhw); > if (ret) > return ret; > > - ret = ops->round_waveform_fromhw(chip, pwm, &wfhw, &wf); > + ret = pwm_round_waveform_fromhw(chip, pwm, &wfhw, &wf); > if (ret) > return ret; > } > diff --git a/include/trace/events/pwm.h b/include/trace/events/pwm.h > index 8022701c446d..3b5b20d2aff0 100644 > --- a/include/trace/events/pwm.h > +++ b/include/trace/events/pwm.h > @@ -8,15 +8,135 @@ > #include <linux/pwm.h> > #include <linux/tracepoint.h> > > +#define TP_PROTO_pwm(args...) \ > + TP_PROTO(struct pwm_device *pwm, args) > + > +#define TP_ARGS_pwm(args...) \ > + TP_ARGS(pwm, args) > + > +#define TP_STRUCT__entry_pwm(args...) \ > + TP_STRUCT__entry( \ > + __field(unsigned int, chipid) \ > + __field(unsigned int, hwpwm) \ > + args) > + > +#define TP_fast_assign_pwm(args...) \ > + TP_fast_assign( \ > + __entry->chipid = pwm->chip->id; \ > + __entry->hwpwm = pwm->hwpwm; \ > + args) > + > +#define TP_printk_pwm(fmt, args...) \ > + TP_printk("pwmchip%u.%u: " fmt, __entry->chipid, __entry->hwpwm, args) > + > +#define __field_pwmwf(wf) \ > + __field(u64, wf ## _period_length) \ > + __field(u64, wf ## _duty_length) \ > + __field(u64, wf ## _duty_offset) \ > + > +#define fast_assign_pwmwf(wf) \ > + __entry->wf ## _period_length = wf->period_length; \ > + __entry->wf ## _duty_length = wf->duty_length; \ > + __entry->wf ## _duty_offset = wf->duty_offset > + > +#define printk_pwmwf_format(wf) \ > + "%lld/%lld [+%lld]" > + > +#define printk_pwmwf_formatargs(wf) \ > + __entry->wf ## _duty_length, __entry->wf ## _period_length, __entry->wf ## _duty_offset > + > +TRACE_EVENT(pwm_round_waveform_tohw, > + > + TP_PROTO_pwm(const struct pwm_waveform *wf, void *wfhw, int err), > + > + TP_ARGS_pwm(wf, wfhw, err), > + > + TP_STRUCT__entry_pwm( > + __field_pwmwf(wf) > + __field(void *, wfhw) > + __field(int, err) > + ), > + > + TP_fast_assign_pwm( > + fast_assign_pwmwf(wf); > + __entry->wfhw = wfhw; > + __entry->err = err; > + ), > + > + TP_printk_pwm(printk_pwmwf_format(wf) " > %p err=%d", > + printk_pwmwf_formatargs(wf), __entry->wfhw, __entry->err) > +); > + > +TRACE_EVENT(pwm_round_waveform_fromhw, > + > + TP_PROTO_pwm(const void *wfhw, struct pwm_waveform *wf, int err), > + > + TP_ARGS_pwm(wfhw, wf, err), > + > + TP_STRUCT__entry_pwm( > + __field(const void *, wfhw) > + __field_pwmwf(wf) > + __field(int, err) > + ), > + > + TP_fast_assign_pwm( > + __entry->wfhw = wfhw; > + fast_assign_pwmwf(wf); > + __entry->err = err; > + ), > + > + TP_printk_pwm("%p > " printk_pwmwf_format(wf) " err=%d", > + __entry->wfhw, printk_pwmwf_formatargs(wf), __entry->err) > +); > + > +TRACE_EVENT(pwm_read_waveform, > + > + TP_PROTO_pwm(void *wfhw, int err), > + > + TP_ARGS_pwm(wfhw, err), > + > + TP_STRUCT__entry_pwm( > + __field(void *, wfhw) > + __field(int, err) > + ), > + > + TP_fast_assign_pwm( > + __entry->wfhw = wfhw; > + __entry->err = err; > + ), > + > + TP_printk_pwm("%p err=%d", > + __entry->wfhw, __entry->err) > +); > + > +TRACE_EVENT(pwm_write_waveform, > + > + TP_PROTO_pwm(const void *wfhw, int err), > + > + TP_ARGS_pwm(wfhw, err), > + > + TP_STRUCT__entry_pwm( > + __field(const void *, wfhw) > + __field(int, err) > + ), > + > + TP_fast_assign_pwm( > + __entry->wfhw = wfhw; > + __entry->err = err; > + ), > + > + TP_printk_pwm("%p err=%d", > + __entry->wfhw, __entry->err) > +); > + > + > DECLARE_EVENT_CLASS(pwm, > > TP_PROTO(struct pwm_device *pwm, const struct pwm_state *state, int err), > > TP_ARGS(pwm, state, err), > > - TP_STRUCT__entry( > - __field(unsigned int, chipid) > - __field(unsigned int, hwpwm) > + TP_STRUCT__entry_pwm( > __field(u64, period) > __field(u64, duty_cycle) > __field(enum pwm_polarity, polarity) > @@ -24,9 +144,7 @@ DECLARE_EVENT_CLASS(pwm, > __field(int, err) > ), > > - TP_fast_assign( > - __entry->chipid = pwm->chip->id; > - __entry->hwpwm = pwm->hwpwm; > + TP_fast_assign_pwm( > __entry->period = state->period; > __entry->duty_cycle = state->duty_cycle; > __entry->polarity = state->polarity; > @@ -34,8 +152,8 @@ DECLARE_EVENT_CLASS(pwm, > __entry->err = err; > ), > > - TP_printk("pwmchip%u.%u: period=%llu duty_cycle=%llu polarity=%d enabled=%d err=%d", > - __entry->chipid, __entry->hwpwm, __entry->period, __entry->duty_cycle, > + TP_printk_pwm("period=%llu duty_cycle=%llu polarity=%d enabled=%d err=%d", > + __entry->period, __entry->duty_cycle, > __entry->polarity, __entry->enabled, __entry->err) > > ); ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/6] pwm: Add tracing for waveform callbacks 2024-07-08 18:14 ` Trevor Gamblin @ 2024-07-09 6:54 ` Uwe Kleine-König 0 siblings, 0 replies; 4+ messages in thread From: Uwe Kleine-König @ 2024-07-09 6:54 UTC (permalink / raw) To: Trevor Gamblin Cc: linux-pwm, Steven Rostedt, Masami Hiramatsu, Mathieu Desnoyers, linux-trace-kernel [-- Attachment #1: Type: text/plain, Size: 153 bytes --] On Mon, Jul 08, 2024 at 02:14:09PM -0400, Trevor Gamblin wrote: > Missing a sign-off? Yes, and a commit log. Thanks for your review! Best regards Uwe [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-07-09 6:54 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-07-08 10:52 [PATCH 0/6] pwm: New abstraction and userspace API Uwe Kleine-König 2024-07-08 10:52 ` [PATCH 4/6] pwm: Add tracing for waveform callbacks Uwe Kleine-König 2024-07-08 18:14 ` Trevor Gamblin 2024-07-09 6:54 ` Uwe Kleine-König
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).