From: khilman@kernel•org (Kevin Hilman)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 1/1] ARM: exynos_defconfig: re-enable USB gadget and max77802 options
Date: Fri, 10 Oct 2014 15:13:52 -0700 [thread overview]
Message-ID: <7hvbnrindr.fsf@deeprootsystems.com> (raw)
In-Reply-To: <CABxcv==bbfoY6=g3P6N2bLw6DqoV8KEpgro5VOdrj6A9OuuTmA@mail.gmail.com> (Javier Martinez Canillas's message of "Fri, 10 Oct 2014 14:25:27 +0200")
Javier Martinez Canillas <javier@dowhile0•org> writes:
> Hello Kevin,
>
> On Fri, Oct 10, 2014 at 1:34 AM, Kevin Hilman <khilman@kernel•org> wrote:
>> Javier Martinez Canillas <javier.martinez@collabora•co.uk> writes:
>>
>>> Commit 43eeaa42e03a ("ARM: exynos_defconfig: savedefconfig") removed a
>>> set of Kconfig symbols. For most of them there were no functional change
>>> since are selected by other Kconfig options or were deprecated but some
>>> options are not explicitly selected so they should not had been removed.
>>>
>>> The options that have to be enabled are USB gadget and the MAX77802 PMIC
>>> support which were enabled in commits:
>>>
>>> 508423bebcda ("ARM: exynos_defconfig: enable USB gadget support")
>>> 6e80e3d87549 ("ARM: exynos_defconfig: Enable MAX77802")
>>>
>>> Enable those options to leave the config in the state before the change.
>>>
>>> Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora•co.uk>
>>
>> Acked-by: Kevin Hilman <khilman@linaro•org>
>> Tested-by: Kevin Hilman <khilman@linaro•org>
>
> Thanks for testing.
>
>>
>> This is needed to get RTC wakeup from suspend working on
>> exynos5800-peach-pi.
>>
>> Note that the s3c-rtc works fine, but the max77802-rtc doesn't seem to
>> work be functional for me:
>>
>> [ 2.408178] max77802-rtc max77802-rtc: rtc core: registered max77802-rtc as rtc0
>> [ 3.595485] s3c-rtc 101e0000.rtc: rtc core: registered s3c as rtc1
>>
>> root@(none):/# hwclock --rtc /dev/rtc0
>> hwclock: ioctl(RTC_RD_TIME) to /dev/rtc0 to read the time failed:
>> Invalid argument
>> root@(none):/# hwclock --rtc /dev/rtc1
>> Thu Oct 9 23:33:06 2014 -0.111978 seconds
>>
>
> Strange, I don't get that error when testing on my Peach Pit with
> linux-next + $subject
>
> [ 2.311591] max77802-rtc max77802-rtc: rtc core: registered
> max77802-rtc as rtc0
> [ 3.594438] s3c-rtc 101e0000.rtc: rtc core: registered s3c as rtc1
>
> # hwclock --rtc /dev/rtc0
> Fri 10 Oct 2014 12:19:23 PM UTC -0.909103 seconds
> # hwclock --rtc /dev/rtc1
> Fri 10 Oct 2014 12:19:26 PM UTC -0.719862 seconds
>
> I'll take a look but if you have a test case that makes it fail
> consistently that would be really helpful.
I'm using:
- next-20141010 + $subject patch
- exynos_defconfig
- exynos5800-peach-pi
- ubuntu-based rootfs, but booting with init=/bin/bash, so not much
userspace involved
Boot-time RTC-related messages:
root@(none):/# dmesg |grep rtc
[ 2.349742] s3c-rtc 101e0000.rtc: failed to find rtc source clock
[ 2.349795] platform 101e0000.rtc: Driver s3c-rtc requests probe deferral
[ 2.373313] max77802-rtc max77802-rtc: rtc core: registered max77802-rtc as rtc0
[ 3.590520] s3c-rtc 101e0000.rtc: rtc core: registered s3c as rtc1
[ 3.925792] max77802-rtc max77802-rtc: hctosys: unable to read the hardware clock
Note there was an "unable to read" failure during boot too.
Then it fails like this every time when trying from userspace:
oot@(none):/# hwclock --rtc /dev/rtc0
hwclock: ioctl(RTC_RD_TIME) to /dev/rtc0 to read the time failed: Invalid argument
root@(none):/# hwclock --rtc /dev/rtc1
Fri Oct 10 22:09:47 2014 -0.375445 seconds
FWIW, I don't think this problem should hold up $SUBJECT patch from
being merged, as it's not directly related.
Kevin
prev parent reply other threads:[~2014-10-10 22:13 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-09 18:36 [PATCH 1/1] ARM: exynos_defconfig: re-enable USB gadget and max77802 options Javier Martinez Canillas
2014-10-09 23:34 ` Kevin Hilman
2014-10-10 12:25 ` Javier Martinez Canillas
2014-10-10 22:13 ` Kevin Hilman [this message]
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=7hvbnrindr.fsf@deeprootsystems.com \
--to=khilman@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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