From: khilman@baylibre.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/3] ARM: davinci: PM: cleanup init, add DT support
Date: Wed, 16 Nov 2016 08:24:59 -0800 [thread overview]
Message-ID: <m260nnxuhw.fsf@baylibre.com> (raw)
In-Reply-To: <c0f1cfa2-4086-72fc-783b-d0d8c837cb08@ti.com> (Sekhar Nori's message of "Wed, 16 Nov 2016 16:50:09 +0530")
Sekhar Nori <nsekhar@ti.com> writes:
> Hi Kevin,
>
> On Wednesday 16 November 2016 01:24 AM, Kevin Hilman wrote:
>> This series removes the fake platform_device used to initialize PM on
>> legacy davinci platforms, and also adds PM support for DT platforms.
>>
>> Tested with legacy boot on da850-evm, and DT boot on da850-lcdk. Used
>> RTC to wake from suspend-to-RAM: e.g. rtcwake -m mem -s 4 -d /dev/rtc0
>
> With these patches applied, I saw the following randconfig build error.
>
> arch/arm/mach-davinci/built-in.o: In function `davinci_pm_init':
> pm_domain.c:(.init.text+0x13d8): undefined reference to `da8xx_get_mem_ctlr'
> pm_domain.c:(.init.text+0x14bc): undefined reference to `da8xx_syscfg1_base'
> make: *** [vmlinux] Error 1
>
> Attached is the defconfig. I did not check to see if your patches
> introduced it or if this is something that always existed and never
> caught.
It looks like a new problem caused by moving those calls into pm.c.
The problem is that currently pm.o can be built even when da850 support
is not enabled, which is the case in your defconfig.
I'll hae a look at how to cleanup that dependency.
Kevin
prev parent reply other threads:[~2016-11-16 16:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-15 19:54 [PATCH v2 0/3] ARM: davinci: PM: cleanup init, add DT support Kevin Hilman
2016-11-15 19:54 ` [PATCH v2 1/3] ARM: davinci: PM: rework init, support DT platforms Kevin Hilman
2016-11-16 9:03 ` Sekhar Nori
2016-11-15 19:54 ` [PATCH v2 2/3] ARM: davinci: PM: cleanup: remove references to pdata Kevin Hilman
2016-11-16 9:09 ` Sekhar Nori
2016-11-15 19:54 ` [PATCH v2 3/3] ARM: davinci: PM: support da8xx DT platforms Kevin Hilman
2016-11-16 9:16 ` Sekhar Nori
[not found] ` <c0f1cfa2-4086-72fc-783b-d0d8c837cb08@ti.com>
2016-11-16 16:24 ` 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=m260nnxuhw.fsf@baylibre.com \
--to=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.