Re: [PATCH v4 4/9] arm64: dts: amlogic: t7: Add PWM pinctrl nodes

Neil Armstrong
2026-03-26T08:52:55Z

On 3/25/26 10:15, Ronald Claveau wrote:
> These pinctrl nodes are required by the PWM drivers to configure
> pin muxing at runtime.
>
> Signed-off-by: Ronald Claveau <linux-kernel-dev@aliel.fr>
> ---
> arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi | 136 ++++++++++++++++++++++++++++
> 1 file changed, 136 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> index 62c87d0ef7065..eb09a26bcd0e0 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> @@ -307,6 +307,142 @@ mux {
> };
> };
>
> + pwm_a_pins: pwm-a {
> + mux {
> + groups = "pwm_a";
> + function = "pwm_a";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_a_pins: pwm-ao-a {
> + mux {
> + groups = "pwm_ao_a";
> + function = "pwm_ao_a";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_b_pins: pwm-ao-b {
> + mux {
> + groups = "pwm_ao_b";
> + function = "pwm_ao_b";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_c_pins: pwm-ao-c {
> + mux {
> + groups = "pwm_ao_c";
> + function = "pwm_ao_c";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_c_hiz_pins: pwm-ao-c-hiz {
> + mux {
> + groups = "pwm_ao_c_hiz";
> + function = "pwm_ao_c_hiz";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_d_pins: pwm-ao-d {
> + mux {
> + groups = "pwm_ao_d";
> + function = "pwm_ao_d";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_e_pins: pwm-ao-e {
> + mux {
> + groups = "pwm_ao_e";
> + function = "pwm_ao_e";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_f_pins: pwm-ao-f {
> + mux {
> + groups = "pwm_ao_f";
> + function = "pwm_ao_f";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_g_pins: pwm-ao-g {
> + mux {
> + groups = "pwm_ao_g";
> + function = "pwm_ao_g";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_g_hiz_pins: pwm-ao-g-hiz {
> + mux {
> + groups = "pwm_ao_g_hiz";
> + function = "pwm_ao_g_hiz";
> + bias-disable;
> + };
> + };
> +
> + pwm_ao_h_pins: pwm-ao-h {
> + mux {
> + groups = "pwm_ao_h";
> + function = "pwm_ao_h";
> + bias-disable;
> + };
> + };
> +
> + pwm_b_pins: pwm-b {
> + mux {
> + groups = "pwm_b";
> + function = "pwm_b";
> + bias-disable;
> + };
> + };
> +
> + pwm_c_pins: pwm-c {
> + mux {
> + groups = "pwm_c";
> + function = "pwm_c";
> + bias-disable;
> + };
> + };
> +
> + pwm_d_pins: pwm-d {
> + mux {
> + groups = "pwm_d";
> + function = "pwm_d";
> + bias-disable;
> + };
> + };
> +
> + pwm_e_pins: pwm-e {
> + mux {
> + groups = "pwm_e";
> + function = "pwm_e";
> + bias-disable;
> + };
> + };
> +
> + pwm_f_pins: pwm-f {
> + mux {
> + groups = "pwm_f";
> + function = "pwm_f";
> + bias-disable;
> + };
> + };
> +
> + pwm_vs_pins: pwm-vs {
> + mux {
> + groups = "pwm_vs";
> + function = "pwm_vs";
> + bias-disable;
> + };
> + };
> +
> sdcard_pins: sdcard {
> mux {
> groups = "sdcard_d0",
>

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>

Thanks,
Neil