diff options
| author | Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> | 2026-04-29 18:56:28 +0200 |
|---|---|---|
| committer | Thierry Reding <treding@nvidia.com> | 2026-07-31 17:55:32 +0200 |
| commit | bf2d4b582eb8bd5f3ab78c59fe2f6738b6c7ab2a (patch) | |
| tree | 48603f50d629e0718479de4f926cffdebd315567 | |
| parent | 93fd84cee4546ea6475b1c4e1250a7ddbd3d5e7f (diff) | |
| download | linux-next-bf2d4b582eb8bd5f3ab78c59fe2f6738b6c7ab2a.tar.gz linux-next-bf2d4b582eb8bd5f3ab78c59fe2f6738b6c7ab2a.zip | |
arm64: tegra: Correct Tegra234 p3737 interrupt flags
GPIO_ACTIVE_x flags are not correct in the context of interrupt flags.
These are simple defines so they could be used in DTS but they will not
have the same meaning:
1. GPIO_ACTIVE_HIGH = 0 => IRQ_TYPE_NONE
2. GPIO_ACTIVE_LOW = 1 => IRQ_TYPE_EDGE_RISING
Realtek RT5640 codec driver requests interrupt on rising edge, so correct
the interrupt flags, assuming the author of the code wanted the similar
logical behavior behind the name "ACTIVE_xxx", this is:
ACTIVE_HIGH => IRQ_TYPE_EDGE_RISING
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
| -rw-r--r-- | arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701.dtsi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701.dtsi b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701.dtsi index f6cad29355e6..420858a8f4a8 100644 --- a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701.dtsi +++ b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701.dtsi @@ -67,7 +67,7 @@ compatible = "realtek,rt5640"; reg = <0x1c>; interrupt-parent = <&gpio>; - interrupts = <TEGRA234_MAIN_GPIO(AC, 5) GPIO_ACTIVE_HIGH>; + interrupts = <TEGRA234_MAIN_GPIO(AC, 5) IRQ_TYPE_EDGE_RISING>; clocks = <&bpmp TEGRA234_CLK_AUD_MCLK>; clock-names = "mclk"; realtek,dmic1-data-pin = <RT5640_DMIC1_DATA_PIN_NONE>; |