* [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
@ 2017-01-23 14:13 Chris Brandt
2017-01-23 15:12 ` Geert Uytterhoeven
0 siblings, 1 reply; 7+ messages in thread
From: Chris Brandt @ 2017-01-23 14:13 UTC (permalink / raw)
To: Geert Uytterhoeven, Simon Horman, Magnus Damm, Rob Herring, Mark Rutland
Cc: devicetree, linux-renesas-soc, Chris Brandt
Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
---
arch/arm/boot/dts/r7s72100.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index 3dd427d..354f4b6 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -466,6 +466,7 @@
GIC_SPI 269 IRQ_TYPE_LEVEL_HIGH
GIC_SPI 267 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&mstp8_clks R7S72100_CLK_MMCIF>;
+ power-domains = <&cpg_clocks>;
reg-io-width = <4>;
bus-width = <8>;
status = "disabled";
--
2.10.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
2017-01-23 14:13 [PATCH] ARM: dts: r7s72100: add power-domains to mmcif Chris Brandt
@ 2017-01-23 15:12 ` Geert Uytterhoeven
[not found] ` <CAMuHMdULpxoNaTUX2Dc7R=d+FUb_V45H9=XXfsW3RKBTowE4Uw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 7+ messages in thread
From: Geert Uytterhoeven @ 2017-01-23 15:12 UTC (permalink / raw)
To: Chris Brandt
Cc: Simon Horman, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
2017-01-23 15:12 ` Geert Uytterhoeven
@ 2017-01-24 12:33 ` Simon Horman
0 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2017-01-24 12:33 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas
On Mon, Jan 23, 2017 at 04:12:16PM +0100, Geert Uytterhoeven wrote:
> On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> > Signed-off-by: Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
>
> Reported-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> Acked-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
Should this be queued up as a fix for v4.10 with the following tag?
Fixes: 887862227ba3 ("ARM: dts: r7s72100: add mmcif to device tree")
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
@ 2017-01-24 12:33 ` Simon Horman
0 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2017-01-24 12:33 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
On Mon, Jan 23, 2017 at 04:12:16PM +0100, Geert Uytterhoeven wrote:
> On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
>
> Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Should this be queued up as a fix for v4.10 with the following tag?
Fixes: 887862227ba3 ("ARM: dts: r7s72100: add mmcif to device tree")
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
2017-01-24 12:33 ` Simon Horman
@ 2017-01-24 16:32 ` Chris Brandt
-1 siblings, 0 replies; 7+ messages in thread
From: Chris Brandt @ 2017-01-24 16:32 UTC (permalink / raw)
To: Simon Horman, Geert Uytterhoeven
Cc: Magnus Damm, Rob Herring, Mark Rutland,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas
On Tuesday, January 24, 2017, Simon Horman wrote:
> On Mon, Jan 23, 2017 at 04:12:16PM +0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
> wrote:
> > > Signed-off-by: Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
> >
> > Reported-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > Acked-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
>
> Should this be queued up as a fix for v4.10 with the following tag?
>
> Fixes: 887862227ba3 ("ARM: dts: r7s72100: add mmcif to device tree")
I assume it can. I'm not sure how it effect the driver one way good or bad
(that's a question for Geert I think).
SDHI for r7s72100 needs it too, I just didn't send that yet because my other
fix will cause a merge conflict because it is modifying the same line. But,
that patch is taking longer than I expected.
My guess is that power-domains is for runtime power management...which seems
to be a sore subject for r7s72100 at this point.
Chris
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
@ 2017-01-24 16:32 ` Chris Brandt
0 siblings, 0 replies; 7+ messages in thread
From: Chris Brandt @ 2017-01-24 16:32 UTC (permalink / raw)
To: Simon Horman, Geert Uytterhoeven
Cc: Magnus Damm, Rob Herring, Mark Rutland, devicetree, Linux-Renesas
On Tuesday, January 24, 2017, Simon Horman wrote:
> On Mon, Jan 23, 2017 at 04:12:16PM +0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt@renesas.com>
> wrote:
> > > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> >
> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Should this be queued up as a fix for v4.10 with the following tag?
>
> Fixes: 887862227ba3 ("ARM: dts: r7s72100: add mmcif to device tree")
I assume it can. I'm not sure how it effect the driver one way good or bad
(that's a question for Geert I think).
SDHI for r7s72100 needs it too, I just didn't send that yet because my other
fix will cause a merge conflict because it is modifying the same line. But,
that patch is taking longer than I expected.
My guess is that power-domains is for runtime power management...which seems
to be a sore subject for r7s72100 at this point.
Chris
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to mmcif
2017-01-24 16:32 ` Chris Brandt
(?)
@ 2017-01-25 10:31 ` Simon Horman
-1 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2017-01-25 10:31 UTC (permalink / raw)
To: Chris Brandt
Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring, Mark Rutland,
devicetree, Linux-Renesas
On Tue, Jan 24, 2017 at 04:32:30PM +0000, Chris Brandt wrote:
> On Tuesday, January 24, 2017, Simon Horman wrote:
> > On Mon, Jan 23, 2017 at 04:12:16PM +0100, Geert Uytterhoeven wrote:
> > > On Mon, Jan 23, 2017 at 3:13 PM, Chris Brandt <chris.brandt@renesas.com>
> > wrote:
> > > > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> > >
> > > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > Should this be queued up as a fix for v4.10 with the following tag?
> >
> > Fixes: 887862227ba3 ("ARM: dts: r7s72100: add mmcif to device tree")
>
> I assume it can. I'm not sure how it effect the driver one way good or bad
> (that's a question for Geert I think).
> SDHI for r7s72100 needs it too, I just didn't send that yet because my other
> fix will cause a merge conflict because it is modifying the same line. But,
> that patch is taking longer than I expected.
>
> My guess is that power-domains is for runtime power management...which seems
> to be a sore subject for r7s72100 at this point.
Geert tells me that this should not have a runtime effect.
Accordingly, I will queued it up for v4.11.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-01-25 10:31 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-23 14:13 [PATCH] ARM: dts: r7s72100: add power-domains to mmcif Chris Brandt
2017-01-23 15:12 ` Geert Uytterhoeven
[not found] ` <CAMuHMdULpxoNaTUX2Dc7R=d+FUb_V45H9=XXfsW3RKBTowE4Uw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-01-24 12:33 ` Simon Horman
2017-01-24 12:33 ` Simon Horman
[not found] ` <20170124123304.GC20529-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2017-01-24 16:32 ` Chris Brandt
2017-01-24 16:32 ` Chris Brandt
2017-01-25 10:31 ` Simon Horman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.