On Thu, Sep 21, 2017 at 12:27 AM, Maxime Ripard maxime.ripard@free-electrons.com wrote:
On Thu, Sep 21, 2017 at 06:07:04AM +0000, Vasily Khoruzhick wrote:
Add PWM definition to sun50i-a64.dtsi - it's compatible with PWM found on H3
Signed-off-by: Vasily Khoruzhick anarsoul@gmail.com
v2: - drop 'sun50i-a64-pwm' compatible string and use 'sun8i-h3-pwm' instead, since A64 PWM is compatible with one on H3
arch/arm/dts/sun50i-a64.dtsi | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/arch/arm/dts/sun50i-a64.dtsi b/arch/arm/dts/sun50i-a64.dtsi index 65a344d9ce..00132855ff 100644 --- a/arch/arm/dts/sun50i-a64.dtsi +++ b/arch/arm/dts/sun50i-a64.dtsi @@ -319,6 +319,14 @@ }; };
pwm: pwm@01c21400 {compatible = "allwinner,sun8i-h3-pwm";You'd need both compatible. The A64 you used to have, plus that one.
Why? Hardware seems to be identical and I see a number of non-A64 compatibles in sun50i-a64.dtsi. What's rationale behind adding a new string?
Maxime
-- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com