From: gregory.clement@free-electrons•com (Gregory CLEMENT)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v2 2/3] soc: dove: add legacy support to PMU driver
Date: Mon, 07 Dec 2015 22:24:58 +0100 [thread overview]
Message-ID: <878u569ck5.fsf@free-electrons.com> (raw)
In-Reply-To: <20151207211656.GZ8644@n2100.arm.linux.org.uk> (Russell King's message of "Mon, 7 Dec 2015 21:16:56 +0000")
Hi Russell King,
On lun., d?c. 07 2015, Russell King - ARM Linux <linux@arm•linux.org.uk> wrote:
> On Mon, Dec 07, 2015 at 10:09:27PM +0100, Gregory CLEMENT wrote:
>> Hi Russell,
>>
>> On lun., d?c. 07 2015, Gregory CLEMENT <gregory.clement@free-electrons•com> wrote:
>>
>> > On lun., d?c. 07 2015, Gregory CLEMENT <gregory.clement@free-electrons•com> wrote:
>> >
>> >> Hi Russell,
>> >>
>> >> On lun., d?c. 07 2015, Russell King <rmk+kernel@arm•linux.org.uk> wrote:
>> >>
>> >>> Add support for legacy non-DT Dove to the PMU driver, so that we can
>> >>> transition the legacy support over.
>> >>>
>> >>> Signed-off-by: Russell King <rmk+kernel@arm•linux.org.uk>
>> >>
>> >> Applied on mvebu/soc (and fixed a conflict in drivers/soc/Makefile)
>> >
>> > Eventually I applied it on mvebu/drivers rather than mvebu/soc
>>
>> Actually I was too optimitic. It doesn't build on mvebu/soc (based on
>> 4.4-rc1).
>>
>> First pm_genpd_poweroff_unused() had been removed since bb4b72fc63d4 "PM
>> / Domains: Remove pm_genpd_poweroff_unused() API ".
>>
>> Then, even by removing the call this function to go furthetr I got other
>> errors such as: drivers/soc/dove/pmu.c:344:46: error:
>> ?IRQ_DOVE_PMU_START? undeclared (first use in this function)
>
> I guess that's with Arnd's patches? Sigh, okay, it'll have to wait yet
> _another_ kernel cycle then.
Yes I think the breakage comes from "ARM: orion: use SPARSE_IRQ
everywhere". With this one mach/irqs.h is no more visible by
drivers/soc/dove/pmu.c. But I am sure we can find a solution. It could
not be the first time there is such issue when moving to multiplatform.
Arnd, do you have some suggestion?
Thanks,
Gregory
>
> --
> RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
> FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
> according to speedtest.net.
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2015-12-07 21:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-06 23:52 [PATCH v2 2/3] soc: dove: add legacy support to PMU driver Russell King
2015-12-07 9:12 ` Arnd Bergmann
2015-12-07 20:37 ` Gregory CLEMENT
2015-12-07 20:42 ` Gregory CLEMENT
2015-12-07 20:44 ` Russell King - ARM Linux
2015-12-07 20:48 ` Gregory CLEMENT
2015-12-07 21:09 ` Gregory CLEMENT
2015-12-07 21:16 ` Russell King - ARM Linux
2015-12-07 21:24 ` Gregory CLEMENT [this message]
2015-12-07 23:15 ` Arnd Bergmann
2015-12-08 10:33 ` Gregory CLEMENT
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=878u569ck5.fsf@free-electrons.com \
--to=gregory.clement@free-electrons$(echo .)com \
--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