* [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
@ 2017-02-09 13:38 Chris Brandt
2017-03-22 13:25 ` Geert Uytterhoeven
0 siblings, 1 reply; 10+ messages in thread
From: Chris Brandt @ 2017-02-09 13:38 UTC (permalink / raw)
To: Geert Uytterhoeven, Simon Horman, Magnus Damm, Rob Herring, Mark Rutland
Cc: devicetree, linux-renesas-soc, Chris Brandt
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
---
arch/arm/boot/dts/r7s72100.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index 614ba79..0b9677f 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -494,6 +494,7 @@
clocks = <&mstp12_clks R7S72100_CLK_SDHI00>,
<&mstp12_clks R7S72100_CLK_SDHI01>;
clock-names = "core", "cd";
+ power-domains = <&cpg_clocks>;
cap-sd-highspeed;
cap-sdio-irq;
status = "disabled";
@@ -509,6 +510,7 @@
clocks = <&mstp12_clks R7S72100_CLK_SDHI10>,
<&mstp12_clks R7S72100_CLK_SDHI11>;
clock-names = "core", "cd";
+ power-domains = <&cpg_clocks>;
cap-sd-highspeed;
cap-sdio-irq;
status = "disabled";
--
2.10.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-02-09 13:38 [PATCH] ARM: dts: r7s72100: add power-domains to sdhi Chris Brandt
@ 2017-03-22 13:25 ` Geert Uytterhoeven
[not found] ` <CAMuHMdXRD3o7KdaADvPYxzeuEVo44nRVWHfCverPc+9s=0owPA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-03-22 13:25 UTC (permalink / raw)
To: Chris Brandt
Cc: Simon Horman, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
(this time reply-to-all)
On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
> Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
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] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-22 13:25 ` Geert Uytterhoeven
@ 2017-03-24 7:02 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-03-24 7:02 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas
On Wed, Mar 22, 2017 at 02:25:04PM +0100, Geert Uytterhoeven wrote:
> (this time reply-to-all)
>
> On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> > Reported-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > Signed-off-by: Chris Brandt <chris.brandt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
> > Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
>
> Acked-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
Thanks, I have queued this up for v4.12.
The fixes tag above indicates this is a fix for v4.10, however, when I
tried to apply it on top of v4.11-rc1 there was a conflict. So I think a
backport will be required if we want it to be considered for v4.11 and be
considered for and in turn v4.10-stable.
This makes things a bit messy with regards to conflicts between v4.11 and
v4.12 and I'm inclined to pass on the backport.
--
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] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
@ 2017-03-24 7:02 ` Simon Horman
0 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-03-24 7:02 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
On Wed, Mar 22, 2017 at 02:25:04PM +0100, Geert Uytterhoeven wrote:
> (this time reply-to-all)
>
> On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> > Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, I have queued this up for v4.12.
The fixes tag above indicates this is a fix for v4.10, however, when I
tried to apply it on top of v4.11-rc1 there was a conflict. So I think a
backport will be required if we want it to be considered for v4.11 and be
considered for and in turn v4.10-stable.
This makes things a bit messy with regards to conflicts between v4.11 and
v4.12 and I'm inclined to pass on the backport.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-24 7:02 ` Simon Horman
(?)
@ 2017-03-24 7:55 ` Geert Uytterhoeven
2017-03-24 9:40 ` Simon Horman
[not found] ` <CAMuHMdUJMEjb8UhaHHK3aE2uEoUpKFtSFXLhYwgJHKu9YDeVHQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
-1 siblings, 2 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-03-24 7:55 UTC (permalink / raw)
To: Simon Horman
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
Hi Simon,
On Fri, Mar 24, 2017 at 8:02 AM, Simon Horman <horms@verge.net.au> wrote:
> On Wed, Mar 22, 2017 at 02:25:04PM +0100, Geert Uytterhoeven wrote:
>> (this time reply-to-all)
>>
>> On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
>> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
>> > Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
>>
>> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Thanks, I have queued this up for v4.12.
>
> The fixes tag above indicates this is a fix for v4.10, however, when I
> tried to apply it on top of v4.11-rc1 there was a conflict. So I think a
> backport will be required if we want it to be considered for v4.11 and be
> considered for and in turn v4.10-stable.
That's because of commit 3d2abda02ad2d06d
("ARM: dts: r7s72100: update sdhi clock bindings")
> This makes things a bit messy with regards to conflicts between v4.11 and
> v4.12 and I'm inclined to pass on the backport.
If you ever want to backport, you'll have two backport changes too the
SDHI driver, too.
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] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-24 7:55 ` Geert Uytterhoeven
@ 2017-03-24 9:40 ` Simon Horman
2017-03-24 9:45 ` Geert Uytterhoeven
[not found] ` <CAMuHMdUJMEjb8UhaHHK3aE2uEoUpKFtSFXLhYwgJHKu9YDeVHQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
1 sibling, 1 reply; 10+ messages in thread
From: Simon Horman @ 2017-03-24 9:40 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
On Fri, Mar 24, 2017 at 08:55:44AM +0100, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Fri, Mar 24, 2017 at 8:02 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Wed, Mar 22, 2017 at 02:25:04PM +0100, Geert Uytterhoeven wrote:
> >> (this time reply-to-all)
> >>
> >> On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
> >> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
> >> > Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
> >>
> >> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > Thanks, I have queued this up for v4.12.
> >
> > The fixes tag above indicates this is a fix for v4.10, however, when I
> > tried to apply it on top of v4.11-rc1 there was a conflict. So I think a
> > backport will be required if we want it to be considered for v4.11 and be
> > considered for and in turn v4.10-stable.
>
> That's because of commit 3d2abda02ad2d06d
> ("ARM: dts: r7s72100: update sdhi clock bindings")
>
> > This makes things a bit messy with regards to conflicts between v4.11 and
> > v4.12 and I'm inclined to pass on the backport.
>
> If you ever want to backport, you'll have two backport changes too the
> SDHI driver, too.
Is that the case if only this patch (and not 3d2abda02ad2d06d) is backported?
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-24 9:40 ` Simon Horman
@ 2017-03-24 9:45 ` Geert Uytterhoeven
0 siblings, 0 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2017-03-24 9:45 UTC (permalink / raw)
To: Simon Horman
Cc: Chris Brandt, Magnus Damm, Rob Herring, Mark Rutland, devicetree,
Linux-Renesas
Hi Simon,
On Fri, Mar 24, 2017 at 10:40 AM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Mar 24, 2017 at 08:55:44AM +0100, Geert Uytterhoeven wrote:
>> On Fri, Mar 24, 2017 at 8:02 AM, Simon Horman <horms@verge.net.au> wrote:
>> > On Wed, Mar 22, 2017 at 02:25:04PM +0100, Geert Uytterhoeven wrote:
>> >> (this time reply-to-all)
>> >>
>> >> On Thu, Feb 9, 2017 at 2:38 PM, Chris Brandt <chris.brandt@renesas.com> wrote:
>> >> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> >> > Signed-off-by: Chris Brandt <chris.brandt@renesas.com>
>> >> > Fixes: 66474697923c ("ARM: dts: r7s72100: add sdhi to device tree")
>> >>
>> >> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> >
>> > Thanks, I have queued this up for v4.12.
>> >
>> > The fixes tag above indicates this is a fix for v4.10, however, when I
>> > tried to apply it on top of v4.11-rc1 there was a conflict. So I think a
>> > backport will be required if we want it to be considered for v4.11 and be
>> > considered for and in turn v4.10-stable.
>>
>> That's because of commit 3d2abda02ad2d06d
>> ("ARM: dts: r7s72100: update sdhi clock bindings")
>>
>> > This makes things a bit messy with regards to conflicts between v4.11 and
>> > v4.12 and I'm inclined to pass on the backport.
>>
>> If you ever want to backport, you'll have two backport changes too the
>> SDHI driver, too.
>
> Is that the case if only this patch (and not 3d2abda02ad2d06d) is backported?
No, but if you backport this patch only, you'll have to handle the conflicts...
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] 10+ messages in thread
* RE: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-24 7:55 ` Geert Uytterhoeven
@ 2017-03-24 11:17 ` Chris Brandt
[not found] ` <CAMuHMdUJMEjb8UhaHHK3aE2uEoUpKFtSFXLhYwgJHKu9YDeVHQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
1 sibling, 0 replies; 10+ messages in thread
From: Chris Brandt @ 2017-03-24 11:17 UTC (permalink / raw)
To: Geert Uytterhoeven, Simon Horman
Cc: Magnus Damm, Rob Herring, Mark Rutland,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1127 bytes --]
On Friday, March 24, 2017, Geert Uytterhoeven wrote:
> > The fixes tag above indicates this is a fix for v4.10, however, when I
> > tried to apply it on top of v4.11-rc1 there was a conflict. So I
> > think a backport will be required if we want it to be considered for
> > v4.11 and be considered for and in turn v4.10-stable.
>
> That's because of commit 3d2abda02ad2d06d
> ("ARM: dts: r7s72100: update sdhi clock bindings")
Between the two, "update sdhi clock bindings" is more important this this patch ("add power-domains").
Without "update sdhi clock bindings", the SDHI /might/ not work.
But, "update sdhi clock bindings", also relies on the driver update 34a1654706c6 ("mmc: sh_mobile_sdhi: add support for 2 clocks").
That one is in v4.11-rc1, but not in 4.10.5.
I just noticed now that 34a1654706c6 does not have "Fixes" in the commit log, so it is probably not marked to go into v4.10-stable, so in that case backporting the dtsi is pointless.
Chris
N§²æìr¸yúèØb²X¬¶Ç§vØ^)Þº{.nÇ+·zøzÚÞz)í
æèw*\x1fjg¬±¨\x1e¶Ý¢j.ïÛ°\½½MúgjÌæa×\x02' ©Þ¢¸\f¢·¦j:+v¨wèjØm¶ÿ¾\a«êçzZ+ùÝ¢j"ú!¶i
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
@ 2017-03-24 11:17 ` Chris Brandt
0 siblings, 0 replies; 10+ messages in thread
From: Chris Brandt @ 2017-03-24 11:17 UTC (permalink / raw)
To: Geert Uytterhoeven, Simon Horman
Cc: Magnus Damm, Rob Herring, Mark Rutland, devicetree, Linux-Renesas
On Friday, March 24, 2017, Geert Uytterhoeven wrote:
> > The fixes tag above indicates this is a fix for v4.10, however, when I
> > tried to apply it on top of v4.11-rc1 there was a conflict. So I
> > think a backport will be required if we want it to be considered for
> > v4.11 and be considered for and in turn v4.10-stable.
>
> That's because of commit 3d2abda02ad2d06d
> ("ARM: dts: r7s72100: update sdhi clock bindings")
Between the two, "update sdhi clock bindings" is more important this this patch ("add power-domains").
Without "update sdhi clock bindings", the SDHI /might/ not work.
But, "update sdhi clock bindings", also relies on the driver update 34a1654706c6 ("mmc: sh_mobile_sdhi: add support for 2 clocks").
That one is in v4.11-rc1, but not in 4.10.5.
I just noticed now that 34a1654706c6 does not have "Fixes" in the commit log, so it is probably not marked to go into v4.10-stable, so in that case backporting the dtsi is pointless.
Chris
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: r7s72100: add power-domains to sdhi
2017-03-24 11:17 ` Chris Brandt
(?)
@ 2017-03-29 8:09 ` Simon Horman
-1 siblings, 0 replies; 10+ messages in thread
From: Simon Horman @ 2017-03-29 8:09 UTC (permalink / raw)
To: Chris Brandt
Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring, Mark Rutland,
devicetree, Linux-Renesas
On Fri, Mar 24, 2017 at 11:17:25AM +0000, Chris Brandt wrote:
> On Friday, March 24, 2017, Geert Uytterhoeven wrote:
> > > The fixes tag above indicates this is a fix for v4.10, however, when I
> > > tried to apply it on top of v4.11-rc1 there was a conflict. So I
> > > think a backport will be required if we want it to be considered for
> > > v4.11 and be considered for and in turn v4.10-stable.
> >
> > That's because of commit 3d2abda02ad2d06d
> > ("ARM: dts: r7s72100: update sdhi clock bindings")
>
>
> Between the two, "update sdhi clock bindings" is more important this this patch ("add power-domains").
> Without "update sdhi clock bindings", the SDHI /might/ not work.
>
>
> But, "update sdhi clock bindings", also relies on the driver update 34a1654706c6 ("mmc: sh_mobile_sdhi: add support for 2 clocks").
> That one is in v4.11-rc1, but not in 4.10.5.
>
> I just noticed now that 34a1654706c6 does not have "Fixes" in the commit log, so it is probably not marked to go into v4.10-stable, so in that case backporting the dtsi is pointless.
We could try to coordinate getting 34a1654706c6 and the dtsi changes into
v4.10-stable if you it is sufficiently important. What are your thoughts on
that?
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-03-29 8:09 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-09 13:38 [PATCH] ARM: dts: r7s72100: add power-domains to sdhi Chris Brandt
2017-03-22 13:25 ` Geert Uytterhoeven
[not found] ` <CAMuHMdXRD3o7KdaADvPYxzeuEVo44nRVWHfCverPc+9s=0owPA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-24 7:02 ` Simon Horman
2017-03-24 7:02 ` Simon Horman
2017-03-24 7:55 ` Geert Uytterhoeven
2017-03-24 9:40 ` Simon Horman
2017-03-24 9:45 ` Geert Uytterhoeven
[not found] ` <CAMuHMdUJMEjb8UhaHHK3aE2uEoUpKFtSFXLhYwgJHKu9YDeVHQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-24 11:17 ` Chris Brandt
2017-03-24 11:17 ` Chris Brandt
2017-03-29 8:09 ` 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.