Re: [PATCH] ATM: dts: at91: sama5d2_xplained: add pwm controller

From	Alexandre Belloni <alexandre.belloni@free-electrons.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ATM: dts: at91: sama5d2_xplained: add pwm controller
Date	2017-03-28 13:40 +0200
Message-ID	<tpXDJ-6Qq-29@gated-at.bofh.it> (permalink)
References	<tdX6p-7c9-1@gated-at.bofh.it> <tdXg6-7i8-13@gated-at.bofh.it> <tpXaF-6Gq-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi,

On 28/03/2017 at 14:04:25 +0300, m18063 wrote:
> >> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> >> index cd459e8..e88785e 100644
> >> --- a/arch/arm/boot/dts/sama5d2.dtsi
> >> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> >> @@ -1098,6 +1098,14 @@
> >>  				status = "disabled";
> >>  			};
> >>  
> >> +			pwm0: pwm@f802c000 {
> >> +				compatible = "atmel,sama5d2-pwm";
> > This compatible is not documented and is not used in any driver. Is that
> > really working?
> 
> At the moment I send this patch another driver patch was send
> to used this device tree bindings and also to document it.
> Meanwhile you reviewed also that other patch.
> 

Yes, I received that patch first, without the other ones. You should
have put everything in the same series.

> Please let me know if you want to send again this DT patch since
> I didn't yet receive any inputs its v2.
> 

I'll take it once the pwm patch are taken by Thierry because it is not
useful as-is, unless you use:
 compatible = "atmel,sama5d2-pwm", "atmel,sama5d3-pwm";


-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] ATM: dts: at91: sama5d2_xplained: add pwm controller m18063 <Claudiu.Beznea@microchip.com> - 2017-03-28 13:10 +0200
  Re: [PATCH] ATM: dts: at91: sama5d2_xplained: add pwm controller Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2017-03-28 13:40 +0200

csiph-web