* [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property
@ 2026-06-01 15:26 Luca Leonardo Scorcia
2026-06-01 17:53 ` Linus Walleij
2026-06-02 16:42 ` Conor Dooley
0 siblings, 2 replies; 4+ messages in thread
From: Luca Leonardo Scorcia @ 2026-06-01 15:26 UTC (permalink / raw)
To: linux-mediatek
Cc: Luca Leonardo Scorcia, Linus Walleij, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
AngeloGioacchino Del Regno, Sean Wang, linux-gpio, devicetree,
linux-kernel, linux-arm-kernel
The driver for MT6795 pinctrl already supports the slew-rate property.
Add its description to the documentation.
Signed-off-by: Luca Leonardo Scorcia <l.scorcia@gmail•com>
---
.../bindings/pinctrl/mediatek,mt6795-pinctrl.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6795-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6795-pinctrl.yaml
index 68e91c05f122..9a937f414cc9 100644
--- a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6795-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6795-pinctrl.yaml
@@ -152,6 +152,14 @@ patternProperties:
$ref: /schemas/types.yaml#/definitions/uint32
enum: [0, 1, 2, 3]
+ slew-rate:
+ description: |
+ Set the slew rate. Valid arguments are described as below:
+ 0: Normal slew rate
+ 1: Slower slew
+ $ref: /schemas/types.yaml#/definitions/uint32
+ enum: [0, 1]
+
required:
- pinmux
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property
2026-06-01 15:26 [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property Luca Leonardo Scorcia
@ 2026-06-01 17:53 ` Linus Walleij
2026-06-01 19:26 ` Luca Leonardo Scorcia
2026-06-02 16:42 ` Conor Dooley
1 sibling, 1 reply; 4+ messages in thread
From: Linus Walleij @ 2026-06-01 17:53 UTC (permalink / raw)
To: Luca Leonardo Scorcia
Cc: linux-mediatek, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Matthias Brugger, AngeloGioacchino Del Regno, Sean Wang,
linux-gpio, devicetree, linux-kernel, linux-arm-kernel
Hi Luca,
thanks for your patch!
On Mon, Jun 1, 2026 at 5:27 PM Luca Leonardo Scorcia
<l.scorcia@gmail•com> wrote:
> The driver for MT6795 pinctrl already supports the slew-rate property.
> Add its description to the documentation.
>
> Signed-off-by: Luca Leonardo Scorcia <l.scorcia@gmail•com>
(...)
> + slew-rate:
> + description: |
> + Set the slew rate. Valid arguments are described as below:
> + 0: Normal slew rate
> + 1: Slower slew
> + $ref: /schemas/types.yaml#/definitions/uint32
> + enum: [0, 1]
Ideally slew-rate is V/s (or V/us, any suitable prefixes).
Of course that would vary for any capacitance on the output line,
so that value would be for an infinite impedance (open circuit).
Do you know that value for the SoC to you can express it like
this and translate it in the driver?
Maybe we should even invent something like
slew-rate-volt-per-microsecond = <...>; to push this into
SI units.
That said, there are many SoC:s doing what you're doing
above so it's no strict requirement.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property
2026-06-01 17:53 ` Linus Walleij
@ 2026-06-01 19:26 ` Luca Leonardo Scorcia
0 siblings, 0 replies; 4+ messages in thread
From: Luca Leonardo Scorcia @ 2026-06-01 19:26 UTC (permalink / raw)
To: Linus Walleij
Cc: linux-mediatek, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Matthias Brugger, AngeloGioacchino Del Regno, Sean Wang,
linux-gpio, devicetree, linux-kernel, linux-arm-kernel
Hi!
> Do you know that value for the SoC to you can express it like
> this and translate it in the driver?
Unfortunately the data sheet I have doesn't say anything more - just
"Normal slew" or "Slower slew", and I don't have a scope to measure
it. All I know is that it flips a single bit in a configuration
register.
I found this missing property while working on the Xiaomi Mi Smart
Clock x04g (mt8167). I ported the current pinctrl driver for
mt8516/mt8167 to the paris platform driver (adding support for slew
rate and r1r0 pullups in the process) but the stock device tree sets a
slower slew rate for the speaker amp. Without this patch it would
trigger a dts error.
Thanks!
--
Luca
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property
2026-06-01 15:26 [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property Luca Leonardo Scorcia
2026-06-01 17:53 ` Linus Walleij
@ 2026-06-02 16:42 ` Conor Dooley
1 sibling, 0 replies; 4+ messages in thread
From: Conor Dooley @ 2026-06-02 16:42 UTC (permalink / raw)
To: Luca Leonardo Scorcia
Cc: linux-mediatek, Linus Walleij, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, AngeloGioacchino Del Regno,
Sean Wang, linux-gpio, devicetree, linux-kernel, linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 75 bytes --]
Acked-by: Conor Dooley <conor.dooley@microchip•com>
pw-bot: not-applicable
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-06-02 16:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-01 15:26 [PATCH] dt-bindings: pinctrl: mediatek: mt6795: document the slew-rate property Luca Leonardo Scorcia
2026-06-01 17:53 ` Linus Walleij
2026-06-01 19:26 ` Luca Leonardo Scorcia
2026-06-02 16:42 ` Conor Dooley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox