2024-03-06 15:47:22

by Fabrice Gasnier

Subject: [PATCH v6] counter: Introduce the COUNTER_COMP_FREQUENCY() macro

Now that there are two users for the "frequency" extension, introduce a
new COUNTER_COMP_FREQUENCY() macro.
This extension is intended to be a read-only signal attribute.

Suggested-by: William Breathitt Gray <[email protected]>
Signed-off-by: Fabrice Gasnier <[email protected]>
---
Changes in v6
- use COUNTER_COMP_SIGNAL_U64() helper macro.

Changes in v5
- "frequency" extension is read-only, so there's no need to provide
a write parameter.
- patch sent separately from "counter: Add stm32 timer events support" [1]
[1] https://lore.kernel.org/lkml/[email protected]/
---
include/linux/counter.h | 3 +++
1 file changed, 3 insertions(+)

diff --git a/include/linux/counter.h b/include/linux/counter.h
index 702e9108bbb4..ac36f6e799f6 100644
--- a/include/linux/counter.h
+++ b/include/linux/counter.h
@@ -602,6 +602,9 @@ struct counter_array {
#define COUNTER_COMP_FLOOR(_read, _write) \
COUNTER_COMP_COUNT_U64("floor", _read, _write)

+#define COUNTER_COMP_FREQUENCY(_read) \
+ COUNTER_COMP_SIGNAL_U64("frequency", _read, NULL)
+
#define COUNTER_COMP_POLARITY(_read, _write, _available) \
{ \
.type = COUNTER_COMP_SIGNAL_POLARITY, \
--
2.25.1

2024-03-06 16:44:08

by William Breathitt Gray

Subject: Re: [PATCH v6] counter: Introduce the COUNTER_COMP_FREQUENCY() macro

On Wed, 06 Mar 2024 16:36:31 +0100, Fabrice Gasnier wrote:
> Now that there are two users for the "frequency" extension, introduce a
> new COUNTER_COMP_FREQUENCY() macro.
> This extension is intended to be a read-only signal attribute.
>
>

Applied, thanks!

[1/1] counter: Introduce the COUNTER_COMP_FREQUENCY() macro
commit: 0e313270e0e91e8d48be7a58e1622228cf9e53f3

William Breathitt Gray