[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ec6b8d6d-6191-d760-10b2-29518010f4b4@ti.com>
Date: Fri, 21 Jul 2023 09:09:03 +0530
From: Vignesh Raghavendra <vigneshr@...com>
To: Nishanth Menon <nm@...com>, Conor Dooley <conor+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Rob Herring <robh+dt@...nel.org>
CC: <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>,
Tero Kristo <kristo@...nel.org>
Subject: Re: [PATCH V2] arm64: dts: ti: k3-pinctrl: Introduce debounce select
mux macros
On 19/06/23 18:46, Nishanth Menon wrote:
> Introduce the debounce select mux macros to allow folks to setup
> debounce configuration for pins. Each configuration selected maps
> to a specific timing register as documented in appropriate Technical
> Reference Manual (example:[1]).
>
> [1] AM625x TRM (section 6.1.2.2): https://www.ti.com/lit/pdf/spruiv7
>
> Signed-off-by: Nishanth Menon <nm@...com>
> ---
> Changes since V1:
> - I have'nt picked up Krystoff's ack since the file modified is now different.
> - Changes now applied to k3-pinctrl.h instead of the ABI header that is
> now set up to be deleted in v6.5-rc1.
>
> V1: https://lore.kernel.org/linux-devicetree/20230308084309.396192-1-nm@ti.com/
>
> arch/arm64/boot/dts/ti/k3-pinctrl.h | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
Reviewed-by: Vignesh Raghavendra <vigneshr@...com>
> diff --git a/arch/arm64/boot/dts/ti/k3-pinctrl.h b/arch/arm64/boot/dts/ti/k3-pinctrl.h
> index c97548a3f42d..6004e0967ec5 100644
> --- a/arch/arm64/boot/dts/ti/k3-pinctrl.h
> +++ b/arch/arm64/boot/dts/ti/k3-pinctrl.h
> @@ -11,6 +11,7 @@
> #define PULLUDEN_SHIFT (16)
> #define PULLTYPESEL_SHIFT (17)
> #define RXACTIVE_SHIFT (18)
> +#define DEBOUNCE_SHIFT (11)
>
> #define PULL_DISABLE (1 << PULLUDEN_SHIFT)
> #define PULL_ENABLE (0 << PULLUDEN_SHIFT)
> @@ -29,6 +30,14 @@
> #define PIN_INPUT_PULLUP (INPUT_EN | PULL_UP)
> #define PIN_INPUT_PULLDOWN (INPUT_EN | PULL_DOWN)
>
> +#define PIN_DEBOUNCE_DISABLE (0 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF1 (1 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF2 (2 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF3 (3 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF4 (4 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF5 (5 << DEBOUNCE_SHIFT)
> +#define PIN_DEBOUNCE_CONF6 (6 << DEBOUNCE_SHIFT)
> +
> #define AM62AX_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) ((val) | (muxmode))
> #define AM62AX_MCU_IOPAD(pa, val, muxmode) (((pa) & 0x1fff)) ((val) | (muxmode))
>
--
Regards
Vignesh
Powered by blists - more mailing lists