Linux Samsung SOC development
 help / color / mirror / Atom feed
From: Joonyoung Shim <jy0922.shim@samsung.com>
To: Mauro Ribeiro <mdrjr0@gmail.com>
Cc: Kevin Hilman <khilman@kernel.org>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	a.kesavan@samsung.com, k.chander@samsung.com,
	Mauro Ribeiro <mauro.ribeiro@hardkernel.com>,
	abhijeet@abhijeet-dev.net,
	"kgene.kim@samsung.com >> Kukjin Kim" <kgene.kim@samsung.com>
Subject: Re: exynos5422-odroid-xu3: MCPM: only 6 of 8 CPUs coming online
Date: Thu, 23 Oct 2014 10:49:13 +0900	[thread overview]
Message-ID: <54485E99.9020004@samsung.com> (raw)
In-Reply-To: <CAGc3s_e86Lx+pF12osoKoROXq4rUNf_GQtfJ8xB9w-X4pGVOmQ@mail.gmail.com>

Hi,

On 10/23/2014 10:42 AM, Mauro Ribeiro wrote:
> Joonyoung,
> 
> Abhilash looks to be the father of it...
> Also I have no idea why I'm appearing as a commiter of that.. as me or
> anyone from Hardkernel doesn't have access to that github account or
> contributed to it.
> I do have a 3.17 branch here locally on my computer but its not public anywhere.

You are just author of the patch on github and it was committed to github by Abhilash.

Thanks.

> 
> On Wed, Oct 22, 2014 at 11:36 PM, Joonyoung Shim
> <jy0922.shim@samsung.com> wrote:
>> On 10/23/2014 10:25 AM, Mauro Ribeiro wrote:
>>> Joonyoung,
>>>
>>> This patch was provided by Samsung and if someone can explain is you
>>> guys from Samsung.
>>> I have no idea of the logics behind this or what it would take to fix it.
>>> Keep in mind that our u-boot and kernel (downstream 3.10) is part of
>>> Samsung provided BSP.
>>> Would be nice if you guys could explain this.
>>>
>>
>> Mauro,
>>
>> Thanks to reply, do you know author of this patch written by Samsung?
>>
>>
>> To Kukjin,
>>
>> The patch[1] seems to come from S.LSI guys, could you contact about this?
>>
>> [1]: https://github.com/abhijeet-dev/linux-samsung/commit/553b7f40465ab4fad6c270f289cd4780942f7fae
>>
>>>
>>>
>>> On Wed, Oct 22, 2014 at 11:17 PM, Joonyoung Shim
>>> <jy0922.shim@samsung.com> wrote:
>>>> Add Cc the patch author and committer from github.
>>>>
>>>> On 10/23/2014 10:04 AM, Joonyoung Shim wrote:
>>>>> Hi,
>>>>>
>>>>> On 10/17/2014 09:06 PM, Kevin Hilman wrote:
>>>>>> On Wed, Oct 15, 2014 at 4:35 AM, Joonyoung Shim <jy0922.shim@samsung.com> wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> Kevin Hilman <khilman <at> kernel.org> writes:
>>>>>>>
>>>>>>>>
>>>>>>>> Hi hdmi_resources_init,
>>>>>>>>
>>>>>>>> Abhilash Kesavan <a.kesavan <at> samsung.com> writes:
>>>>>>>>
>>>>>>>>> I had noticed a similar issue where only 6 cores would come up during
>>>>>>>>> development on 5420. This was because there was a check in our u-boot
>>>>>>>>> secondary boot-up path which would put a core into WFI if a flag was
>>>>>>>>> not present and hence fail to boot. I wonder if it is the same
>>>>>>>>> situation in your case too.
>>>>>>>>> Is the u-boot for odroid-xu3 publically available or can you check if
>>>>>>> there is any such code present in your
>>>>>>>> secondary bring-up path.
>>>>>>>>
>>>>>>>> I'm using the hardkernel u-boot[1], branch: odroidxu3-v2012.07.  I'm
>>>>>>>> currently using the pre-built binaries that are in the
>>>>>>>> sd_fuse/hardkernel directory of that branch, but I can rebuild u-boot if
>>>>>>>> needed.
>>>>>>>>
>>>>>>>> I'm not very familiar with u-boot, but if f you have some pointers where
>>>>>>>> to start looking in that code, that would be helpful.
>>>>>>>>
>>>>>>>> Kevin
>>>>>>>>
>>>>>>>> [1] https://github.com/hardkernel/u-boot.git
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> It seems to need magic code.
>>>>>>>
>>>>>>> Please refer
>>>>>>> https://github.com/abhijeet-dev/linux-samsung/commit/553b7f40465ab4fad6c270f289cd4780942f7fae
>>>>>>
>>>>>> Thanks, Abhilash had pointed me to the same code, and it works.  But...
>>>>>>
>>>>>> Can someone explain this "magic", ideally in the form of a changelog
>>>>>> and patch submitted upstream?
>>>>>>
>>>>>
>>>>> I also really wonder.
>>>>>
>>>>> One more problem, cpu0 doesn't boot after offline and system is halted.
>>>>>
>>>>> # echo 0 > /sys/devices/system/cpu/cpu0/online
>>>>> [   28.027852] IRQ152 no longer affine to CPU0
>>>>> [   28.028087] IRQ256 no longer affine to CPU0
>>>>> [   28.028344] CPU0: shutdown
>>>>> #
>>>>> # echo 1 > /sys/devices/system/cpu/cpu0/online
>>>>>
>>>>> So, it's failed script[1] to test MCPM functionalities
>>>>>
>>>>> Thanks.
>>>>>
>>>>> [1]: http://www.spinics.net/lists/arm-kernel/msg330946.html
>>>>>
>>>>
>>>> --
>>>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
>>>> the body of a message to majordomo@vger.kernel.org
>>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

  reply	other threads:[~2014-10-23  1:49 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-09  3:06 exynos5422-odroid-xu3: MCPM: only 6 of 8 CPUs coming online Abhilash Kesavan
2014-10-09 21:58 ` Kevin Hilman
2014-10-15 11:35   ` Joonyoung Shim
2014-10-17 12:06     ` Kevin Hilman
2014-10-23  1:04       ` Joonyoung Shim
2014-10-23  1:17         ` Joonyoung Shim
2014-10-23  1:25           ` Mauro Ribeiro
2014-10-23  1:36             ` Joonyoung Shim
2014-10-23  1:42               ` Mauro Ribeiro
2014-10-23  1:49                 ` Joonyoung Shim [this message]
2014-10-23  1:54                   ` Joonyoung Shim
2014-10-23  8:36                     ` Abhijeet Dev
     [not found]                     ` <CAG30XQJ29cM6=OFh=TTbR2rfWGt3AK1o2=PMqdoq6ejUJtUwGA@mail.gmail.com>
     [not found]                       ` <A804D1798D614199AFA7AEDC00248EA9@sisodomain.com>
2014-10-23 15:56                         ` Kevin Hilman
2014-10-24 14:33                           ` Abhilash Kesavan
2014-10-24 16:00                             ` Kevin Hilman
2014-10-25 12:47                               ` Mauro Ribeiro
2014-10-27 19:00                                 ` Kevin Hilman
2014-10-23 18:01           ` Kevin Hilman
  -- strict thread matches above, loose matches on Subject: below --
2014-10-08 18:58 Kevin Hilman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54485E99.9020004@samsung.com \
    --to=jy0922.shim@samsung.com \
    --cc=a.kesavan@samsung.com \
    --cc=abhijeet@abhijeet-dev.net \
    --cc=k.chander@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=khilman@kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=mauro.ribeiro@hardkernel.com \
    --cc=mdrjr0@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox