Re: [PATCH v2 1/2] dt-bindings: pwm: add Axiado AX3000 PWM

Re: [PATCH v2 1/2] dt-bindings: pwm: add Axiado AX3000 PWM

[Krzysztof Kozlowski]

On Thu, Jul 23, 2026 at 02:32:18AM -0700, Petar Stepanovic wrote:
> The Axiado AX3000 and AX3005 SoCs include PWM controllers that can be
> used to generate configurable PWM output signals.
> 
> Signed-off-by: Petar Stepanovic <pstepanovic@axiado.com>
> ---
>  .../devicetree/bindings/pwm/axiado,ax3000-pwm.yaml | 51 ++++++++++++++++++++++
>  MAINTAINERS                                        |  8 ++++
>  2 files changed, 59 insertions(+)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Best regards,
Krzysztof

Re: [PATCH v2 2/2] pwm: add Axiado AX3000 PWM driver

[Krzysztof Kozlowski]

On Thu, Jul 23, 2026 at 02:32:19AM -0700, Petar Stepanovic wrote:
> +	chip->ops = &axiado_pwm_ops;
> +	chip->atomic = true;
> +
> +	ret = devm_pwmchip_add(dev, chip);
> +	if (ret)
> +		return dev_err_probe(dev, ret, "Failed to add PWM chip\n");
> +
> +	return 0;
> +}
> +
> +static const struct of_device_id axiado_pwm_match[] = {
> +	{ .compatible = "axiado,ax3000-pwm" },
> +	{ .compatible = "axiado,ax3005-pwm" },

This suggests deviecs are fully compatible, so drop the second entry and
express compatibility (see writing bindings, example schema, DTS101
talk/slides).

Best regards,
Krzysztof