* [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
@ 2013-12-09 6:38 Yuvaraj Kumar C D
2013-12-09 6:38 ` [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi Yuvaraj Kumar C D
[not found] ` <1386571118-17793-1-git-send-email-yuvaraj.cd-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 2 replies; 12+ messages in thread
From: Yuvaraj Kumar C D @ 2013-12-09 6:38 UTC (permalink / raw)
To: linux-samsung-soc, kgene.kim, rob.herring, pawel.moll,
mark.rutland, swarren, ian.campbell, devicetree, a.kesavan
Cc: t.figa, alim.akhtar, sonnyrao, dianders, Yuvaraj Kumar C D
Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
for exynos5250") missed out handling the exynos5250 snow dts file.
Signed-off-by : Abhilash Kesavan <a.kesavan@samsung.com>
Signed-off-by: Yuvaraj Kumar C D <yuvaraj.cd@samsung.com>
---
arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
2 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
index 6470536..1ceb62e 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -247,10 +247,6 @@
};
};
- mmc@12210000 {
- status = "disabled";
- };
-
mmc@12220000 {
num-slots = <1>;
supports-highspeed;
diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index a9395c4..67484d1 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -171,11 +171,20 @@
};
};
+ mmc@12200000 {
+ status = "okay";
+ };
+
+ mmc@12220000 {
+ status = "okay";
+ };
+
/*
* On Snow we've got SIP WiFi and so can keep drive strengths low to
* reduce EMI.
*/
mmc@12230000 {
+ status = "okay";
slot@0 {
pinctrl-names = "default";
pinctrl-0 = <&sd3_clk &sd3_cmd &sd3_bus4>;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi
2013-12-09 6:38 [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board Yuvaraj Kumar C D
@ 2013-12-09 6:38 ` Yuvaraj Kumar C D
2013-12-09 16:19 ` Doug Anderson
[not found] ` <1386571118-17793-1-git-send-email-yuvaraj.cd-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
1 sibling, 1 reply; 12+ messages in thread
From: Yuvaraj Kumar C D @ 2013-12-09 6:38 UTC (permalink / raw)
To: linux-samsung-soc, kgene.kim, rob.herring, pawel.moll,
mark.rutland, swarren, ian.campbell, devicetree, a.kesavan
Cc: t.figa, alim.akhtar, sonnyrao, dianders, Yuvaraj Kumar C D
Commit 64c138a ("ARM: dts: Move fifo-depth property from exynos5250
board dts") missed out handling the exynos5250 snow dts file.
Deletes the fifo-depth property, as this property has been moved to
SOC specific exynos5250.dtsi file.
Signed-off-by : Abhilash Kesavan <a.kesavan@samsung.com>
Signed-off-by: Yuvaraj Kumar C D <yuvaraj.cd@samsung.com>
---
arch/arm/boot/dts/cros5250-common.dtsi | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
index 1ceb62e..2dd70e9 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -233,7 +233,6 @@
num-slots = <1>;
supports-highspeed;
broken-cd;
- fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
samsung,dw-mshc-sdr-timing = <2 3>;
@@ -250,7 +249,6 @@
mmc@12220000 {
num-slots = <1>;
supports-highspeed;
- fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
samsung,dw-mshc-sdr-timing = <2 3>;
@@ -269,7 +267,6 @@
num-slots = <1>;
supports-highspeed;
broken-cd;
- fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
samsung,dw-mshc-sdr-timing = <2 3>;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
[not found] ` <1386571118-17793-1-git-send-email-yuvaraj.cd-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2013-12-09 16:16 ` Doug Anderson
2013-12-09 20:56 ` Kukjin Kim
0 siblings, 1 reply; 12+ messages in thread
From: Doug Anderson @ 2013-12-09 16:16 UTC (permalink / raw)
To: Yuvaraj Kumar C D
Cc: linux-samsung-soc, Kukjin Kim, Rob Herring,
pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8, Stephen Warren,
Ian Campbell, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Abhilash Kesavan, Tomasz Figa, Alim Akhtar, Sonny Rao,
Yuvaraj Kumar C D
Yuvaraj,
On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D <yuvaraj.cd-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
> for exynos5250") missed out handling the exynos5250 snow dts file.
>
> Signed-off-by : Abhilash Kesavan <a.kesavan-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
>
> Signed-off-by: Yuvaraj Kumar C D <yuvaraj.cd-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
> ---
> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
> 2 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
> index 6470536..1ceb62e 100644
> --- a/arch/arm/boot/dts/cros5250-common.dtsi
> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
> @@ -247,10 +247,6 @@
> };
> };
>
> - mmc@12210000 {
> - status = "disabled";
> - };
> -
> mmc@12220000 {
> num-slots = <1>;
> supports-highspeed;
> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
> index a9395c4..67484d1 100644
> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> @@ -171,11 +171,20 @@
> };
> };
>
> + mmc@12200000 {
> + status = "okay";
> + };
> +
> + mmc@12220000 {
> + status = "okay";
> + };
> +
I probably would have put the 12200000 and 12220000 ones in
cros5250-common since that's where all of the other stuff relating to
those ports is. Some of it may need to be moved around once we get
the HP Chromebook 11 (exynos5250-spring) in place, but until then it
seems best to keep it all together.
-Doug
--
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] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi
2013-12-09 6:38 ` [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi Yuvaraj Kumar C D
@ 2013-12-09 16:19 ` Doug Anderson
2013-12-09 20:45 ` Kukjin Kim
0 siblings, 1 reply; 12+ messages in thread
From: Doug Anderson @ 2013-12-09 16:19 UTC (permalink / raw)
To: Yuvaraj Kumar C D
Cc: linux-samsung-soc, Kukjin Kim, Rob Herring, pawel.moll,
mark.rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
Yuvaraj,
On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D <yuvaraj.cd@gmail.com> wrote:
> Commit 64c138a ("ARM: dts: Move fifo-depth property from exynos5250
> board dts") missed out handling the exynos5250 snow dts file.
> Deletes the fifo-depth property, as this property has been moved to
> SOC specific exynos5250.dtsi file.
>
> Signed-off-by : Abhilash Kesavan <a.kesavan@samsung.com>
>
> Signed-off-by: Yuvaraj Kumar C D <yuvaraj.cd@samsung.com>
Reviewed-by: Doug Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi
2013-12-09 16:19 ` Doug Anderson
@ 2013-12-09 20:45 ` Kukjin Kim
0 siblings, 0 replies; 12+ messages in thread
From: Kukjin Kim @ 2013-12-09 20:45 UTC (permalink / raw)
To: Doug Anderson
Cc: Yuvaraj Kumar C D, linux-samsung-soc, Kukjin Kim, Rob Herring,
pawel.moll, mark.rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On 12/10/13 01:19, Doug Anderson wrote:
> Yuvaraj,
>
> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D<yuvaraj.cd@gmail.com> wrote:
>> Commit 64c138a ("ARM: dts: Move fifo-depth property from exynos5250
>> board dts") missed out handling the exynos5250 snow dts file.
>> Deletes the fifo-depth property, as this property has been moved to
>> SOC specific exynos5250.dtsi file.
>>
>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
No need space between "by" and ":".
>>
>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>
> Reviewed-by: Doug Anderson<dianders@chromium.org>
Looks good, applied.
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-09 16:16 ` [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board Doug Anderson
@ 2013-12-09 20:56 ` Kukjin Kim
2013-12-18 18:38 ` Olof Johansson
0 siblings, 1 reply; 12+ messages in thread
From: Kukjin Kim @ 2013-12-09 20:56 UTC (permalink / raw)
To: Doug Anderson
Cc: Yuvaraj Kumar C D, linux-samsung-soc, Kukjin Kim, Rob Herring,
pawel.moll, mark.rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On 12/10/13 01:16, Doug Anderson wrote:
> Yuvaraj,
>
> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D<yuvaraj.cd@gmail.com> wrote:
>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
>> for exynos5250") missed out handling the exynos5250 snow dts file.
>>
>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
>>
>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>> ---
>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
>> 2 files changed, 9 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
>> index 6470536..1ceb62e 100644
>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>> @@ -247,10 +247,6 @@
>> };
>> };
>>
>> - mmc@12210000 {
>> - status = "disabled";
>> - };
>> -
>> mmc@12220000 {
>> num-slots =<1>;
>> supports-highspeed;
>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
>> index a9395c4..67484d1 100644
>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>> @@ -171,11 +171,20 @@
>> };
>> };
>>
>> + mmc@12200000 {
>> + status = "okay";
>> + };
>> +
>> + mmc@12220000 {
>> + status = "okay";
>> + };
>> +
>
> I probably would have put the 12200000 and 12220000 ones in
> cros5250-common since that's where all of the other stuff relating to
> those ports is. Some of it may need to be moved around once we get
> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
> seems best to keep it all together.
>
> -Doug
OK, sounds better, I don't know well about the HP Chromebook though ;-)
- Kukjin
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-09 20:56 ` Kukjin Kim
@ 2013-12-18 18:38 ` Olof Johansson
2013-12-18 19:35 ` Kukjin Kim
0 siblings, 1 reply; 12+ messages in thread
From: Olof Johansson @ 2013-12-18 18:38 UTC (permalink / raw)
To: Kukjin Kim
Cc: Doug Anderson, Yuvaraj Kumar C D, linux-samsung-soc, Rob Herring,
Pawel Moll, Mark Rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> On 12/10/13 01:16, Doug Anderson wrote:
>>
>> Yuvaraj,
>>
>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D<yuvaraj.cd@gmail.com>
>> wrote:
>>>
>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
>>> for exynos5250") missed out handling the exynos5250 snow dts file.
>>>
>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
>>>
>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>>> ---
>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
>>> 2 files changed, 9 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
>>> b/arch/arm/boot/dts/cros5250-common.dtsi
>>> index 6470536..1ceb62e 100644
>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>>> @@ -247,10 +247,6 @@
>>> };
>>> };
>>>
>>> - mmc@12210000 {
>>> - status = "disabled";
>>> - };
>>> -
>>> mmc@12220000 {
>>> num-slots =<1>;
>>> supports-highspeed;
>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
>>> b/arch/arm/boot/dts/exynos5250-snow.dts
>>> index a9395c4..67484d1 100644
>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>>> @@ -171,11 +171,20 @@
>>> };
>>> };
>>>
>>> + mmc@12200000 {
>>> + status = "okay";
>>> + };
>>> +
>>> + mmc@12220000 {
>>> + status = "okay";
>>> + };
>>> +
>>
>>
>> I probably would have put the 12200000 and 12220000 ones in
>> cros5250-common since that's where all of the other stuff relating to
>> those ports is. Some of it may need to be moved around once we get
>> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
>> seems best to keep it all together.
>>
>> -Doug
>
>
> OK, sounds better, I don't know well about the HP Chromebook though ;-)
This broke linux-next on snow ever since the original patch went in.
Please squash the above into the original patch or at the very least
apply it as soon as possible.
-Olof
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-18 18:38 ` Olof Johansson
@ 2013-12-18 19:35 ` Kukjin Kim
2013-12-23 21:43 ` Olof Johansson
0 siblings, 1 reply; 12+ messages in thread
From: Kukjin Kim @ 2013-12-18 19:35 UTC (permalink / raw)
To: Olof Johansson
Cc: Kukjin Kim, Doug Anderson, Yuvaraj Kumar C D, linux-samsung-soc,
Rob Herring, Pawel Moll, Mark Rutland, Stephen Warren,
Ian Campbell, devicetree@vger.kernel.org, Abhilash Kesavan,
Tomasz Figa, Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On 12/19/13 03:38, Olof Johansson wrote:
> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@samsung.com> wrote:
>> On 12/10/13 01:16, Doug Anderson wrote:
>>>
>>> Yuvaraj,
>>>
>>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D<yuvaraj.cd@gmail.com>
>>> wrote:
>>>>
>>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
>>>> for exynos5250") missed out handling the exynos5250 snow dts file.
>>>>
>>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
>>>>
>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>>>> ---
>>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
>>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
>>>> 2 files changed, 9 insertions(+), 4 deletions(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
>>>> b/arch/arm/boot/dts/cros5250-common.dtsi
>>>> index 6470536..1ceb62e 100644
>>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>>>> @@ -247,10 +247,6 @@
>>>> };
>>>> };
>>>>
>>>> - mmc@12210000 {
>>>> - status = "disabled";
>>>> - };
>>>> -
>>>> mmc@12220000 {
>>>> num-slots =<1>;
>>>> supports-highspeed;
>>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
>>>> b/arch/arm/boot/dts/exynos5250-snow.dts
>>>> index a9395c4..67484d1 100644
>>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>>>> @@ -171,11 +171,20 @@
>>>> };
>>>> };
>>>>
>>>> + mmc@12200000 {
>>>> + status = "okay";
>>>> + };
>>>> +
>>>> + mmc@12220000 {
>>>> + status = "okay";
>>>> + };
>>>> +
>>>
>>>
>>> I probably would have put the 12200000 and 12220000 ones in
>>> cros5250-common since that's where all of the other stuff relating to
>>> those ports is. Some of it may need to be moved around once we get
>>> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
>>> seems best to keep it all together.
>>>
>>> -Doug
>>
>>
>> OK, sounds better, I don't know well about the HP Chromebook though ;-)
>
> This broke linux-next on snow ever since the original patch went in.
> Please squash the above into the original patch or at the very least
> apply it as soon as possible.
>
Oops, OK. I've applied this separately just now.
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-18 19:35 ` Kukjin Kim
@ 2013-12-23 21:43 ` Olof Johansson
2013-12-24 1:26 ` Kukjin Kim
0 siblings, 1 reply; 12+ messages in thread
From: Olof Johansson @ 2013-12-23 21:43 UTC (permalink / raw)
To: Kukjin Kim
Cc: Doug Anderson, Yuvaraj Kumar C D, linux-samsung-soc, Rob Herring,
Pawel Moll, Mark Rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On Wed, Dec 18, 2013 at 11:35 AM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> On 12/19/13 03:38, Olof Johansson wrote:
>>
>> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@samsung.com> wrote:
>>>
>>> On 12/10/13 01:16, Doug Anderson wrote:
>>>>
>>>>
>>>> Yuvaraj,
>>>>
>>>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C D<yuvaraj.cd@gmail.com>
>>>> wrote:
>>>>>
>>>>>
>>>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
>>>>> for exynos5250") missed out handling the exynos5250 snow dts file.
>>>>>
>>>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
>>>>>
>>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>>>>> ---
>>>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
>>>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
>>>>> 2 files changed, 9 insertions(+), 4 deletions(-)
>>>>>
>>>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
>>>>> b/arch/arm/boot/dts/cros5250-common.dtsi
>>>>> index 6470536..1ceb62e 100644
>>>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>>>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>>>>> @@ -247,10 +247,6 @@
>>>>> };
>>>>> };
>>>>>
>>>>> - mmc@12210000 {
>>>>> - status = "disabled";
>>>>> - };
>>>>> -
>>>>> mmc@12220000 {
>>>>> num-slots =<1>;
>>>>> supports-highspeed;
>>>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
>>>>> b/arch/arm/boot/dts/exynos5250-snow.dts
>>>>> index a9395c4..67484d1 100644
>>>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>>>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>>>>> @@ -171,11 +171,20 @@
>>>>> };
>>>>> };
>>>>>
>>>>> + mmc@12200000 {
>>>>> + status = "okay";
>>>>> + };
>>>>> +
>>>>> + mmc@12220000 {
>>>>> + status = "okay";
>>>>> + };
>>>>> +
>>>>
>>>>
>>>>
>>>> I probably would have put the 12200000 and 12220000 ones in
>>>> cros5250-common since that's where all of the other stuff relating to
>>>> those ports is. Some of it may need to be moved around once we get
>>>> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
>>>> seems best to keep it all together.
>>>>
>>>> -Doug
>>>
>>>
>>>
>>> OK, sounds better, I don't know well about the HP Chromebook though ;-)
>>
>>
>> This broke linux-next on snow ever since the original patch went in.
>> Please squash the above into the original patch or at the very least
>> apply it as soon as possible.
>>
> Oops, OK. I've applied this separately just now.
Since you hadn't applied this to the branch you sent me, I now have an
arm-soc for-next branch that doesn't boot on Chromebooks.
I've picked up the above patch directly into arm-soc. Please drop it
from your branch.
-Olof
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-23 21:43 ` Olof Johansson
@ 2013-12-24 1:26 ` Kukjin Kim
2013-12-24 2:55 ` Olof Johansson
0 siblings, 1 reply; 12+ messages in thread
From: Kukjin Kim @ 2013-12-24 1:26 UTC (permalink / raw)
To: 'Olof Johansson'
Cc: 'Doug Anderson', 'Yuvaraj Kumar C D',
'linux-samsung-soc', 'Rob Herring',
'Pawel Moll', 'Mark Rutland',
'Stephen Warren', 'Ian Campbell', devicetree,
'Abhilash Kesavan', 'Tomasz Figa',
'Alim Akhtar', 'Sonny Rao',
'Yuvaraj Kumar C D'
Olof Johansson wrote:
>
> On Wed, Dec 18, 2013 at 11:35 AM, Kukjin Kim <kgene.kim@samsung.com>
wrote:
> > On 12/19/13 03:38, Olof Johansson wrote:
> >>
> >> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@samsung.com>
> wrote:
> >>>
> >>> On 12/10/13 01:16, Doug Anderson wrote:
> >>>>
> >>>>
> >>>> Yuvaraj,
> >>>>
> >>>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C
> D<yuvaraj.cd@gmail.com>
> >>>> wrote:
> >>>>>
> >>>>>
> >>>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
> >>>>> for exynos5250") missed out handling the exynos5250 snow dts file.
> >>>>>
> >>>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
> >>>>>
> >>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
> >>>>> ---
> >>>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
> >>>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
> >>>>> 2 files changed, 9 insertions(+), 4 deletions(-)
> >>>>>
> >>>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
> >>>>> b/arch/arm/boot/dts/cros5250-common.dtsi
> >>>>> index 6470536..1ceb62e 100644
> >>>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
> >>>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
> >>>>> @@ -247,10 +247,6 @@
> >>>>> };
> >>>>> };
> >>>>>
> >>>>> - mmc@12210000 {
> >>>>> - status = "disabled";
> >>>>> - };
> >>>>> -
> >>>>> mmc@12220000 {
> >>>>> num-slots =<1>;
> >>>>> supports-highspeed;
> >>>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
> >>>>> b/arch/arm/boot/dts/exynos5250-snow.dts
> >>>>> index a9395c4..67484d1 100644
> >>>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> >>>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> >>>>> @@ -171,11 +171,20 @@
> >>>>> };
> >>>>> };
> >>>>>
> >>>>> + mmc@12200000 {
> >>>>> + status = "okay";
> >>>>> + };
> >>>>> +
> >>>>> + mmc@12220000 {
> >>>>> + status = "okay";
> >>>>> + };
> >>>>> +
> >>>>
> >>>>
> >>>>
> >>>> I probably would have put the 12200000 and 12220000 ones in
> >>>> cros5250-common since that's where all of the other stuff relating to
> >>>> those ports is. Some of it may need to be moved around once we get
> >>>> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
> >>>> seems best to keep it all together.
> >>>>
> >>>> -Doug
> >>>
> >>>
> >>>
> >>> OK, sounds better, I don't know well about the HP Chromebook though
;-)
> >>
> >>
> >> This broke linux-next on snow ever since the original patch went in.
> >> Please squash the above into the original patch or at the very least
> >> apply it as soon as possible.
> >>
> > Oops, OK. I've applied this separately just now.
>
> Since you hadn't applied this to the branch you sent me, I now have an
> arm-soc for-next branch that doesn't boot on Chromebooks.
>
Because 1st pull request already has been sent out to arm-soc when I picked
this into samsung tree after your inform. So I couldn't.
> I've picked up the above patch directly into arm-soc. Please drop it
> from your branch.
>
You mean, I need to re-sort out my 2nd pull-request already sent out? IMHO,
would be better to us if you could pull my 2nd pull-request and drop your
picking up...
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-24 1:26 ` Kukjin Kim
@ 2013-12-24 2:55 ` Olof Johansson
2013-12-24 3:12 ` Kukjin Kim
0 siblings, 1 reply; 12+ messages in thread
From: Olof Johansson @ 2013-12-24 2:55 UTC (permalink / raw)
To: Kukjin Kim
Cc: Doug Anderson, Yuvaraj Kumar C D, linux-samsung-soc, Rob Herring,
Pawel Moll, Mark Rutland, Stephen Warren, Ian Campbell,
devicetree@vger.kernel.org, Abhilash Kesavan, Tomasz Figa,
Alim Akhtar, Sonny Rao, Yuvaraj Kumar C D
On Mon, Dec 23, 2013 at 5:26 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Olof Johansson wrote:
>>
>> On Wed, Dec 18, 2013 at 11:35 AM, Kukjin Kim <kgene.kim@samsung.com>
> wrote:
>> > On 12/19/13 03:38, Olof Johansson wrote:
>> >>
>> >> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@samsung.com>
>> wrote:
>> >>>
>> >>> On 12/10/13 01:16, Doug Anderson wrote:
>> >>>>
>> >>>>
>> >>>> Yuvaraj,
>> >>>>
>> >>>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C
>> D<yuvaraj.cd@gmail.com>
>> >>>> wrote:
>> >>>>>
>> >>>>>
>> >>>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
>> >>>>> for exynos5250") missed out handling the exynos5250 snow dts file.
>> >>>>>
>> >>>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
>> >>>>>
>> >>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
>> >>>>> ---
>> >>>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
>> >>>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
>> >>>>> 2 files changed, 9 insertions(+), 4 deletions(-)
>> >>>>>
>> >>>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
>> >>>>> b/arch/arm/boot/dts/cros5250-common.dtsi
>> >>>>> index 6470536..1ceb62e 100644
>> >>>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>> >>>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>> >>>>> @@ -247,10 +247,6 @@
>> >>>>> };
>> >>>>> };
>> >>>>>
>> >>>>> - mmc@12210000 {
>> >>>>> - status = "disabled";
>> >>>>> - };
>> >>>>> -
>> >>>>> mmc@12220000 {
>> >>>>> num-slots =<1>;
>> >>>>> supports-highspeed;
>> >>>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
>> >>>>> b/arch/arm/boot/dts/exynos5250-snow.dts
>> >>>>> index a9395c4..67484d1 100644
>> >>>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>> >>>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>> >>>>> @@ -171,11 +171,20 @@
>> >>>>> };
>> >>>>> };
>> >>>>>
>> >>>>> + mmc@12200000 {
>> >>>>> + status = "okay";
>> >>>>> + };
>> >>>>> +
>> >>>>> + mmc@12220000 {
>> >>>>> + status = "okay";
>> >>>>> + };
>> >>>>> +
>> >>>>
>> >>>>
>> >>>>
>> >>>> I probably would have put the 12200000 and 12220000 ones in
>> >>>> cros5250-common since that's where all of the other stuff relating to
>> >>>> those ports is. Some of it may need to be moved around once we get
>> >>>> the HP Chromebook 11 (exynos5250-spring) in place, but until then it
>> >>>> seems best to keep it all together.
>> >>>>
>> >>>> -Doug
>> >>>
>> >>>
>> >>>
>> >>> OK, sounds better, I don't know well about the HP Chromebook though
> ;-)
>> >>
>> >>
>> >> This broke linux-next on snow ever since the original patch went in.
>> >> Please squash the above into the original patch or at the very least
>> >> apply it as soon as possible.
>> >>
>> > Oops, OK. I've applied this separately just now.
>>
>> Since you hadn't applied this to the branch you sent me, I now have an
>> arm-soc for-next branch that doesn't boot on Chromebooks.
>>
> Because 1st pull request already has been sent out to arm-soc when I picked
> this into samsung tree after your inform. So I couldn't.
Ah. A comment on the old pull request to give us a heads up would have
been useful.
>> I've picked up the above patch directly into arm-soc. Please drop it
>> from your branch.
>>
> You mean, I need to re-sort out my 2nd pull-request already sent out? IMHO,
> would be better to us if you could pull my 2nd pull-request and drop your
> picking up...
Ok, I'll look at doing that. It's unfortunate that it's broken across
a wider set of changes though, if we ever need to bisect across it for
some reason.
-Olof
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board
2013-12-24 2:55 ` Olof Johansson
@ 2013-12-24 3:12 ` Kukjin Kim
0 siblings, 0 replies; 12+ messages in thread
From: Kukjin Kim @ 2013-12-24 3:12 UTC (permalink / raw)
To: 'Olof Johansson'
Cc: 'Doug Anderson', 'Yuvaraj Kumar C D',
'linux-samsung-soc', 'Rob Herring',
'Pawel Moll', 'Mark Rutland',
'Stephen Warren', 'Ian Campbell', devicetree,
'Abhilash Kesavan', 'Tomasz Figa',
'Alim Akhtar', 'Sonny Rao',
'Yuvaraj Kumar C D'
Olof Johansson wrote:
>
> On Mon, Dec 23, 2013 at 5:26 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> > Olof Johansson wrote:
> >>
> >> On Wed, Dec 18, 2013 at 11:35 AM, Kukjin Kim <kgene.kim@samsung.com>
> > wrote:
> >> > On 12/19/13 03:38, Olof Johansson wrote:
> >> >>
> >> >> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@samsung.com>
> >> wrote:
> >> >>>
> >> >>> On 12/10/13 01:16, Doug Anderson wrote:
> >> >>>>
> >> >>>>
> >> >>>> Yuvaraj,
> >> >>>>
> >> >>>> On Sun, Dec 8, 2013 at 10:38 PM, Yuvaraj Kumar C
> >> D<yuvaraj.cd@gmail.com>
> >> >>>> wrote:
> >> >>>>>
> >> >>>>>
> >> >>>>> Commit 0c3de788 ("ARM: dts: change status property of dwmmc nodes
> >> >>>>> for exynos5250") missed out handling the exynos5250 snow dts
file.
> >> >>>>>
> >> >>>>> Signed-off-by : Abhilash Kesavan<a.kesavan@samsung.com>
> >> >>>>>
> >> >>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@samsung.com>
> >> >>>>> ---
> >> >>>>> arch/arm/boot/dts/cros5250-common.dtsi | 4 ----
> >> >>>>> arch/arm/boot/dts/exynos5250-snow.dts | 9 +++++++++
> >> >>>>> 2 files changed, 9 insertions(+), 4 deletions(-)
> >> >>>>>
> >> >>>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi
> >> >>>>> b/arch/arm/boot/dts/cros5250-common.dtsi
> >> >>>>> index 6470536..1ceb62e 100644
> >> >>>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
> >> >>>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
> >> >>>>> @@ -247,10 +247,6 @@
> >> >>>>> };
> >> >>>>> };
> >> >>>>>
> >> >>>>> - mmc@12210000 {
> >> >>>>> - status = "disabled";
> >> >>>>> - };
> >> >>>>> -
> >> >>>>> mmc@12220000 {
> >> >>>>> num-slots =<1>;
> >> >>>>> supports-highspeed;
> >> >>>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts
> >> >>>>> b/arch/arm/boot/dts/exynos5250-snow.dts
> >> >>>>> index a9395c4..67484d1 100644
> >> >>>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> >> >>>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> >> >>>>> @@ -171,11 +171,20 @@
> >> >>>>> };
> >> >>>>> };
> >> >>>>>
> >> >>>>> + mmc@12200000 {
> >> >>>>> + status = "okay";
> >> >>>>> + };
> >> >>>>> +
> >> >>>>> + mmc@12220000 {
> >> >>>>> + status = "okay";
> >> >>>>> + };
> >> >>>>> +
> >> >>>>
> >> >>>>
> >> >>>>
> >> >>>> I probably would have put the 12200000 and 12220000 ones in
> >> >>>> cros5250-common since that's where all of the other stuff relating
> to
> >> >>>> those ports is. Some of it may need to be moved around once we
> get
> >> >>>> the HP Chromebook 11 (exynos5250-spring) in place, but until then
> it
> >> >>>> seems best to keep it all together.
> >> >>>>
> >> >>>> -Doug
> >> >>>
> >> >>>
> >> >>>
> >> >>> OK, sounds better, I don't know well about the HP Chromebook though
> > ;-)
> >> >>
> >> >>
> >> >> This broke linux-next on snow ever since the original patch went in.
> >> >> Please squash the above into the original patch or at the very least
> >> >> apply it as soon as possible.
> >> >>
> >> > Oops, OK. I've applied this separately just now.
> >>
> >> Since you hadn't applied this to the branch you sent me, I now have an
> >> arm-soc for-next branch that doesn't boot on Chromebooks.
> >>
> > Because 1st pull request already has been sent out to arm-soc when I
> picked
> > this into samsung tree after your inform. So I couldn't.
>
> Ah. A comment on the old pull request to give us a heads up would have
> been useful.
>
You're right. OK, I will.
> >> I've picked up the above patch directly into arm-soc. Please drop it
> >> from your branch.
> >>
> > You mean, I need to re-sort out my 2nd pull-request already sent out?
> IMHO,
> > would be better to us if you could pull my 2nd pull-request and drop
> your
> > picking up...
>
> Ok, I'll look at doing that. It's unfortunate that it's broken across
> a wider set of changes though, if we ever need to bisect across it for
> some reason.
>
Agreed, I will look at my branch with more attention.
Thanks a lot.
- Kukjin
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-12-24 3:12 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-09 6:38 [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board Yuvaraj Kumar C D
2013-12-09 6:38 ` [PATCH 2/2] ARM: dts:Move fifo-depth property from cros5250-common.dtsi Yuvaraj Kumar C D
2013-12-09 16:19 ` Doug Anderson
2013-12-09 20:45 ` Kukjin Kim
[not found] ` <1386571118-17793-1-git-send-email-yuvaraj.cd-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-09 16:16 ` [PATCH 1/2] ARM: dts: Fix status property of mmc nodes for snow board Doug Anderson
2013-12-09 20:56 ` Kukjin Kim
2013-12-18 18:38 ` Olof Johansson
2013-12-18 19:35 ` Kukjin Kim
2013-12-23 21:43 ` Olof Johansson
2013-12-24 1:26 ` Kukjin Kim
2013-12-24 2:55 ` Olof Johansson
2013-12-24 3:12 ` Kukjin Kim
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).