Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-24 Thread Ryder Lee
+John HI John, On Mon, 2019-01-21 at 16:49 +0800, Uwe Kleine-König wrote: > On Sat, Jan 19, 2019 at 10:54:47AM +0800, Ryder Lee wrote: > > On Fri, 2019-01-18 at 09:43 +0100, Matthias Brugger wrote: > > > > > > On 18/01/2019 04:24, Ryder Lee wrote: > > > > This adds a property "mediatek,num-pwms"

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-24 Thread Ryder Lee
+John HI John, On Mon, 2019-01-21 at 16:49 +0800, Uwe Kleine-König wrote: > On Sat, Jan 19, 2019 at 10:54:47AM +0800, Ryder Lee wrote: > > On Fri, 2019-01-18 at 09:43 +0100, Matthias Brugger wrote: > > > > > > On 18/01/2019 04:24, Ryder Lee wrote: > > > > This adds a property "mediatek,num-pwms"

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-21 Thread Uwe Kleine-König
On Sat, Jan 19, 2019 at 10:54:47AM +0800, Ryder Lee wrote: > On Fri, 2019-01-18 at 09:43 +0100, Matthias Brugger wrote: > > > > On 18/01/2019 04:24, Ryder Lee wrote: > > > This adds a property "mediatek,num-pwms" to avoid having an endless > > > list of compatibles with no differences for the same

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Ryder Lee
On Fri, 2019-01-18 at 09:43 +0100, Matthias Brugger wrote: > > On 18/01/2019 04:24, Ryder Lee wrote: > > This adds a property "mediatek,num-pwms" to avoid having an endless > > list of compatibles with no differences for the same driver. > > > > Thus, the driver should have backwards compatibilit

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Ryder Lee
On Fri, 2019-01-18 at 10:53 +0100, Uwe Kleine-König wrote: > Hello Ryder, > > On Fri, Jan 18, 2019 at 05:42:54PM +0800, Ryder Lee wrote: > > On Fri, 2019-01-18 at 08:59 +0100, Uwe Kleine-König wrote: > > > Hello, > > > > > > On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > > > > This

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Uwe Kleine-König
Hello Ryder, On Fri, Jan 18, 2019 at 05:42:54PM +0800, Ryder Lee wrote: > On Fri, 2019-01-18 at 08:59 +0100, Uwe Kleine-König wrote: > > Hello, > > > > On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > > > This adds a property "mediatek,num-pwms" to avoid having an endless > > > list o

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Ryder Lee
On Fri, 2019-01-18 at 09:05 +0100, Uwe Kleine-König wrote: > Hello, > > just realized another issue while looking up a driver detail ... > > On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > > This adds a property "mediatek,num-pwms" to avoid having an endless > > list of compatibles w

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Ryder Lee
On Fri, 2019-01-18 at 08:59 +0100, Uwe Kleine-König wrote: > Hello, > > On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > > This adds a property "mediatek,num-pwms" to avoid having an endless > > list of compatibles with no differences for the same driver. > > > > Thus, the driver shou

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Matthias Brugger
On 18/01/2019 04:24, Ryder Lee wrote: > This adds a property "mediatek,num-pwms" to avoid having an endless > list of compatibles with no differences for the same driver. > > Thus, the driver should have backwards compatibility to older DTs. > > Signed-off-by: Ryder Lee > --- > Changes since

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Uwe Kleine-König
Hello, just realized another issue while looking up a driver detail ... On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > This adds a property "mediatek,num-pwms" to avoid having an endless > list of compatibles with no differences for the same driver. > > Thus, the driver should have

Re: [PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-18 Thread Uwe Kleine-König
Hello, On Fri, Jan 18, 2019 at 11:24:41AM +0800, Ryder Lee wrote: > This adds a property "mediatek,num-pwms" to avoid having an endless > list of compatibles with no differences for the same driver. > > Thus, the driver should have backwards compatibility to older DTs. I still think Thierry shou

[PATCH v1 1/5] pwm: mediatek: add a property "mediatek,num-pwms"

2019-01-17 Thread Ryder Lee
This adds a property "mediatek,num-pwms" to avoid having an endless list of compatibles with no differences for the same driver. Thus, the driver should have backwards compatibility to older DTs. Signed-off-by: Ryder Lee --- Changes since v1: add some checks for backwards compatibility. --- dri