* [PATCH] ARM: bcm283x: Fix SMP for old dtb files
@ 2017-08-23 19:08 Stefan Wahren
2017-08-23 19:44 ` Arnd Bergmann
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Wahren @ 2017-08-23 19:08 UTC (permalink / raw)
To: linux-arm-kernel
Since "irqchip: bcm2836: Move SMP startup code to arch/arm" the SMP
for BCM2836 and BCM2837 (32 bit) depends on commit "ARM: dts: bcm283x:
Add 32-bit enable method for SMP". This patch fixes SMP for older
dtb files.
Reported-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
arch/arm/mach-bcm/board_bcm2835.c | 6 +++++-
arch/arm/mach-bcm/platsmp.c | 2 +-
arch/arm/mach-bcm/platsmp.h | 10 ++++++++++
3 files changed, 16 insertions(+), 2 deletions(-)
create mode 100644 arch/arm/mach-bcm/platsmp.h
diff --git a/arch/arm/mach-bcm/board_bcm2835.c b/arch/arm/mach-bcm/board_bcm2835.c
index 24af33f..8cff865 100644
--- a/arch/arm/mach-bcm/board_bcm2835.c
+++ b/arch/arm/mach-bcm/board_bcm2835.c
@@ -19,16 +19,20 @@
#include <asm/mach/arch.h>
#include <asm/mach/map.h>
+#include "platsmp.h"
+
static const char * const bcm2835_compat[] = {
#ifdef CONFIG_ARCH_MULTI_V6
"brcm,bcm2835",
#endif
#ifdef CONFIG_ARCH_MULTI_V7
"brcm,bcm2836",
+ "brcm,bcm2837",
#endif
NULL
};
DT_MACHINE_START(BCM2835, "BCM2835")
- .dt_compat = bcm2835_compat
+ .dt_compat = bcm2835_compat,
+ .smp = smp_ops(bcm2836_smp_ops),
MACHINE_END
diff --git a/arch/arm/mach-bcm/platsmp.c b/arch/arm/mach-bcm/platsmp.c
index 7811160..7d95483 100644
--- a/arch/arm/mach-bcm/platsmp.c
+++ b/arch/arm/mach-bcm/platsmp.c
@@ -339,7 +339,7 @@ static const struct smp_operations nsp_smp_ops __initconst = {
};
CPU_METHOD_OF_DECLARE(bcm_smp_nsp, "brcm,bcm-nsp-smp", &nsp_smp_ops);
-static const struct smp_operations bcm2836_smp_ops __initconst = {
+const struct smp_operations bcm2836_smp_ops __initconst = {
.smp_boot_secondary = bcm2836_boot_secondary,
};
CPU_METHOD_OF_DECLARE(bcm_smp_bcm2836, "brcm,bcm2836-smp", &bcm2836_smp_ops);
diff --git a/arch/arm/mach-bcm/platsmp.h b/arch/arm/mach-bcm/platsmp.h
new file mode 100644
index 0000000..b8b8b3f
--- /dev/null
+++ b/arch/arm/mach-bcm/platsmp.h
@@ -0,0 +1,10 @@
+/*
+ * Copyright (C) 2017 Stefan Wahren <stefan.wahren@i2se.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ */
+
+extern const struct smp_operations bcm2836_smp_ops;
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] ARM: bcm283x: Fix SMP for old dtb files
2017-08-23 19:08 [PATCH] ARM: bcm283x: Fix SMP for old dtb files Stefan Wahren
@ 2017-08-23 19:44 ` Arnd Bergmann
2017-08-24 18:38 ` Eric Anholt
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2017-08-23 19:44 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 23, 2017 at 9:08 PM, Stefan Wahren <stefan.wahren@i2se.com> wrote:
> Since "irqchip: bcm2836: Move SMP startup code to arch/arm" the SMP
> for BCM2836 and BCM2837 (32 bit) depends on commit "ARM: dts: bcm283x:
> Add 32-bit enable method for SMP". This patch fixes SMP for older
> dtb files.
>
> Reported-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Looks good to me, thanks for the update.
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] ARM: bcm283x: Fix SMP for old dtb files
2017-08-23 19:44 ` Arnd Bergmann
@ 2017-08-24 18:38 ` Eric Anholt
0 siblings, 0 replies; 3+ messages in thread
From: Eric Anholt @ 2017-08-24 18:38 UTC (permalink / raw)
To: linux-arm-kernel
Arnd Bergmann <arnd@arndb.de> writes:
> On Wed, Aug 23, 2017 at 9:08 PM, Stefan Wahren <stefan.wahren@i2se.com> wrote:
>> Since "irqchip: bcm2836: Move SMP startup code to arch/arm" the SMP
>> for BCM2836 and BCM2837 (32 bit) depends on commit "ARM: dts: bcm283x:
>> Add 32-bit enable method for SMP". This patch fixes SMP for older
>> dtb files.
>>
>> Reported-by: Arnd Bergmann <arnd@arndb.de>
>> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
>
> Looks good to me, thanks for the update.
>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
Great! I've squashed this into the original commit, tested, and I'll
generate a new PR without the DT merge.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170824/b7c39bda/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-24 18:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-23 19:08 [PATCH] ARM: bcm283x: Fix SMP for old dtb files Stefan Wahren
2017-08-23 19:44 ` Arnd Bergmann
2017-08-24 18:38 ` Eric Anholt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).