Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-03-02 Thread Philipp Zabel
Hi Tony, On Thu, 2018-03-01 at 10:14 -0800, Tony Lindgren wrote: > * Tony Lindgren [180116 07:03]: > > * Philipp Zabel [180116 09:52]: > > > Looks good to me. Can I move the compatible property when applying? > > > > Oops, here's a better version. I also left out the "prcm" part as > > at some

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-03-01 Thread Tony Lindgren
* Tony Lindgren [180116 07:03]: > * Philipp Zabel [180116 09:52]: > > Looks good to me. Can I move the compatible property when applying? > > Oops, here's a better version. I also left out the "prcm" part as > at some point that should have just ranges instead of both reg and > ranges that it cu

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-22 Thread Tony Lindgren
* Tony Lindgren [180120 00:56]: > * Suman Anna [180120 00:43]: > > We definitely do not want the two resets here for sure, as the rst2 > > belongs to the dsp core (I believe it would be a sibling node to mmu_dsp > > here), and cannot be released from reset without programming the MMU and > > load

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Tony Lindgren
* Suman Anna [180120 00:43]: > On 01/19/2018 05:49 PM, Tony Lindgren wrote: > > * Suman Anna [180119 23:30]: > >> On 01/19/2018 03:33 PM, Tony Lindgren wrote: > >>> OK let's do some tests on that, I'll take a look at doing a dts > >>> file over next few weeks. > >> > >> You can look up the patche

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Suman Anna
On 01/19/2018 05:49 PM, Tony Lindgren wrote: > * Suman Anna [180119 23:30]: >> On 01/19/2018 03:33 PM, Tony Lindgren wrote: >>> OK let's do some tests on that, I'll take a look at doing a dts >>> file over next few weeks. >> >> You can look up the patches folder in the above repo, there are some >

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Tony Lindgren
* Suman Anna [180119 23:30]: > On 01/19/2018 03:33 PM, Tony Lindgren wrote: > > OK let's do some tests on that, I'll take a look at doing a dts > > file over next few weeks. > > You can look up the patches folder in the above repo, there are some > example nodes there already, they are rather str

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Suman Anna
Hi Tony, On 01/19/2018 03:33 PM, Tony Lindgren wrote: > * Suman Anna [180119 20:23]: >> On 01/16/2018 05:22 PM, Tony Lindgren wrote: >>> The RSTST registers mostly tell the device internal reset reason >>> like watchdog reset for an accelerator. I'm not sure how the >>> API for those would look l

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Tony Lindgren
* Suman Anna [180119 20:23]: > On 01/16/2018 05:22 PM, Tony Lindgren wrote: > > The RSTST registers mostly tell the device internal reset reason > > like watchdog reset for an accelerator. I'm not sure how the > > API for those would look like, do you have some ideas? > > There are multiple RSTST

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-19 Thread Suman Anna
Hi Tony, On 01/16/2018 05:22 PM, Tony Lindgren wrote: > Hi, > > * Suman Anna [180116 21:23]: >> While this adaptation is very simple for replacing the RSTCTRL registers >> from the hwmod data into an existing reset driver, I am afraid that it >> doesn't fit well when you want to use the reset AP

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-16 Thread Tony Lindgren
Hi, * Suman Anna [180116 21:23]: > While this adaptation is very simple for replacing the RSTCTRL registers > from the hwmod data into an existing reset driver, I am afraid that it > doesn't fit well when you want to use the reset API from client drivers. Well the reset controller framework is t

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-16 Thread Suman Anna
Hi Tony, On 01/16/2018 09:03 AM, Tony Lindgren wrote: > Hi, > > * Philipp Zabel [180116 09:52]: >> On Mon, 2018-01-15 at 17:11 -0800, Tony Lindgren wrote: >>> +Example: >>> + >>> + prcm: prcm@20 { >>> + compatible = "ti,am3-prcm", "simple-bus"; >>> + reg = <0x20 0x4

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-16 Thread Tony Lindgren
Hi, * Philipp Zabel [180116 09:52]: > On Mon, 2018-01-15 at 17:11 -0800, Tony Lindgren wrote: > > +Example: > > + > > + prcm: prcm@20 { > > + compatible = "ti,am3-prcm", "simple-bus"; > > + reg = <0x20 0x4000>; > > + #address-cells = <1>; > > + #s

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-16 Thread Philipp Zabel
Hi Tony, On Mon, 2018-01-15 at 17:11 -0800, Tony Lindgren wrote: > We can support the RSTCTRL reset registers on many TI SoCs with > reset-simple. > > Cc: Dave Gerlach > Cc: Mark Rutland > Cc: Nishant Menon > Cc: Philipp Zabel > Cc: Rob Herring > Cc: Suman Anna > Cc: Tero Kristo > Signed-o

Re: [PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-15 Thread Tero Kristo
On 16/01/18 03:11, Tony Lindgren wrote: We can support the RSTCTRL reset registers on many TI SoCs with reset-simple. Cc: Dave Gerlach Cc: Mark Rutland Cc: Nishant Menon Cc: Philipp Zabel Cc: Rob Herring Cc: Suman Anna Cc: Tero Kristo Signed-off-by: Tony Lindgren --- That's all there is

[PATCH] reset: ti-rstctrl: use the reset-simple driver

2018-01-15 Thread Tony Lindgren
We can support the RSTCTRL reset registers on many TI SoCs with reset-simple. Cc: Dave Gerlach Cc: Mark Rutland Cc: Nishant Menon Cc: Philipp Zabel Cc: Rob Herring Cc: Suman Anna Cc: Tero Kristo Signed-off-by: Tony Lindgren --- That's all there is to it :) Naturally this can wait for v4.1