[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK8P3a3z8tOxz=Rem6N7m_0_8OaU9Sxf7yViZJP5fmYGeaxUvQ@mail.gmail.com>
Date: Mon, 28 Jan 2019 17:41:55 +0100
From: Arnd Bergmann <arnd@...db.de>
To: Sasha Levin <sashal@...nel.org>
Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
"# 3.4.x" <stable@...r.kernel.org>,
Olof Johansson <olof@...om.net>,
Lubomir Rintel <lkundrak@...sk>
Subject: Re: [PATCH AUTOSEL 4.20 175/304] ARM: mmp: fix timer_init calls
On Mon, Jan 28, 2019 at 4:52 PM Sasha Levin <sashal@...nel.org> wrote:
>
> From: Arnd Bergmann <arnd@...db.de>
>
> [ Upstream commit 12d3a30db4a3b3df5fbadf5974b9cf50544a9950 ]
>
> The change to passing the timer frequency as a function argument
> was a good idea, but caused a build failure for one user that
> was missed in the update:
>
> arch/arm/mach-mmp/time.c: In function 'mmp_dt_init_timer':
> arch/arm/mach-mmp/time.c:242:2: error: implicit declaration of function 'timer_init'; did you mean 'hrtimer_init'? [-Werror=implicit-function-declaration]
>
> Change that as well to fix the build error, and rename the
> function to put it into a proper namespace and make it clearer
> what is actually going on.
>
> I saw that the high 6500000 HZ frequency was previously only
> set with CONFIG_MMP2, but is now also used with MMP (pxa910),
> so I'm changing that back here. Please make sure that the
> frequencies are all correct now.
>
> Fixes: f36797ee4380 ("ARM: mmp/mmp2: dt: enable the clock")
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> Signed-off-by: Olof Johansson <olof@...om.net>
> Signed-off-by: Sasha Levin <sashal@...nel.org>
I did not think that Lubumir's patch was meant for stable,
but as you've picked that one already we clearly
want mine as well, so this is good for all stable kernels.
Arnd
Powered by blists - more mailing lists