Linux Samsung SOC development
 help / color / mirror / Atom feed
From: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Subject: Re: exynos4412: misc issues on Hardkernel Odroid boards
Date: Sun, 03 May 2015 14:39:28 +0200	[thread overview]
Message-ID: <55461700.1000308@math.uni-bielefeld.de> (raw)
In-Reply-To: <553A27EE.5000800@samsung.com>

Hello Marek,

Marek Szyprowski wrote:
> Hello Tobias,
> 
> On 2015-03-03 15:10, Marek Szyprowski wrote:
>> Hello,
>>
>> On 2015-02-24 14:39, Tobias Jakobi wrote:
>>> Hello Marek,
>>>
>>> sorry for the late response!
>>>
>>>
>>> On 2015-02-10 12:38, Marek Szyprowski wrote:
>>>> Hello,
>>>>
>>>> On 2015-02-09 21:28, Tobias Jakobi wrote:
>>>>> Hello!
>>>>>
>>>>> Marek Szyprowski wrote:
>>>>>> I didn't observe any issues with 'ondemand' governor. I've just
>>>>>> tested it
>>>>>> on next-20150204 with "[PATCH v2 0/3] Add support for hardware
>>>>>> reset of
>>>>>> eMMC
>>>>>> card on reboot" patches added
>>>>>> (http://www.spinics.net/lists/linux-mmc/msg30621.html) and reboot
>>>>>> works
>>>>>> fine.
>>>>> Since I'm not using an eMMC card, I don't think the pwrseq stuff is
>>>>> going to make a difference here.
>>>>> Have you checked this with the PS_HOLD patch applied, or without?
>>>>> Anyway, guess I'm going to recheck this, maybe with some other
>>>>> governors
>>>>> too.
>>>>
>>>> PS_HOLD patch is related to shutdown behavior. Reboot doesn't use
>>>> it. Could you
>>>> point which exactly version did you use and provide the .config
>>>> which causes
>>>> problems with normal reboot when system runs from SD card? For eMMC
>>>> you would
>>>> need patches from last linux-next.
>>>
>>> I'm currently using this tree:
>>> https://github.com/tobiasjakobi/linux-odroid
>>>
>>> In this is the config:
>>> https://raw.githubusercontent.com/tobiasjakobi/odroid-environment/master/sourcecode/system/vanilla-3.19-debug.conf
>>>
>>>
>>> Upon issuing reboot I get this here on the UART:
>>> chidori ~ # reboot
>>>
>>> Broadcast message from root@chidori (ttySAC1) (Mon Feb 23 17:04:44
>>> 2015):
>>>
>>> The system is going down for reboot NOW!
>>> INIT: Switching to runlevel: 6
>>> INIT: Sending processes the TERM signal
>>> chidori ~ #  * Stopping
>>> local                                             [ ok ]
>>>  * Stopping vixie-cron
>>> ...                                                [ ok ]
>>>  * Saving random seed
>>> ...                                                 [ ok ]
>>>  * Deactivating additional swap space
>>> ...                                 [ ok ]
>>>  * Stopping sshd
>>> ...                                                      [ ok ]
>>>  * Unmounting network filesystems
>>> ...                                     [ ok ]
>>>  * Stopping chronyd
>>> ...                                                   [ ok ]
>>>  * Stopping BlueTooth NAP
>>> ...                                             [ ok ]
>>>  * Stopping dnsmasq
>>> ...                                                   [ ok ]
>>>  * Stopping syslog-ng
>>> ...                                                 [ ok ]
>>>  * Shutting down bluetooth
>>> ...                                            [ ok ]
>>>  * Stopping D-BUS system messagebus
>>> ...                                   [ ok ]
>>>  * Unmounting loop devices
>>>  * Unmounting filesystems
>>>  *   Unmounting /usr/local/emu
>>> ...                                        [ ok ]
>>>  *   Unmounting /home
>>> ...                                                 [ ok ]
>>>  *   Unmounting /var
>>> ...                                                  [ ok ]
>>>  * Deactivating swap devices
>>> ...                                          [ ok ]
>>>  * Stopping udev
>>> ...                                                      [ ok ]
>>>  * Saving the shutdown time
>>> ...                                           [ ok ]
>>>  * Terminating remaining processes
>>> ...                                    [ ok ]
>>>  * Killing remaining processes
>>> ...                                        [ ok ]
>>>  * Saving dependency cache
>>> ...                                            [ ok ]
>>>  * Remounting remaining filesystems read-only ...
>>>  *   Remounting / read only ...
>>> [  113.526626] EXT4-fs (mmcblk0p2): re-mounted. Opts:
>>> (null)              [ ok ]
>>> [  115.609592] sd 0:0:0:0: [sda] Synchronizing SCSI cache
>>> [
>>>
>>>
>>> Notice the open bracket in the last line. Seems like there is still
>>> output in the kernel buffer, but the UART is maybe switched off
>>> before one can see that. At least there is nothing after that, and I
>>> have to manually restart the board then.
>>
>> I've just checked and it works fine with my Odroid U3+ board booted
>> from SD card. I have really no idea what wrong here... Could you check
>> if the same problem exist with vanilla v4.0-rc1 kernel?
> 
> I believe that patch e91d863d1aaeda0349aed42dfbd75c98e042a2c9 ("serial:
> samsung:
> Clear operation mode on UART shutdown") solves this issue. It has been
> merged to
> v4.0-rc7. Could you confirm it?
I've tried some reboot cycles without modifying the cpu governor and so
far all reboot worked. So this issue looks like its fixed.


With best wishes,
Tobias


> 
> Best regards

  reply	other threads:[~2015-05-03 12:39 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-08 15:00 exynos4412: misc issues on Hardkernel Odroid boards Tobias Jakobi
2015-02-08  9:52 ` Krzysztof Kozlowski
2015-02-09  8:00 ` Marek Szyprowski
2015-02-09 20:28   ` Tobias Jakobi
     [not found]     ` <54D9EDC8.6020508@samsung.com>
2015-02-24 13:39       ` Tobias Jakobi
2015-03-03 14:10         ` Marek Szyprowski
2015-04-24 11:24           ` Marek Szyprowski
2015-05-03 12:39             ` Tobias Jakobi [this message]
2015-02-14 18:14   ` Tobias Jakobi
2015-02-17 13:24     ` Krzysztof Kozlowski
2015-02-17 13:32       ` Tobias Jakobi
2015-03-20  1:20         ` Tobias Jakobi
2015-03-20  8:25           ` Krzysztof Kozlowski
2015-03-23 16:30             ` Paul Osmialowski

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=55461700.1000308@math.uni-bielefeld.de \
    --to=tjakobi@math.uni-bielefeld.de \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.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