Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-20 Thread Uwe Kleine-König
On Thu, Dec 20, 2018 at 06:39:04PM +0100, Thierry Reding wrote: > On Mon, Dec 17, 2018 at 08:53:21AM +0100, Uwe Kleine-König wrote: > > On Sun, Dec 16, 2018 at 03:18:52PM +0100, Paul Cercueil wrote: > > > Hi, > > > > > > Le ven. 14 déc. 2018 à 15:26, Uwe Kleine-König > > > a écrit : > > > > Hello

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-20 Thread Thierry Reding
On Mon, Dec 17, 2018 at 08:53:21AM +0100, Uwe Kleine-König wrote: > On Sun, Dec 16, 2018 at 03:18:52PM +0100, Paul Cercueil wrote: > > Hi, > > > > Le ven. 14 déc. 2018 à 15:26, Uwe Kleine-König > > a écrit : > > > Hello, > > > > > > On Fri, Dec 14, 2018 at 02:50:20PM +0100, Linus Walleij wrote:

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-16 Thread Uwe Kleine-König
On Sun, Dec 16, 2018 at 03:18:52PM +0100, Paul Cercueil wrote: > Hi, > > Le ven. 14 déc. 2018 à 15:26, Uwe Kleine-König > a écrit : > > Hello, > > > > On Fri, Dec 14, 2018 at 02:50:20PM +0100, Linus Walleij wrote: > > > On Thu, Dec 13, 2018 at 9:42 PM Uwe Kleine-König > > > wrote: > > > > [Addi

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-16 Thread Paul Cercueil
Hi, Le ven. 14 déc. 2018 à 15:26, Uwe Kleine-König a écrit : Hello, On Fri, Dec 14, 2018 at 02:50:20PM +0100, Linus Walleij wrote: On Thu, Dec 13, 2018 at 9:42 PM Uwe Kleine-König wrote: > [Adding Linus Walleij to Cc:] > On Thu, Dec 13, 2018 at 03:03:15PM +0100, Paul Cercueil wrote: >

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-14 Thread Linus Walleij
On Fri, Dec 14, 2018 at 3:26 PM Uwe Kleine-König wrote: > On Fri, Dec 14, 2018 at 02:50:20PM +0100, Linus Walleij wrote: > > So if the PWM case is something similar, then by all means add > > num-pwms. > > .. or "npwms" to use the same nomenclature as the gpio binding? Either works for me, I don

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-14 Thread Uwe Kleine-König
Hello, On Fri, Dec 14, 2018 at 02:50:20PM +0100, Linus Walleij wrote: > On Thu, Dec 13, 2018 at 9:42 PM Uwe Kleine-König > wrote: > > [Adding Linus Walleij to Cc:] > > On Thu, Dec 13, 2018 at 03:03:15PM +0100, Paul Cercueil wrote: > > > Le jeu. 13 déc. 2018 à 10:24, Uwe Kleine-König > > > a écri

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-14 Thread Linus Walleij
On Thu, Dec 13, 2018 at 9:42 PM Uwe Kleine-König wrote: > [Adding Linus Walleij to Cc:] > > Hello, > > On Thu, Dec 13, 2018 at 03:03:15PM +0100, Paul Cercueil wrote: > > Le jeu. 13 déc. 2018 à 10:24, Uwe Kleine-König > > a écrit : > > > On Wed, Dec 12, 2018 at 11:09:10PM +0100, Paul Cercueil wrot

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-13 Thread Uwe Kleine-König
[Adding Linus Walleij to Cc:] Hello, On Thu, Dec 13, 2018 at 03:03:15PM +0100, Paul Cercueil wrote: > Le jeu. 13 déc. 2018 à 10:24, Uwe Kleine-König > a écrit : > > On Wed, Dec 12, 2018 at 11:09:10PM +0100, Paul Cercueil wrote: > > > The PWM in the JZ4725B works the same as in the JZ4740, excep

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-13 Thread Thierry Reding
On Thu, Dec 13, 2018 at 03:03:15PM +0100, Paul Cercueil wrote: > Hi, > > Le jeu. 13 déc. 2018 à 10:24, Uwe Kleine-König > a écrit : > > Hello, > > > > On Wed, Dec 12, 2018 at 11:09:10PM +0100, Paul Cercueil wrote: > > > The PWM in the JZ4725B works the same as in the JZ4740, except that > > > i

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-13 Thread Paul Cercueil
Hi, Le jeu. 13 déc. 2018 à 10:24, Uwe Kleine-König a écrit : Hello, On Wed, Dec 12, 2018 at 11:09:10PM +0100, Paul Cercueil wrote: The PWM in the JZ4725B works the same as in the JZ4740, except that it only has 6 channels available instead of 8. this driver is probed only from device tr

Re: [PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-13 Thread Uwe Kleine-König
Hello, On Wed, Dec 12, 2018 at 11:09:10PM +0100, Paul Cercueil wrote: > The PWM in the JZ4725B works the same as in the JZ4740, except that it > only has 6 channels available instead of 8. this driver is probed only from device tree? If yes, it might be sensible to specify the number of PWMs ther

[PATCH v8 15/26] pwm: jz4740: Add support for the JZ4725B

2018-12-12 Thread Paul Cercueil
The PWM in the JZ4725B works the same as in the JZ4740, except that it only has 6 channels available instead of 8. Signed-off-by: Paul Cercueil Acked-by: Thierry Reding --- Notes: v5: New patch v6: - Move of_device_id structure back at the bottom (less noise in patch)