On Wed, Dec 18, 2013 at 11:35 AM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote:
> On 12/19/13 03:38, Olof Johansson wrote:
>>
>> On Mon, Dec 9, 2013 at 12:56 PM, Kukjin Kim<kgene.kim@xxxxxxxxxxx> 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@xxxxxxxxx>
>>>> 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@xxxxxxxxxxx>
>>>>>
>>>>> Signed-off-by: Yuvaraj Kumar C D<yuvaraj.cd@xxxxxxxxxxx>
>>>>> ---
>>>>> 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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
