From: Sebastian Reichel <sre@kernel.org>
To: Tony Lindgren <tony@atomide.com>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>,
Matthijs van Duin <matthijsvanduin@gmail.com>,
Pavel Machek <pavel@ucw.cz>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: ARM errata 430973 on multi platform kernels
Date: Fri, 17 Apr 2015 20:41:56 +0200 [thread overview]
Message-ID: <20150417184156.GA3326@earth> (raw)
In-Reply-To: <20150416160858.GH18048@atomide.com>
[-- Attachment #1: Type: text/plain, Size: 2506 bytes --]
On Thu, Apr 16, 2015 at 09:08:58AM -0700, Tony Lindgren wrote:
> * Sebastian Reichel <sre@kernel.org> [150415 09:32]:
> > Hi,
> >
> > On Thu, Apr 09, 2015 at 02:48:43PM +0100, Russell King - ARM Linux wrote:
> > > On Thu, Apr 09, 2015 at 12:06:58AM +0100, Russell King - ARM Linux wrote:
> > > > On Tue, Apr 07, 2015 at 08:22:08AM -0700, Tony Lindgren wrote:
> > > > > Works for me. The above needs the following fix folded in to build:
> > > > >
> > > > > --- a/arch/arm/mm/proc-v7.S
> > > > > +++ b/arch/arm/mm/proc-v7.S
> > > > > @@ -532,7 +532,7 @@ __v7_ca9mp_proc_info:
> > > > > __v7_ca8_proc_info:
> > > > > .long 0x410fc080
> > > > > .long 0xff0ffff0
> > > > > - __v7_proc __v7_ca8mp_proc_info, proc_fns = ca8_processor_functions
> > > > > + __v7_proc __v7_ca8_proc_info, __v7_setup, proc_fns = ca8_processor_functions
> > > > > .size __v7_ca8_proc_info, . - __v7_ca8_proc_info
> > > > >
> > > > > #endif /* CONFIG_ARM_LPAE */
> > > >
> > > > Thanks, merged into the original patch.
> > >
> > > Do you want to give me an ack for this, thanks?
> >
> > I tried to test this together with Tony's follow up patch, but I get
> > this after applying the patch to v4.0:
> >
> > sre@earth ~/src/linux [430973-fix] % make -j4
> > CHK include/config/kernel.release
> > CHK include/generated/uapi/linux/version.h
> > CHK include/generated/utsrelease.h
> > make[1]: 'include/generated/mach-types.h' is up to date.
> > CALL scripts/checksyscalls.sh
> > CHK include/generated/compile.h
> > AS arch/arm/mm/proc-v7.o
> > arch/arm/mm/proc-v7.S: Assembler messages:
> > arch/arm/mm/proc-v7.S:535: Error: invalid operands (*ABS* and .text sections) for `|'
> > arch/arm/mm/proc-v7.S:535: Error: invalid operands (*ABS* and .text sections) for `|'
> > scripts/Makefile.build:294: recipe for target 'arch/arm/mm/proc-v7.o' failed
> > make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
> > Makefile:947: recipe for target 'arch/arm/mm' failed
> > make: *** [arch/arm/mm] Error 2
> > make: *** Waiting for unfinished jobs....
>
> Maybe test the version in Linux next:
>
> a6d746789825 ("ARM: proc-v7: avoid errata 430973 workaround for non-Cortex A8 CPUs")
DONE with your your patch added on top:
Tested-By: Sebastian Reichel <sre@kernel.org>
(on N900)
I guess we should also drop the CONFIG_ARM_ERRATA_430973 check from
pdata-quirks' nokia_n900_legacy_init() and just enable it unconditionally.
-- Sebastian
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: sre@kernel.org (Sebastian Reichel)
To: linux-arm-kernel@lists.infradead.org
Subject: ARM errata 430973 on multi platform kernels
Date: Fri, 17 Apr 2015 20:41:56 +0200 [thread overview]
Message-ID: <20150417184156.GA3326@earth> (raw)
In-Reply-To: <20150416160858.GH18048@atomide.com>
On Thu, Apr 16, 2015 at 09:08:58AM -0700, Tony Lindgren wrote:
> * Sebastian Reichel <sre@kernel.org> [150415 09:32]:
> > Hi,
> >
> > On Thu, Apr 09, 2015 at 02:48:43PM +0100, Russell King - ARM Linux wrote:
> > > On Thu, Apr 09, 2015 at 12:06:58AM +0100, Russell King - ARM Linux wrote:
> > > > On Tue, Apr 07, 2015 at 08:22:08AM -0700, Tony Lindgren wrote:
> > > > > Works for me. The above needs the following fix folded in to build:
> > > > >
> > > > > --- a/arch/arm/mm/proc-v7.S
> > > > > +++ b/arch/arm/mm/proc-v7.S
> > > > > @@ -532,7 +532,7 @@ __v7_ca9mp_proc_info:
> > > > > __v7_ca8_proc_info:
> > > > > .long 0x410fc080
> > > > > .long 0xff0ffff0
> > > > > - __v7_proc __v7_ca8mp_proc_info, proc_fns = ca8_processor_functions
> > > > > + __v7_proc __v7_ca8_proc_info, __v7_setup, proc_fns = ca8_processor_functions
> > > > > .size __v7_ca8_proc_info, . - __v7_ca8_proc_info
> > > > >
> > > > > #endif /* CONFIG_ARM_LPAE */
> > > >
> > > > Thanks, merged into the original patch.
> > >
> > > Do you want to give me an ack for this, thanks?
> >
> > I tried to test this together with Tony's follow up patch, but I get
> > this after applying the patch to v4.0:
> >
> > sre at earth ~/src/linux [430973-fix] % make -j4
> > CHK include/config/kernel.release
> > CHK include/generated/uapi/linux/version.h
> > CHK include/generated/utsrelease.h
> > make[1]: 'include/generated/mach-types.h' is up to date.
> > CALL scripts/checksyscalls.sh
> > CHK include/generated/compile.h
> > AS arch/arm/mm/proc-v7.o
> > arch/arm/mm/proc-v7.S: Assembler messages:
> > arch/arm/mm/proc-v7.S:535: Error: invalid operands (*ABS* and .text sections) for `|'
> > arch/arm/mm/proc-v7.S:535: Error: invalid operands (*ABS* and .text sections) for `|'
> > scripts/Makefile.build:294: recipe for target 'arch/arm/mm/proc-v7.o' failed
> > make[1]: *** [arch/arm/mm/proc-v7.o] Error 1
> > Makefile:947: recipe for target 'arch/arm/mm' failed
> > make: *** [arch/arm/mm] Error 2
> > make: *** Waiting for unfinished jobs....
>
> Maybe test the version in Linux next:
>
> a6d746789825 ("ARM: proc-v7: avoid errata 430973 workaround for non-Cortex A8 CPUs")
DONE with your your patch added on top:
Tested-By: Sebastian Reichel <sre@kernel.org>
(on N900)
I guess we should also drop the CONFIG_ARM_ERRATA_430973 check from
pdata-quirks' nokia_n900_legacy_init() and just enable it unconditionally.
-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150417/809fb17c/attachment-0001.sig>
next prev parent reply other threads:[~2015-04-17 18:42 UTC|newest]
Thread overview: 123+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-01 19:07 [PATCH RESEND] ARM: dts: OMAP3-N900: Add microphone bias voltages Jarkko Nikula
2015-03-30 16:30 ` Jarkko Nikula
[not found] ` <55197A12.1050009-FVTvWyuFUl3QT0dZR+AlfA@public.gmane.org>
2015-03-30 16:42 ` Tony Lindgren
2015-03-30 17:45 ` Jarkko Nikula
[not found] ` <55198BA4.5010207-FVTvWyuFUl3QT0dZR+AlfA@public.gmane.org>
2015-03-30 17:50 ` Tony Lindgren
2015-03-31 12:32 ` Sebastian Reichel
2015-04-01 19:47 ` Tony Lindgren
2015-04-03 16:35 ` ARM errata 430973 on multi platform kernels (was: OMAP3-N900: Add microphone bias voltages) Sebastian Reichel
2015-04-03 18:39 ` Tony Lindgren
2015-04-03 18:39 ` Tony Lindgren
2015-04-03 19:21 ` Robert Nelson
2015-04-03 19:21 ` Robert Nelson
2015-04-05 13:00 ` Sebastian Reichel
2015-04-05 13:00 ` Sebastian Reichel
2015-04-05 13:26 ` Pali Rohár
2015-04-05 13:26 ` Pali Rohár
2015-04-05 13:45 ` Sebastian Reichel
2015-04-05 13:45 ` Sebastian Reichel
2015-04-05 13:52 ` Pali Rohár
2015-04-05 13:52 ` Pali Rohár
2015-04-06 17:38 ` Sebastian Reichel
2015-04-06 17:38 ` Sebastian Reichel
2015-04-03 20:42 ` Pavel Machek
2015-04-03 20:42 ` Pavel Machek
2015-04-03 22:08 ` ARM errata 430973 on multi platform kernels Ivaylo Dimitrov
2015-04-03 22:08 ` Ivaylo Dimitrov
2015-04-03 22:15 ` Tony Lindgren
2015-04-03 22:15 ` Tony Lindgren
2015-04-03 22:47 ` Ivaylo Dimitrov
2015-04-03 22:47 ` Ivaylo Dimitrov
2015-04-03 22:52 ` Tony Lindgren
2015-04-03 22:52 ` Tony Lindgren
2015-04-05 4:13 ` Matthijs van Duin
2015-04-05 4:13 ` Matthijs van Duin
2015-04-05 7:23 ` Ivaylo Dimitrov
2015-04-05 7:23 ` Ivaylo Dimitrov
2015-04-05 16:50 ` Matthijs van Duin
2015-04-05 16:50 ` Matthijs van Duin
2015-04-05 16:52 ` Matthijs van Duin
2015-04-05 16:52 ` Matthijs van Duin
2015-04-05 21:08 ` Ivaylo Dimitrov
2015-04-05 21:08 ` Ivaylo Dimitrov
2015-04-05 23:52 ` Matthijs van Duin
2015-04-05 23:52 ` Matthijs van Duin
2015-04-06 15:19 ` Tony Lindgren
2015-04-06 15:19 ` Tony Lindgren
2015-04-06 15:40 ` Tony Lindgren
2015-04-06 15:40 ` Tony Lindgren
2015-04-06 17:14 ` Ivaylo Dimitrov
2015-04-06 17:14 ` Ivaylo Dimitrov
2015-04-06 17:42 ` Tony Lindgren
2015-04-06 17:42 ` Tony Lindgren
2015-04-06 18:14 ` Matthijs van Duin
2015-04-06 18:14 ` Matthijs van Duin
2015-04-07 2:23 ` Tony Lindgren
2015-04-07 2:23 ` Tony Lindgren
2015-04-07 3:12 ` Sebastian Reichel
2015-04-07 3:12 ` Sebastian Reichel
2015-04-07 3:49 ` Matthijs van Duin
2015-04-07 3:49 ` Matthijs van Duin
2015-04-07 14:48 ` Tony Lindgren
2015-04-07 14:48 ` Tony Lindgren
2015-04-09 22:37 ` Grazvydas Ignotas
2015-04-09 22:37 ` Grazvydas Ignotas
2015-04-09 22:44 ` Tony Lindgren
2015-04-09 22:44 ` Tony Lindgren
2015-04-09 23:44 ` Nishanth Menon
2015-04-09 23:44 ` Nishanth Menon
2015-04-10 22:05 ` Grazvydas Ignotas
2015-04-10 22:05 ` Grazvydas Ignotas
2015-04-10 23:08 ` Tony Lindgren
2015-04-10 23:08 ` Tony Lindgren
2015-04-16 16:53 ` Matthijs van Duin
2015-04-16 16:53 ` Matthijs van Duin
2015-04-07 13:58 ` Russell King - ARM Linux
2015-04-07 13:58 ` Russell King - ARM Linux
2015-04-07 13:57 ` Russell King - ARM Linux
2015-04-07 13:57 ` Russell King - ARM Linux
2015-04-07 15:22 ` Tony Lindgren
2015-04-07 15:22 ` Tony Lindgren
2015-04-07 15:44 ` Tony Lindgren
2015-04-07 15:44 ` Tony Lindgren
2015-04-08 23:08 ` Russell King - ARM Linux
2015-04-08 23:08 ` Russell King - ARM Linux
2015-04-08 23:15 ` Tony Lindgren
2015-04-08 23:15 ` Tony Lindgren
2015-04-08 23:06 ` Russell King - ARM Linux
2015-04-08 23:06 ` Russell King - ARM Linux
2015-04-09 13:48 ` Russell King - ARM Linux
2015-04-09 13:48 ` Russell King - ARM Linux
2015-04-09 15:09 ` Tony Lindgren
2015-04-09 15:09 ` Tony Lindgren
2015-04-09 15:30 ` Russell King - ARM Linux
2015-04-09 15:30 ` Russell King - ARM Linux
2015-04-15 16:31 ` Sebastian Reichel
2015-04-15 16:31 ` Sebastian Reichel
2015-04-16 16:08 ` Tony Lindgren
2015-04-16 16:08 ` Tony Lindgren
2015-04-17 18:41 ` Sebastian Reichel [this message]
2015-04-17 18:41 ` Sebastian Reichel
2015-04-20 23:40 ` Tony Lindgren
2015-04-20 23:40 ` Tony Lindgren
2015-04-23 10:25 ` Russell King - ARM Linux
2015-04-23 10:25 ` Russell King - ARM Linux
2015-04-23 14:17 ` Tony Lindgren
2015-04-23 14:17 ` Tony Lindgren
2015-04-28 18:13 ` Russell King - ARM Linux
2015-04-28 18:13 ` Russell King - ARM Linux
2015-04-29 14:40 ` Tony Lindgren
2015-04-29 14:40 ` Tony Lindgren
2015-05-04 14:24 ` Tony Lindgren
2015-05-04 14:24 ` Tony Lindgren
2015-04-24 8:54 ` Matthijs van Duin
2015-04-24 8:54 ` Matthijs van Duin
2015-04-28 18:11 ` Russell King - ARM Linux
2015-04-28 18:11 ` Russell King - ARM Linux
2015-05-02 6:51 ` Matthijs van Duin
2015-05-02 6:51 ` Matthijs van Duin
2015-04-05 13:39 ` Sebastian Reichel
2015-04-05 13:39 ` Sebastian Reichel
2015-04-06 15:24 ` Tony Lindgren
2015-04-06 15:24 ` Tony Lindgren
2015-05-04 16:07 ` [PATCH RESEND] ARM: dts: OMAP3-N900: Add microphone bias voltages Tony Lindgren
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=20150417184156.GA3326@earth \
--to=sre@kernel.org \
--cc=ivo.g.dimitrov.75@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=matthijsvanduin@gmail.com \
--cc=pavel@ucw.cz \
--cc=tony@atomide.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 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.