* [PATCH v7 1/4] remoteproc/davinci: use the reset framework
2018-06-21 7:37 [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Bartosz Golaszewski
@ 2018-06-21 7:37 ` Bartosz Golaszewski
2018-07-23 8:03 ` Bartosz Golaszewski
2018-07-31 4:25 ` Bjorn Andersson
2018-06-21 7:37 ` [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert() Bartosz Golaszewski
` (3 subsequent siblings)
4 siblings, 2 replies; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-06-21 7:37 UTC (permalink / raw)
To: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Switch to using the reset framework instead of handcoded reset routines
we used so far.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: Sekhar Nori <nsekhar@ti.com>
Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
---
drivers/remoteproc/da8xx_remoteproc.c | 34 +++++++++++++++++++++++----
1 file changed, 29 insertions(+), 5 deletions(-)
diff --git a/drivers/remoteproc/da8xx_remoteproc.c b/drivers/remoteproc/da8xx_remoteproc.c
index b668e32996e2..76c06b70a1c6 100644
--- a/drivers/remoteproc/da8xx_remoteproc.c
+++ b/drivers/remoteproc/da8xx_remoteproc.c
@@ -10,6 +10,7 @@
#include <linux/bitops.h>
#include <linux/clk.h>
+#include <linux/reset.h>
#include <linux/err.h>
#include <linux/interrupt.h>
#include <linux/io.h>
@@ -20,8 +21,6 @@
#include <linux/platform_device.h>
#include <linux/remoteproc.h>
-#include <mach/clock.h> /* for davinci_clk_reset_assert/deassert() */
-
#include "remoteproc_internal.h"
static char *da8xx_fw_name;
@@ -72,6 +71,7 @@ struct da8xx_rproc {
struct da8xx_rproc_mem *mem;
int num_mems;
struct clk *dsp_clk;
+ struct reset_control *dsp_reset;
void (*ack_fxn)(struct irq_data *data);
struct irq_data *irq_data;
void __iomem *chipsig;
@@ -138,6 +138,7 @@ static int da8xx_rproc_start(struct rproc *rproc)
struct device *dev = rproc->dev.parent;
struct da8xx_rproc *drproc = (struct da8xx_rproc *)rproc->priv;
struct clk *dsp_clk = drproc->dsp_clk;
+ struct reset_control *dsp_reset = drproc->dsp_reset;
int ret;
/* hw requires the start (boot) address be on 1KB boundary */
@@ -155,7 +156,12 @@ static int da8xx_rproc_start(struct rproc *rproc)
return ret;
}
- davinci_clk_reset_deassert(dsp_clk);
+ ret = reset_control_deassert(dsp_reset);
+ if (ret) {
+ dev_err(dev, "reset_control_deassert() failed: %d\n", ret);
+ clk_disable_unprepare(dsp_clk);
+ return ret;
+ }
return 0;
}
@@ -163,8 +169,15 @@ static int da8xx_rproc_start(struct rproc *rproc)
static int da8xx_rproc_stop(struct rproc *rproc)
{
struct da8xx_rproc *drproc = rproc->priv;
+ struct device *dev = rproc->dev.parent;
+ int ret;
+
+ ret = reset_control_assert(drproc->dsp_reset);
+ if (ret) {
+ dev_err(dev, "reset_control_assert() failed: %d\n", ret);
+ return ret;
+ }
- davinci_clk_reset_assert(drproc->dsp_clk);
clk_disable_unprepare(drproc->dsp_clk);
return 0;
@@ -232,6 +245,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
struct resource *bootreg_res;
struct resource *chipsig_res;
struct clk *dsp_clk;
+ struct reset_control *dsp_reset;
void __iomem *chipsig;
void __iomem *bootreg;
int irq;
@@ -268,6 +282,15 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
return PTR_ERR(dsp_clk);
}
+ dsp_reset = devm_reset_control_get_exclusive(dev, NULL);
+ if (IS_ERR(dsp_reset)) {
+ if (PTR_ERR(dsp_reset) != -EPROBE_DEFER)
+ dev_err(dev, "unable to get reset control: %ld\n",
+ PTR_ERR(dsp_reset));
+
+ return PTR_ERR(dsp_reset);
+ }
+
if (dev->of_node) {
ret = of_reserved_mem_device_init(dev);
if (ret) {
@@ -287,6 +310,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
drproc = rproc->priv;
drproc->rproc = rproc;
drproc->dsp_clk = dsp_clk;
+ drproc->dsp_reset = dsp_reset;
rproc->has_iommu = false;
ret = da8xx_rproc_get_internal_memories(pdev, drproc);
@@ -309,7 +333,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
* *not* in reset, but da8xx_rproc_start() needs the DSP to be
* held in reset at the time it is called.
*/
- ret = davinci_clk_reset_assert(drproc->dsp_clk);
+ ret = reset_control_assert(dsp_reset);
if (ret)
goto free_rproc;
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH v7 1/4] remoteproc/davinci: use the reset framework
2018-06-21 7:37 ` [PATCH v7 1/4] remoteproc/davinci: use " Bartosz Golaszewski
@ 2018-07-23 8:03 ` Bartosz Golaszewski
2018-07-31 4:25 ` Bjorn Andersson
1 sibling, 0 replies; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-07-23 8:03 UTC (permalink / raw)
To: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
2018-06-21 9:37 GMT+02:00 Bartosz Golaszewski <brgl@bgdev.pl>:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Switch to using the reset framework instead of handcoded reset routines
> we used so far.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: Sekhar Nori <nsekhar@ti.com>
> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
> drivers/remoteproc/da8xx_remoteproc.c | 34 +++++++++++++++++++++++----
> 1 file changed, 29 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/remoteproc/da8xx_remoteproc.c b/drivers/remoteproc/da8xx_remoteproc.c
> index b668e32996e2..76c06b70a1c6 100644
> --- a/drivers/remoteproc/da8xx_remoteproc.c
> +++ b/drivers/remoteproc/da8xx_remoteproc.c
> @@ -10,6 +10,7 @@
>
> #include <linux/bitops.h>
> #include <linux/clk.h>
> +#include <linux/reset.h>
> #include <linux/err.h>
> #include <linux/interrupt.h>
> #include <linux/io.h>
> @@ -20,8 +21,6 @@
> #include <linux/platform_device.h>
> #include <linux/remoteproc.h>
>
> -#include <mach/clock.h> /* for davinci_clk_reset_assert/deassert() */
> -
> #include "remoteproc_internal.h"
>
> static char *da8xx_fw_name;
> @@ -72,6 +71,7 @@ struct da8xx_rproc {
> struct da8xx_rproc_mem *mem;
> int num_mems;
> struct clk *dsp_clk;
> + struct reset_control *dsp_reset;
> void (*ack_fxn)(struct irq_data *data);
> struct irq_data *irq_data;
> void __iomem *chipsig;
> @@ -138,6 +138,7 @@ static int da8xx_rproc_start(struct rproc *rproc)
> struct device *dev = rproc->dev.parent;
> struct da8xx_rproc *drproc = (struct da8xx_rproc *)rproc->priv;
> struct clk *dsp_clk = drproc->dsp_clk;
> + struct reset_control *dsp_reset = drproc->dsp_reset;
> int ret;
>
> /* hw requires the start (boot) address be on 1KB boundary */
> @@ -155,7 +156,12 @@ static int da8xx_rproc_start(struct rproc *rproc)
> return ret;
> }
>
> - davinci_clk_reset_deassert(dsp_clk);
> + ret = reset_control_deassert(dsp_reset);
> + if (ret) {
> + dev_err(dev, "reset_control_deassert() failed: %d\n", ret);
> + clk_disable_unprepare(dsp_clk);
> + return ret;
> + }
>
> return 0;
> }
> @@ -163,8 +169,15 @@ static int da8xx_rproc_start(struct rproc *rproc)
> static int da8xx_rproc_stop(struct rproc *rproc)
> {
> struct da8xx_rproc *drproc = rproc->priv;
> + struct device *dev = rproc->dev.parent;
> + int ret;
> +
> + ret = reset_control_assert(drproc->dsp_reset);
> + if (ret) {
> + dev_err(dev, "reset_control_assert() failed: %d\n", ret);
> + return ret;
> + }
>
> - davinci_clk_reset_assert(drproc->dsp_clk);
> clk_disable_unprepare(drproc->dsp_clk);
>
> return 0;
> @@ -232,6 +245,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> struct resource *bootreg_res;
> struct resource *chipsig_res;
> struct clk *dsp_clk;
> + struct reset_control *dsp_reset;
> void __iomem *chipsig;
> void __iomem *bootreg;
> int irq;
> @@ -268,6 +282,15 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> return PTR_ERR(dsp_clk);
> }
>
> + dsp_reset = devm_reset_control_get_exclusive(dev, NULL);
> + if (IS_ERR(dsp_reset)) {
> + if (PTR_ERR(dsp_reset) != -EPROBE_DEFER)
> + dev_err(dev, "unable to get reset control: %ld\n",
> + PTR_ERR(dsp_reset));
> +
> + return PTR_ERR(dsp_reset);
> + }
> +
> if (dev->of_node) {
> ret = of_reserved_mem_device_init(dev);
> if (ret) {
> @@ -287,6 +310,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> drproc = rproc->priv;
> drproc->rproc = rproc;
> drproc->dsp_clk = dsp_clk;
> + drproc->dsp_reset = dsp_reset;
> rproc->has_iommu = false;
>
> ret = da8xx_rproc_get_internal_memories(pdev, drproc);
> @@ -309,7 +333,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> * *not* in reset, but da8xx_rproc_start() needs the DSP to be
> * held in reset at the time it is called.
> */
> - ret = davinci_clk_reset_assert(drproc->dsp_clk);
> + ret = reset_control_assert(dsp_reset);
> if (ret)
> goto free_rproc;
>
> --
> 2.17.1
>
Hi Bjorn, Sekhar,
I'm not seeing this patch in next, did you agree on how to pick it up for 4.19?
Thanks in advance,
Bartosz
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 1/4] remoteproc/davinci: use the reset framework
2018-06-21 7:37 ` [PATCH v7 1/4] remoteproc/davinci: use " Bartosz Golaszewski
2018-07-23 8:03 ` Bartosz Golaszewski
@ 2018-07-31 4:25 ` Bjorn Andersson
2018-07-31 7:55 ` Bartosz Golaszewski
1 sibling, 1 reply; 21+ messages in thread
From: Bjorn Andersson @ 2018-07-31 4:25 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, linux-arm-kernel, devicetree, linux-kernel,
linux-clk, linux-remoteproc, Bartosz Golaszewski
On Thu 21 Jun 00:37 PDT 2018, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Switch to using the reset framework instead of handcoded reset routines
> we used so far.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: Sekhar Nori <nsekhar@ti.com>
> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Regards,
Bjorn
> ---
> drivers/remoteproc/da8xx_remoteproc.c | 34 +++++++++++++++++++++++----
> 1 file changed, 29 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/remoteproc/da8xx_remoteproc.c b/drivers/remoteproc/da8xx_remoteproc.c
> index b668e32996e2..76c06b70a1c6 100644
> --- a/drivers/remoteproc/da8xx_remoteproc.c
> +++ b/drivers/remoteproc/da8xx_remoteproc.c
> @@ -10,6 +10,7 @@
>
> #include <linux/bitops.h>
> #include <linux/clk.h>
> +#include <linux/reset.h>
> #include <linux/err.h>
> #include <linux/interrupt.h>
> #include <linux/io.h>
> @@ -20,8 +21,6 @@
> #include <linux/platform_device.h>
> #include <linux/remoteproc.h>
>
> -#include <mach/clock.h> /* for davinci_clk_reset_assert/deassert() */
> -
> #include "remoteproc_internal.h"
>
> static char *da8xx_fw_name;
> @@ -72,6 +71,7 @@ struct da8xx_rproc {
> struct da8xx_rproc_mem *mem;
> int num_mems;
> struct clk *dsp_clk;
> + struct reset_control *dsp_reset;
> void (*ack_fxn)(struct irq_data *data);
> struct irq_data *irq_data;
> void __iomem *chipsig;
> @@ -138,6 +138,7 @@ static int da8xx_rproc_start(struct rproc *rproc)
> struct device *dev = rproc->dev.parent;
> struct da8xx_rproc *drproc = (struct da8xx_rproc *)rproc->priv;
> struct clk *dsp_clk = drproc->dsp_clk;
> + struct reset_control *dsp_reset = drproc->dsp_reset;
> int ret;
>
> /* hw requires the start (boot) address be on 1KB boundary */
> @@ -155,7 +156,12 @@ static int da8xx_rproc_start(struct rproc *rproc)
> return ret;
> }
>
> - davinci_clk_reset_deassert(dsp_clk);
> + ret = reset_control_deassert(dsp_reset);
> + if (ret) {
> + dev_err(dev, "reset_control_deassert() failed: %d\n", ret);
> + clk_disable_unprepare(dsp_clk);
> + return ret;
> + }
>
> return 0;
> }
> @@ -163,8 +169,15 @@ static int da8xx_rproc_start(struct rproc *rproc)
> static int da8xx_rproc_stop(struct rproc *rproc)
> {
> struct da8xx_rproc *drproc = rproc->priv;
> + struct device *dev = rproc->dev.parent;
> + int ret;
> +
> + ret = reset_control_assert(drproc->dsp_reset);
> + if (ret) {
> + dev_err(dev, "reset_control_assert() failed: %d\n", ret);
> + return ret;
> + }
>
> - davinci_clk_reset_assert(drproc->dsp_clk);
> clk_disable_unprepare(drproc->dsp_clk);
>
> return 0;
> @@ -232,6 +245,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> struct resource *bootreg_res;
> struct resource *chipsig_res;
> struct clk *dsp_clk;
> + struct reset_control *dsp_reset;
> void __iomem *chipsig;
> void __iomem *bootreg;
> int irq;
> @@ -268,6 +282,15 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> return PTR_ERR(dsp_clk);
> }
>
> + dsp_reset = devm_reset_control_get_exclusive(dev, NULL);
> + if (IS_ERR(dsp_reset)) {
> + if (PTR_ERR(dsp_reset) != -EPROBE_DEFER)
> + dev_err(dev, "unable to get reset control: %ld\n",
> + PTR_ERR(dsp_reset));
> +
> + return PTR_ERR(dsp_reset);
> + }
> +
> if (dev->of_node) {
> ret = of_reserved_mem_device_init(dev);
> if (ret) {
> @@ -287,6 +310,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> drproc = rproc->priv;
> drproc->rproc = rproc;
> drproc->dsp_clk = dsp_clk;
> + drproc->dsp_reset = dsp_reset;
> rproc->has_iommu = false;
>
> ret = da8xx_rproc_get_internal_memories(pdev, drproc);
> @@ -309,7 +333,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
> * *not* in reset, but da8xx_rproc_start() needs the DSP to be
> * held in reset at the time it is called.
> */
> - ret = davinci_clk_reset_assert(drproc->dsp_clk);
> + ret = reset_control_assert(dsp_reset);
> if (ret)
> goto free_rproc;
>
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 1/4] remoteproc/davinci: use the reset framework
2018-07-31 4:25 ` Bjorn Andersson
@ 2018-07-31 7:55 ` Bartosz Golaszewski
2018-07-31 10:04 ` Sekhar Nori
0 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-07-31 7:55 UTC (permalink / raw)
To: Bjorn Andersson, Sekhar Nori
Cc: Kevin Hilman, Rob Herring, Mark Rutland, Russell King,
David Lechner, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
2018-07-31 6:25 GMT+02:00 Bjorn Andersson <bjorn.andersson@linaro.org>:
> On Thu 21 Jun 00:37 PDT 2018, Bartosz Golaszewski wrote:
>
>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> Switch to using the reset framework instead of handcoded reset routines
>> we used so far.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> Reviewed-by: Sekhar Nori <nsekhar@ti.com>
>> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>
Sekhar,
can you take this through your tree or is it already too late?
Bart
> Regards,
> Bjorn
>
>> ---
>> drivers/remoteproc/da8xx_remoteproc.c | 34 +++++++++++++++++++++++----
>> 1 file changed, 29 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/remoteproc/da8xx_remoteproc.c b/drivers/remoteproc/da8xx_remoteproc.c
>> index b668e32996e2..76c06b70a1c6 100644
>> --- a/drivers/remoteproc/da8xx_remoteproc.c
>> +++ b/drivers/remoteproc/da8xx_remoteproc.c
>> @@ -10,6 +10,7 @@
>>
>> #include <linux/bitops.h>
>> #include <linux/clk.h>
>> +#include <linux/reset.h>
>> #include <linux/err.h>
>> #include <linux/interrupt.h>
>> #include <linux/io.h>
>> @@ -20,8 +21,6 @@
>> #include <linux/platform_device.h>
>> #include <linux/remoteproc.h>
>>
>> -#include <mach/clock.h> /* for davinci_clk_reset_assert/deassert() */
>> -
>> #include "remoteproc_internal.h"
>>
>> static char *da8xx_fw_name;
>> @@ -72,6 +71,7 @@ struct da8xx_rproc {
>> struct da8xx_rproc_mem *mem;
>> int num_mems;
>> struct clk *dsp_clk;
>> + struct reset_control *dsp_reset;
>> void (*ack_fxn)(struct irq_data *data);
>> struct irq_data *irq_data;
>> void __iomem *chipsig;
>> @@ -138,6 +138,7 @@ static int da8xx_rproc_start(struct rproc *rproc)
>> struct device *dev = rproc->dev.parent;
>> struct da8xx_rproc *drproc = (struct da8xx_rproc *)rproc->priv;
>> struct clk *dsp_clk = drproc->dsp_clk;
>> + struct reset_control *dsp_reset = drproc->dsp_reset;
>> int ret;
>>
>> /* hw requires the start (boot) address be on 1KB boundary */
>> @@ -155,7 +156,12 @@ static int da8xx_rproc_start(struct rproc *rproc)
>> return ret;
>> }
>>
>> - davinci_clk_reset_deassert(dsp_clk);
>> + ret = reset_control_deassert(dsp_reset);
>> + if (ret) {
>> + dev_err(dev, "reset_control_deassert() failed: %d\n", ret);
>> + clk_disable_unprepare(dsp_clk);
>> + return ret;
>> + }
>>
>> return 0;
>> }
>> @@ -163,8 +169,15 @@ static int da8xx_rproc_start(struct rproc *rproc)
>> static int da8xx_rproc_stop(struct rproc *rproc)
>> {
>> struct da8xx_rproc *drproc = rproc->priv;
>> + struct device *dev = rproc->dev.parent;
>> + int ret;
>> +
>> + ret = reset_control_assert(drproc->dsp_reset);
>> + if (ret) {
>> + dev_err(dev, "reset_control_assert() failed: %d\n", ret);
>> + return ret;
>> + }
>>
>> - davinci_clk_reset_assert(drproc->dsp_clk);
>> clk_disable_unprepare(drproc->dsp_clk);
>>
>> return 0;
>> @@ -232,6 +245,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
>> struct resource *bootreg_res;
>> struct resource *chipsig_res;
>> struct clk *dsp_clk;
>> + struct reset_control *dsp_reset;
>> void __iomem *chipsig;
>> void __iomem *bootreg;
>> int irq;
>> @@ -268,6 +282,15 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
>> return PTR_ERR(dsp_clk);
>> }
>>
>> + dsp_reset = devm_reset_control_get_exclusive(dev, NULL);
>> + if (IS_ERR(dsp_reset)) {
>> + if (PTR_ERR(dsp_reset) != -EPROBE_DEFER)
>> + dev_err(dev, "unable to get reset control: %ld\n",
>> + PTR_ERR(dsp_reset));
>> +
>> + return PTR_ERR(dsp_reset);
>> + }
>> +
>> if (dev->of_node) {
>> ret = of_reserved_mem_device_init(dev);
>> if (ret) {
>> @@ -287,6 +310,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
>> drproc = rproc->priv;
>> drproc->rproc = rproc;
>> drproc->dsp_clk = dsp_clk;
>> + drproc->dsp_reset = dsp_reset;
>> rproc->has_iommu = false;
>>
>> ret = da8xx_rproc_get_internal_memories(pdev, drproc);
>> @@ -309,7 +333,7 @@ static int da8xx_rproc_probe(struct platform_device *pdev)
>> * *not* in reset, but da8xx_rproc_start() needs the DSP to be
>> * held in reset at the time it is called.
>> */
>> - ret = davinci_clk_reset_assert(drproc->dsp_clk);
>> + ret = reset_control_assert(dsp_reset);
>> if (ret)
>> goto free_rproc;
>>
>> --
>> 2.17.1
>>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 1/4] remoteproc/davinci: use the reset framework
2018-07-31 7:55 ` Bartosz Golaszewski
@ 2018-07-31 10:04 ` Sekhar Nori
0 siblings, 0 replies; 21+ messages in thread
From: Sekhar Nori @ 2018-07-31 10:04 UTC (permalink / raw)
To: Bartosz Golaszewski, Bjorn Andersson
Cc: Kevin Hilman, Rob Herring, Mark Rutland, Russell King,
David Lechner, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
Bjorn,
On Tuesday 31 July 2018 01:25 PM, Bartosz Golaszewski wrote:
> 2018-07-31 6:25 GMT+02:00 Bjorn Andersson <bjorn.andersson@linaro.org>:
>> On Thu 21 Jun 00:37 PDT 2018, Bartosz Golaszewski wrote:
>>
>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>
>>> Switch to using the reset framework instead of handcoded reset routines
>>> we used so far.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> Reviewed-by: Sekhar Nori <nsekhar@ti.com>
>>> Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
>>
>> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
>>
>
> Sekhar,
>
> can you take this through your tree or is it already too late?
The last -rc is already tagged and I am not sure ARM-SoC will be open to
new stuff now (there is also a bit of lag after I send my pull request).
Is this something you can take in your tree along with Suman's patch?
There are no build dependencies with my tree, so it will be safe to
queue from your tree. It will be great if this can be merged late in the
cycle though to avoid breaking remoteproc even for short while during
the merge window.
Or if you okay with it, it will be safest to send it soon after
v4.19-rc1 is tagged.
Let me know.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert()
2018-06-21 7:37 [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Bartosz Golaszewski
2018-06-21 7:37 ` [PATCH v7 1/4] remoteproc/davinci: use " Bartosz Golaszewski
@ 2018-06-21 7:37 ` Bartosz Golaszewski
2018-07-06 17:54 ` David Lechner
2018-06-21 7:37 ` [PATCH v7 3/4] ARM: davinci: dts: make psc0 a reset provider Bartosz Golaszewski
` (2 subsequent siblings)
4 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-06-21 7:37 UTC (permalink / raw)
To: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
This code is no longer used. Remove it.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: David Lechner <david@lechnology.com>
Acked-by: Stephen Boyd <sboyd@kernel.org>
---
arch/arm/mach-davinci/include/mach/clock.h | 21 ---------------------
drivers/clk/davinci/psc.c | 18 ------------------
2 files changed, 39 deletions(-)
delete mode 100644 arch/arm/mach-davinci/include/mach/clock.h
diff --git a/arch/arm/mach-davinci/include/mach/clock.h b/arch/arm/mach-davinci/include/mach/clock.h
deleted file mode 100644
index 42ed4f2f5ce4..000000000000
--- a/arch/arm/mach-davinci/include/mach/clock.h
+++ /dev/null
@@ -1,21 +0,0 @@
-/*
- * arch/arm/mach-davinci/include/mach/clock.h
- *
- * Clock control driver for DaVinci - header file
- *
- * Authors: Vladimir Barinov <source@mvista.com>
- *
- * 2007 (c) MontaVista Software, Inc. This file is licensed under
- * the terms of the GNU General Public License version 2. This program
- * is licensed "as is" without any warranty of any kind, whether express
- * or implied.
- */
-#ifndef __ASM_ARCH_DAVINCI_CLOCK_H
-#define __ASM_ARCH_DAVINCI_CLOCK_H
-
-struct clk;
-
-int davinci_clk_reset_assert(struct clk *c);
-int davinci_clk_reset_deassert(struct clk *c);
-
-#endif
diff --git a/drivers/clk/davinci/psc.c b/drivers/clk/davinci/psc.c
index fffbed5e263b..5b69e24a224f 100644
--- a/drivers/clk/davinci/psc.c
+++ b/drivers/clk/davinci/psc.c
@@ -303,24 +303,6 @@ static int davinci_lpsc_clk_reset(struct clk *clk, bool reset)
return 0;
}
-/*
- * REVISIT: These exported functions can be removed after a non-DT lookup is
- * added to the reset controller framework and the davinci-rproc driver is
- * updated to use the generic reset controller framework.
- */
-
-int davinci_clk_reset_assert(struct clk *clk)
-{
- return davinci_lpsc_clk_reset(clk, true);
-}
-EXPORT_SYMBOL(davinci_clk_reset_assert);
-
-int davinci_clk_reset_deassert(struct clk *clk)
-{
- return davinci_lpsc_clk_reset(clk, false);
-}
-EXPORT_SYMBOL(davinci_clk_reset_deassert);
-
static int davinci_psc_reset_assert(struct reset_controller_dev *rcdev,
unsigned long id)
{
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert()
2018-06-21 7:37 ` [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert() Bartosz Golaszewski
@ 2018-07-06 17:54 ` David Lechner
2018-10-02 15:08 ` Bartosz Golaszewski
0 siblings, 1 reply; 21+ messages in thread
From: David Lechner @ 2018-07-06 17:54 UTC (permalink / raw)
To: Bartosz Golaszewski, Sekhar Nori, Kevin Hilman, Rob Herring,
Mark Rutland, Russell King, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
On 06/21/2018 02:37 AM, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> This code is no longer used. Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: David Lechner <david@lechnology.com>
> Acked-by: Stephen Boyd <sboyd@kernel.org>
> ---
I've picked this up in clk-davinci-4.20, so it will just hang out
there for a while until the rest of this series gets merged.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert()
2018-07-06 17:54 ` David Lechner
@ 2018-10-02 15:08 ` Bartosz Golaszewski
2018-10-02 15:28 ` David Lechner
0 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-10-02 15:08 UTC (permalink / raw)
To: David Lechner
Cc: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Bjorn Andersson, Linux ARM, devicetree,
Linux Kernel Mailing List, linux-clk, linux-remoteproc,
Bartosz Golaszewski
pt., 6 lip 2018 o 19:54 David Lechner <david@lechnology.com> napisał(a):
>
> On 06/21/2018 02:37 AM, Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> >
> > This code is no longer used. Remove it.
> >
> > Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > Reviewed-by: David Lechner <david@lechnology.com>
> > Acked-by: Stephen Boyd <sboyd@kernel.org>
> > ---
>
> I've picked this up in clk-davinci-4.20, so it will just hang out
> there for a while until the rest of this series gets merged.
>
Hi David,
I don't see this patch in next yet. Is it queued for 4.20?
Thanks in advance,
Bartosz
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert()
2018-10-02 15:08 ` Bartosz Golaszewski
@ 2018-10-02 15:28 ` David Lechner
2018-10-02 15:54 ` Stephen Boyd
0 siblings, 1 reply; 21+ messages in thread
From: David Lechner @ 2018-10-02 15:28 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Bjorn Andersson, Linux ARM, devicetree,
Linux Kernel Mailing List, linux-clk, linux-remoteproc,
Bartosz Golaszewski
On 10/02/2018 10:08 AM, Bartosz Golaszewski wrote:
> pt., 6 lip 2018 o 19:54 David Lechner <david@lechnology.com> napisał(a):
>>
>> On 06/21/2018 02:37 AM, Bartosz Golaszewski wrote:
>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>
>>> This code is no longer used. Remove it.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> Reviewed-by: David Lechner <david@lechnology.com>
>>> Acked-by: Stephen Boyd <sboyd@kernel.org>
>>> ---
>>
>> I've picked this up in clk-davinci-4.20, so it will just hang out
>> there for a while until the rest of this series gets merged.
>>
>
> Hi David,
>
> I don't see this patch in next yet. Is it queued for 4.20?
>
> Thanks in advance,
> Bartosz
>
This is currently the only clk-davinici patch I have queued, so it
might be easier to just resend it and let Stephen/Mike pick it up
directly instead of me sending a pull request for one patch.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert()
2018-10-02 15:28 ` David Lechner
@ 2018-10-02 15:54 ` Stephen Boyd
0 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2018-10-02 15:54 UTC (permalink / raw)
To: Bartosz Golaszewski, David Lechner
Cc: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, Michael Turquette, Ohad Ben-Cohen, Bjorn Andersson,
Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
Quoting David Lechner (2018-10-02 08:28:44)
> On 10/02/2018 10:08 AM, Bartosz Golaszewski wrote:
> > pt., 6 lip 2018 o 19:54 David Lechner <david@lechnology.com> napisał(a):
> >>
> >> On 06/21/2018 02:37 AM, Bartosz Golaszewski wrote:
> >>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> >>>
> >>> This code is no longer used. Remove it.
> >>>
> >>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> >>> Reviewed-by: David Lechner <david@lechnology.com>
> >>> Acked-by: Stephen Boyd <sboyd@kernel.org>
> >>> ---
> >>
> >> I've picked this up in clk-davinci-4.20, so it will just hang out
> >> there for a while until the rest of this series gets merged.
> >>
> >
> > Hi David,
> >
> > I don't see this patch in next yet. Is it queued for 4.20?
> >
> > Thanks in advance,
> > Bartosz
> >
>
> This is currently the only clk-davinici patch I have queued, so it
> might be easier to just resend it and let Stephen/Mike pick it up
> directly instead of me sending a pull request for one patch.
>
Ok I applied it directly to clk-next.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH v7 3/4] ARM: davinci: dts: make psc0 a reset provider
2018-06-21 7:37 [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Bartosz Golaszewski
2018-06-21 7:37 ` [PATCH v7 1/4] remoteproc/davinci: use " Bartosz Golaszewski
2018-06-21 7:37 ` [PATCH v7 2/4] clk: davinci: kill davinci_clk_reset_assert/deassert() Bartosz Golaszewski
@ 2018-06-21 7:37 ` Bartosz Golaszewski
2018-07-02 11:57 ` Sekhar Nori
2018-06-21 7:37 ` [PATCH v7 4/4] ARM: davinci: dts: add a reset control to the dsp node Bartosz Golaszewski
2018-06-21 10:52 ` [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Sekhar Nori
4 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-06-21 7:37 UTC (permalink / raw)
To: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
The psc driver registers with the reset framework as a provider. Add
the #reset-cells property to the psc0 node.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: David Lechner <david@lechnology.com>
---
arch/arm/boot/dts/da850.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 3fc8b8fd816e..1bc0e4bc0697 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -132,6 +132,7 @@
compatible = "ti,da850-psc0";
reg = <0x10000 0x1000>;
#clock-cells = <1>;
+ #reset-cells = <1>;
#power-domain-cells = <1>;
clocks = <&pll0_sysclk 1>, <&pll0_sysclk 2>,
<&pll0_sysclk 4>, <&pll0_sysclk 6>,
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH v7 3/4] ARM: davinci: dts: make psc0 a reset provider
2018-06-21 7:37 ` [PATCH v7 3/4] ARM: davinci: dts: make psc0 a reset provider Bartosz Golaszewski
@ 2018-07-02 11:57 ` Sekhar Nori
0 siblings, 0 replies; 21+ messages in thread
From: Sekhar Nori @ 2018-07-02 11:57 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> The psc driver registers with the reset framework as a provider. Add
> the #reset-cells property to the psc0 node.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: David Lechner <david@lechnology.com>
Applied to v4.19/dt branch of my tree.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH v7 4/4] ARM: davinci: dts: add a reset control to the dsp node
2018-06-21 7:37 [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Bartosz Golaszewski
` (2 preceding siblings ...)
2018-06-21 7:37 ` [PATCH v7 3/4] ARM: davinci: dts: make psc0 a reset provider Bartosz Golaszewski
@ 2018-06-21 7:37 ` Bartosz Golaszewski
2018-07-02 12:02 ` Sekhar Nori
2018-06-21 10:52 ` [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Sekhar Nori
4 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-06-21 7:37 UTC (permalink / raw)
To: Sekhar Nori, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
The davinci-rproc driver will soon use the reset framework. Add the
resets property to the dsp node in da850.dtsi.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: David Lechner <david@lechnology.com>
---
arch/arm/boot/dts/da850.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 1bc0e4bc0697..225e0cf4b7bc 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -118,6 +118,7 @@
interrupt-parent = <&intc>;
interrupts = <28>;
clocks = <&psc0 15>;
+ resets = <&psc0 15>;
status = "disabled";
};
soc@1c00000 {
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH v7 4/4] ARM: davinci: dts: add a reset control to the dsp node
2018-06-21 7:37 ` [PATCH v7 4/4] ARM: davinci: dts: add a reset control to the dsp node Bartosz Golaszewski
@ 2018-07-02 12:02 ` Sekhar Nori
0 siblings, 0 replies; 21+ messages in thread
From: Sekhar Nori @ 2018-07-02 12:02 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> The davinci-rproc driver will soon use the reset framework. Add the
> resets property to the dsp node in da850.dtsi.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: David Lechner <david@lechnology.com>
> ---
> arch/arm/boot/dts/da850.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
> index 1bc0e4bc0697..225e0cf4b7bc 100644
> --- a/arch/arm/boot/dts/da850.dtsi
> +++ b/arch/arm/boot/dts/da850.dtsi
> @@ -118,6 +118,7 @@
> interrupt-parent = <&intc>;
> interrupts = <28>;
> clocks = <&psc0 15>;
> + resets = <&psc0 15>;
Applied to v4.19/dt
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-06-21 7:37 [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Bartosz Golaszewski
` (3 preceding siblings ...)
2018-06-21 7:37 ` [PATCH v7 4/4] ARM: davinci: dts: add a reset control to the dsp node Bartosz Golaszewski
@ 2018-06-21 10:52 ` Sekhar Nori
2018-06-21 11:41 ` Bartosz Golaszewski
4 siblings, 1 reply; 21+ messages in thread
From: Sekhar Nori @ 2018-06-21 10:52 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-arm-kernel, devicetree, linux-kernel, linux-clk,
linux-remoteproc, Bartosz Golaszewski
Hi Bartosz,
On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> These are the remaining patches that still need to be merged in order
> to complete the conversion of the davinci dsp driver to using the reset
> framework.
>
> They apply on top of v4.18-rc1 with David Lechner's remaining patches
> merged.
Series looks good to me.
To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
and #2 ?
Given the dependencies and to preserve bisect its easiest if I take the
series with acks from remoteproc and clock maintainers.
Open to other suggestions as well.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-06-21 10:52 ` [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework Sekhar Nori
@ 2018-06-21 11:41 ` Bartosz Golaszewski
2018-07-02 12:08 ` Sekhar Nori
0 siblings, 1 reply; 21+ messages in thread
From: Bartosz Golaszewski @ 2018-06-21 11:41 UTC (permalink / raw)
To: Sekhar Nori
Cc: Kevin Hilman, Rob Herring, Mark Rutland, Russell King,
David Lechner, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Bjorn Andersson, Linux ARM, devicetree,
Linux Kernel Mailing List, linux-clk, linux-remoteproc,
Bartosz Golaszewski
2018-06-21 12:52 GMT+02:00 Sekhar Nori <nsekhar@ti.com>:
> Hi Bartosz,
>
> On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> These are the remaining patches that still need to be merged in order
>> to complete the conversion of the davinci dsp driver to using the reset
>> framework.
>>
>> They apply on top of v4.18-rc1 with David Lechner's remaining patches
>> merged.
>
> Series looks good to me.
>
> To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
> and #2 ?
>
> Given the dependencies and to preserve bisect its easiest if I take the
> series with acks from remoteproc and clock maintainers.
>
> Open to other suggestions as well.
>
> Thanks,
> Sekhar
Oops you're right about the order. Do you want me to resend?
Bart
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-06-21 11:41 ` Bartosz Golaszewski
@ 2018-07-02 12:08 ` Sekhar Nori
2018-07-02 15:27 ` David Lechner
2018-07-31 4:34 ` Bjorn Andersson
0 siblings, 2 replies; 21+ messages in thread
From: Sekhar Nori @ 2018-07-02 12:08 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Linux ARM, devicetree, Linux Kernel Mailing List,
linux-clk, linux-remoteproc, Bartosz Golaszewski
Hi Bjorn,
On Thursday 21 June 2018 05:11 PM, Bartosz Golaszewski wrote:
> 2018-06-21 12:52 GMT+02:00 Sekhar Nori <nsekhar@ti.com>:
>> Hi Bartosz,
>>
>> On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>
>>> These are the remaining patches that still need to be merged in order
>>> to complete the conversion of the davinci dsp driver to using the reset
>>> framework.
>>>
>>> They apply on top of v4.18-rc1 with David Lechner's remaining patches
>>> merged.
>>
>> Series looks good to me.
>>
>> To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
>> and #2 ?
>>
>> Given the dependencies and to preserve bisect its easiest if I take the
>> series with acks from remoteproc and clock maintainers.
>>
>> Open to other suggestions as well.
>>
>> Thanks,
>> Sekhar
>
> Oops you're right about the order. Do you want me to resend?
With your ack, I can queue 1/4 for v4.19 and provide an immutable commit
to you (on top of v4.18-rc1) for you to merge any further changes you
want to queue from your tree.
Bartosz, given the number of moving pieces, I think its better to keep
2/4 for v4.20 release - once all other other dependencies have been merged.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-07-02 12:08 ` Sekhar Nori
@ 2018-07-02 15:27 ` David Lechner
2018-07-03 4:42 ` Sekhar Nori
2018-07-31 4:34 ` Bjorn Andersson
1 sibling, 1 reply; 21+ messages in thread
From: David Lechner @ 2018-07-02 15:27 UTC (permalink / raw)
To: Sekhar Nori, Bjorn Andersson
Cc: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
On 07/02/2018 07:08 AM, Sekhar Nori wrote:
> Hi Bjorn,
>
> On Thursday 21 June 2018 05:11 PM, Bartosz Golaszewski wrote:
>> 2018-06-21 12:52 GMT+02:00 Sekhar Nori <nsekhar@ti.com>:
>>> Hi Bartosz,
>>>
>>> On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
>>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>>
>>>> These are the remaining patches that still need to be merged in order
>>>> to complete the conversion of the davinci dsp driver to using the reset
>>>> framework.
>>>>
>>>> They apply on top of v4.18-rc1 with David Lechner's remaining patches
>>>> merged.
>>>
>>> Series looks good to me.
>>>
>>> To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
>>> and #2 ?
>>>
>>> Given the dependencies and to preserve bisect its easiest if I take the
>>> series with acks from remoteproc and clock maintainers.
>>>
>>> Open to other suggestions as well.
>>>
>>> Thanks,
>>> Sekhar
>>
>> Oops you're right about the order. Do you want me to resend?
>
> With your ack, I can queue 1/4 for v4.19 and provide an immutable commit
> to you (on top of v4.18-rc1) for you to merge any further changes you
> want to queue from your tree.
>
> Bartosz, given the number of moving pieces, I think its better to keep
> 2/4 for v4.20 release - once all other other dependencies have been merged.
I was thinking the same thing as well. I will pick it up in a
clk-davinci-4.20 branch if that sounds OK.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-07-02 15:27 ` David Lechner
@ 2018-07-03 4:42 ` Sekhar Nori
0 siblings, 0 replies; 21+ messages in thread
From: Sekhar Nori @ 2018-07-03 4:42 UTC (permalink / raw)
To: David Lechner, Bjorn Andersson
Cc: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, Michael Turquette, Stephen Boyd, Ohad Ben-Cohen,
Linux ARM, devicetree, Linux Kernel Mailing List, linux-clk,
linux-remoteproc, Bartosz Golaszewski
On Monday 02 July 2018 08:57 PM, David Lechner wrote:
> On 07/02/2018 07:08 AM, Sekhar Nori wrote:
>> Hi Bjorn,
>>
>> On Thursday 21 June 2018 05:11 PM, Bartosz Golaszewski wrote:
>>> 2018-06-21 12:52 GMT+02:00 Sekhar Nori <nsekhar@ti.com>:
>>>> Hi Bartosz,
>>>>
>>>> On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
>>>>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>>>>
>>>>> These are the remaining patches that still need to be merged in order
>>>>> to complete the conversion of the davinci dsp driver to using the
>>>>> reset
>>>>> framework.
>>>>>
>>>>> They apply on top of v4.18-rc1 with David Lechner's remaining patches
>>>>> merged.
>>>>
>>>> Series looks good to me.
>>>>
>>>> To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
>>>> and #2 ?
>>>>
>>>> Given the dependencies and to preserve bisect its easiest if I take the
>>>> series with acks from remoteproc and clock maintainers.
>>>>
>>>> Open to other suggestions as well.
>>>>
>>>> Thanks,
>>>> Sekhar
>>>
>>> Oops you're right about the order. Do you want me to resend?
>>
>> With your ack, I can queue 1/4 for v4.19 and provide an immutable commit
>> to you (on top of v4.18-rc1) for you to merge any further changes you
>> want to queue from your tree.
>>
>> Bartosz, given the number of moving pieces, I think its better to keep
>> 2/4 for v4.20 release - once all other other dependencies have been
>> merged.
>
> I was thinking the same thing as well. I will pick it up in a
> clk-davinci-4.20 branch if that sounds OK.
Sounds good. Just check that there really are no users of that API
before queuing :)
Regards,
Sekhar
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH v7 0/4] ARM: davinci: complete the conversion to using the reset framework
2018-07-02 12:08 ` Sekhar Nori
2018-07-02 15:27 ` David Lechner
@ 2018-07-31 4:34 ` Bjorn Andersson
1 sibling, 0 replies; 21+ messages in thread
From: Bjorn Andersson @ 2018-07-31 4:34 UTC (permalink / raw)
To: Sekhar Nori
Cc: Bartosz Golaszewski, Kevin Hilman, Rob Herring, Mark Rutland,
Russell King, David Lechner, Michael Turquette, Stephen Boyd,
Ohad Ben-Cohen, Linux ARM, devicetree, Linux Kernel Mailing List,
linux-clk, linux-remoteproc, Bartosz Golaszewski
On Mon 02 Jul 05:08 PDT 2018, Sekhar Nori wrote:
> Hi Bjorn,
>
> On Thursday 21 June 2018 05:11 PM, Bartosz Golaszewski wrote:
> > 2018-06-21 12:52 GMT+02:00 Sekhar Nori <nsekhar@ti.com>:
> >> Hi Bartosz,
> >>
> >> On Thursday 21 June 2018 01:07 PM, Bartosz Golaszewski wrote:
> >>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> >>>
> >>> These are the remaining patches that still need to be merged in order
> >>> to complete the conversion of the davinci dsp driver to using the reset
> >>> framework.
> >>>
> >>> They apply on top of v4.18-rc1 with David Lechner's remaining patches
> >>> merged.
> >>
> >> Series looks good to me.
> >>
> >> To preserve bisect, shouldn't the order of applying be patch #3, #4, #1
> >> and #2 ?
> >>
> >> Given the dependencies and to preserve bisect its easiest if I take the
> >> series with acks from remoteproc and clock maintainers.
> >>
> >> Open to other suggestions as well.
> >>
> >> Thanks,
> >> Sekhar
> >
> > Oops you're right about the order. Do you want me to resend?
>
> With your ack, I can queue 1/4 for v4.19 and provide an immutable commit
> to you (on top of v4.18-rc1) for you to merge any further changes you
> want to queue from your tree.
>
I'm not sure why I didn't see your request earlier, sorry about that.
I seem to have one other davinci patch from Suman, which should be
possible to merge without any conflicts. So there's no need for an
immutable branch at this time.
Regards,
Bjorn
^ permalink raw reply [flat|nested] 21+ messages in thread