[PATCH 0/3]

[PATCH 0/3]

[Fan Rong]

Fan Rong (3):
  Add smp support for Allwinner A20(sunxi 7i).
  Add cpuconfig nodes in dts for smp configure.
  Add arch count timer node in dts for Allwinner A20(sunxi 7i).

 arch/arm/boot/dts/sun7i-a20.dtsi |  19 ++-
 arch/arm/mach-sunxi/Makefile     |   2 +
 arch/arm/mach-sunxi/headsmp.S    |  12 ++
 arch/arm/mach-sunxi/platform.h   | 347 +++++++++++++++++++++++++++++++++++++++
 arch/arm/mach-sunxi/platsmp.c    | 100 +++++++++++
 arch/arm/mach-sunxi/sunxi.c      |  34 +++-
 6 files changed, 511 insertions(+), 3 deletions(-)
 create mode 100644 arch/arm/mach-sunxi/headsmp.S
 create mode 100644 arch/arm/mach-sunxi/platform.h
 create mode 100644 arch/arm/mach-sunxi/platsmp.c

-- 
1.8.1.2

[PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Fan Rong]

Signed-off-by: Fan Rong <cinifr@gmail.com>
---
 arch/arm/mach-sunxi/Makefile   |   2 +
 arch/arm/mach-sunxi/headsmp.S  |  12 ++
 arch/arm/mach-sunxi/platform.h | 347 +++++++++++++++++++++++++++++++++++++++++
 arch/arm/mach-sunxi/platsmp.c  | 100 ++++++++++++
 arch/arm/mach-sunxi/sunxi.c    |  34 +++-
 5 files changed, 494 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm/mach-sunxi/headsmp.S
 create mode 100644 arch/arm/mach-sunxi/platform.h
 create mode 100644 arch/arm/mach-sunxi/platsmp.c

diff --git a/arch/arm/mach-sunxi/Makefile b/arch/arm/mach-sunxi/Makefile
index 93bebfc..d7f1ef4 100644
--- a/arch/arm/mach-sunxi/Makefile
+++ b/arch/arm/mach-sunxi/Makefile
@@ -1 +1,3 @@
 obj-$(CONFIG_ARCH_SUNXI) += sunxi.o
+obj-$(CONFIG_ARCH_SUNXI) += platsmp.o
+obj-$(CONFIG_ARCH_SUNXI) += headsmp.o
diff --git a/arch/arm/mach-sunxi/headsmp.S b/arch/arm/mach-sunxi/headsmp.S
new file mode 100644
index 0000000..b400602
--- /dev/null
+++ b/arch/arm/mach-sunxi/headsmp.S
@@ -0,0 +1,12 @@
+#include <linux/linkage.h>
+#include <linux/init.h>
+
+        .section ".text.head", "ax"
+	__CPUINIT
+
+ENTRY(sun7i_secondary_startup)
+	msr	cpsr_fsxc, #0xd3
+	mov r0, #0
+	ldr r1, =0xffffffff
+	b       secondary_startup
+ENDPROC(sun7i_secondary_startup)
diff --git a/arch/arm/mach-sunxi/platform.h b/arch/arm/mach-sunxi/platform.h
new file mode 100644
index 0000000..7e127c9
--- /dev/null
+++ b/arch/arm/mach-sunxi/platform.h
@@ -0,0 +1,347 @@
+/*
+ * arch/arm/plat-sunxi/include/plat/platform.h
+ *
+ * (C) Copyright 2007-2012
+ * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
+ * Benn Huang <benn@allwinnertech.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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+extern struct smp_operations sunxi7i_smp_ops;
+extern void __iomem *sunxi7i_cc_base;
+void sun7i_secondary_startup(void);
+
+#ifndef __SW_PLATFORM_H
+#define __SW_PLATFORM_H
+
+
+
+/* Physical Address */
+#define SW_PA_BROM_START                  0xffff0000
+#define SW_PA_BROM_END                    0xffff7fff   /* 32KB */
+
+#define SW_PA_SRAM_BASE                   0x00000000
+
+/* sun7i sram addresses */
+#define SW_PA_SRAM_A1_BASE                0x00000000
+#define SW_PA_SRAM_A2_BASE                0x00004000
+#define SW_PA_SRAM_A3_BASE                0x00008000
+#define SW_PA_SRAM_A4_BASE                0x0000b400
+#define SW_PA_SRAM_D_BASE                 0x00010000
+#define SW_PA_SRAM_B_BASE                 0x00020000
+
+#define SW_PA_SDRAM_START                 0x40000000
+#define SW_PA_IO_BASE                     0x01c00000
+#define SW_PA_SRAM_IO_BASE                0x01c00000   /* 4KB */
+#define SW_PA_DRAM_IO_BASE                0x01c01000
+#define SW_PA_DMAC_IO_BASE                0x01c02000
+#define SW_PA_NANDFLASHC_IO_BASE          0x01c03000
+#define SW_PA_TSI_IO_BASE                 0x01c04000
+#define SW_PA_SPI0_IO_BASE                0x01c05000
+#define SW_PA_SPI1_IO_BASE                0x01c06000
+#define SW_PA_MSCC_IO_BASE                0x01c07000
+#define SW_PA_TVD_IO_BASE                 0x01c08000
+#define SW_PA_CSI0_IO_BASE                0x01c09000
+#define SW_PA_TVE_IO_BASE                 0x01c0a000
+#define SW_PA_EMAC_IO_BASE                0x01c0b000
+#define SW_PA_TCON0_IO_BASE               0x01c0c000
+#define SW_PA_TCON1_IO_BASE               0x01c0d000
+#define SW_PA_VE_IO_BASE                  0x01c0e000
+#define SW_PA_SDC0_IO_BASE                0x01c0f000
+#define SW_PA_SDC1_IO_BASE                0x01c10000
+#define SW_PA_SDC2_IO_BASE                0x01c11000
+#define SW_PA_SDC3_IO_BASE                0x01c12000
+#define SW_PA_USB0_IO_BASE                0x01c13000
+#define SW_PA_USB1_IO_BASE                0x01c14000
+#define SW_PA_SSE_IO_BASE                 0x01c15000
+#define SW_PA_HDMI_IO_BASE                0x01c16000
+#define SW_PA_SPI2_IO_BASE                0x01c17000
+#define SW_PA_SATA_IO_BASE                0x01c18000
+#define SW_PA_PATA_IO_BASE                0x01c19000
+#define SW_PA_ACE_IO_BASE                 0x01c1a000
+#define SW_PA_TVE1_IO_BASE                0x01c1b000
+#define SW_PA_USB2_IO_BASE                0x01c1c000
+#define SW_PA_CSI1_IO_BASE                0x01c1d000
+#define SW_PA_TZASC_IO_BASE               0x01c1e000
+#define SW_PA_SPI3_IO_BASE                0x01c1f000
+#define SW_PA_CCM_IO_BASE                 0x01c20000
+#define SW_PA_INT_IO_BASE                 0x01c20400
+#define SW_PA_PORTC_IO_BASE               0x01c20800
+#define SW_PA_TIMERC_IO_BASE              0x01c20c00
+#define SW_PA_SPDIF_IO_BASE               0x01c21000
+#define SW_PA_AC97_IO_BASE                0x01c21400
+#define SW_PA_IR0_IO_BASE                 0x01c21800
+#define SW_PA_IR1_IO_BASE                 0x01c21c00
+#define SW_PA_IIS1_IO_BASE                0x01c22000
+#define SW_PA_IIS_IO_BASE                 0x01c22400
+#define SW_PA_LRADC_IO_BASE               0x01c22800
+#define SW_PA_ADDA_IO_BASE                0x01c22c00
+#define SW_PA_KEYPAD_IO_BASE              0x01c23000
+#define SW_PA_TZPC_IO_BASE                0x01c23400
+#define SW_PA_SID_IO_BASE                 0x01c23800
+#define SW_PA_SJTAG_IO_BASE               0x01c23c00
+#define SW_PA_IIS2_IO_BASE                0x01c24400
+#define SW_PA_TP_IO_BASE                  0x01c25000
+#define SW_PA_PMU_IO_BASE                 0x01c25400
+#define SW_PA_CPUCFG_IO_BASE              0x01c25c00
+#define SW_PA_UART0_IO_BASE               0x01c28000
+#define SW_PA_UART1_IO_BASE               0x01c28400
+#define SW_PA_UART2_IO_BASE               0x01c28800
+#define SW_PA_UART3_IO_BASE               0x01c28c00
+#define SW_PA_UART4_IO_BASE               0x01c29000
+#define SW_PA_UART5_IO_BASE               0x01c29400
+#define SW_PA_UART6_IO_BASE               0x01c29800
+#define SW_PA_UART7_IO_BASE               0x01c29c00
+#define SW_PA_PS20_IO_BASE                0x01c2a000
+#define SW_PA_PS21_IO_BASE                0x01c2a400
+#define SW_PA_TWI0_IO_BASE                0x01c2ac00
+#define SW_PA_TWI1_IO_BASE                0x01c2b000
+#define SW_PA_TWI2_IO_BASE                0x01c2b400
+#define SW_PA_TWI3_IO_BASE                0x01c2b800
+#define SW_PA_CAN0_IO_BASE                0x01c2bc00
+#define SW_PA_CAN1_IO_BASE                0x01c2c000 /* sun4i */
+#define SW_PA_TWI4_IO_BASE                0x01c2c000 /* sun7i */
+#define SW_PA_SCR_IO_BASE                 0x01c2c400
+#define SW_PA_GPS_IO_BASE                 0x01c30000
+#define SW_PA_MALI_IO_BASE                0x01c40000
+#define SW_PA_GMAC_IO_BASE                0x01c50000
+#define SW_PA_HSTIMER_IO_BASE             0x01c60000
+#define SW_PA_GIC_IO_BASE                 0x01c80000
+#define SW_PA_GIC_DIST_IO_BASE            0x01c81000
+#define SW_PA_GIC_CPU_IO_BASE             0x01c82000
+#define SW_PA_SCU_IO_BASE                 0x01c80000
+
+#define SW_PA_TIMER_G_IO_BASE             0x01c80200
+/* CPU global timer, not used */
+
+#define SW_PA_TIMER_P_IO_BASE             0x01c80600
+/* CPU private timer, not used */
+
+#define SW_PA_HDMI1_IO_BASE               0x01ce0000
+#define SW_PA_SRAM_C_IO_BASE              0x01d00000
+#define SW_PA_DEFE0_IO_BASE               0x01e00000
+#define SW_PA_DEFE1_IO_BASE               0x01e20000
+#define SW_PA_DEBE0_IO_BASE               0x01e60000
+#define SW_PA_DEBE1_IO_BASE               0x01e40000
+#define SW_PA_MP_IO_BASE                  0x01e80000
+#define SW_PA_AVG_IO_BASE                 0x01ea0000
+#define SW_PA_CPUBIST_IO_BASE             0x3f501000
+#define SW_PA_BROM_BASE                   0xffff0000
+
+/* Virtual Address */
+#define SW_VA_SRAM_BASE                   0xf0000000	/*16KB*/
+
+/* sun7i sram addresses */
+#define SW_VA_SRAM_A1_BASE                0xf0000000
+#define SW_VA_SRAM_A2_BASE                0xf0004000
+#define SW_VA_SRAM_A3_BASE                0xf0008000
+#define SW_VA_SRAM_A4_BASE                0xf000b400
+#define SW_VA_SRAM_D_BASE                 0xf0010000
+#define SW_VA_SRAM_B_BASE                 0xf0020000
+
+#define SW_VA_BROM_BASE                   0xf0100000	/*64KB*/
+#define SW_VA_BROM_START                  0xf0100000
+#define SW_VA_BROM_END                    0xf0107fff
+
+#define SW_VA_IO_BASE                     0xf1c00000
+#define SW_VA_SRAM_IO_BASE                0xf1c00000   /* 4KB */
+#define SW_VA_DRAM_IO_BASE                0xf1c01000
+#define SW_VA_DMAC_IO_BASE                0xf1c02000
+#define SW_VA_NANDFLASHC_IO_BASE          0xf1c03000
+#define SW_VA_TSI_IO_BASE                 0xf1c04000
+#define SW_VA_SPI0_IO_BASE                0xf1c05000
+#define SW_VA_SPI1_IO_BASE                0xf1c06000
+#define SW_VA_MSCC_IO_BASE                0xf1c07000
+#define SW_VA_TVD_IO_BASE                 0xf1c08000
+#define SW_VA_CSI0_IO_BASE                0xf1c09000
+#define SW_VA_TVE_IO_BASE                 0xf1c0a000
+#define SW_VA_EMAC_IO_BASE                0xf1c0b000
+#define SW_VA_TCON0_IO_BASE               0xf1c0c000
+#define SW_VA_TCON1_IO_BASE               0xf1c0d000
+#define SW_VA_VE_IO_BASE                  0xf1c0e000
+#define SW_VA_SDC0_IO_BASE                0xf1c0f000
+#define SW_VA_SDC1_IO_BASE                0xf1c10000
+#define SW_VA_SDC2_IO_BASE                0xf1c11000
+#define SW_VA_SDC3_IO_BASE                0xf1c12000
+#define SW_VA_USB0_IO_BASE                0xf1c13000
+#define SW_VA_USB1_IO_BASE                0xf1c14000
+#define SW_VA_SSE_IO_BASE                 0xf1c15000
+#define SW_VA_HDMI_IO_BASE                0xf1c16000
+#define SW_VA_SPI2_IO_BASE                0xf1c17000
+#define SW_VA_SATA_IO_BASE                0xf1c18000
+#define SW_VA_PATA_IO_BASE                0xf1c19000
+#define SW_VA_ACE_IO_BASE                 0xf1c1a000
+#define SW_VA_TVE1_IO_BASE                0xf1c1b000
+#define SW_VA_USB2_IO_BASE                0xf1c1c000
+#define SW_VA_CSI1_IO_BASE                0xf1c1d000
+#define SW_VA_TZASC_IO_BASE               0xf1c1e000
+#define SW_VA_SPI3_IO_BASE                0xf1c1f000
+#define SW_VA_CCM_IO_BASE                 0xf1c20000
+#define SW_VA_INT_IO_BASE                 0xf1c20400
+#define SW_VA_PORTC_IO_BASE               0xf1c20800
+#define SW_VA_TIMERC_IO_BASE              0xf1c20c00
+#define SW_VA_SPDIF_IO_BASE               0xf1c21000
+#define SW_VA_AC97_IO_BASE                0xf1c21400
+#define SW_VA_IR0_IO_BASE                 0xf1c21800
+#define SW_VA_IR1_IO_BASE                 0xf1c21c00
+#define SW_VA_IIS1_IO_BASE                0xf1c22000
+#define SW_VA_IIS_IO_BASE                 0xf1c22400
+#define SW_VA_LRADC_IO_BASE               0xf1c22800
+#define SW_VA_ADDA_IO_BASE                0xf1c22c00
+#define SW_VA_KEYPAD_IO_BASE              0xf1c23000
+#define SW_VA_TZPC_IO_BASE                0xf1c23400
+#define SW_VA_SID_IO_BASE                 0xf1c23800
+#define SW_VA_SJTAG_IO_BASE               0xf1c23c00
+#define SW_VA_IIS2_IO_BASE                0xf1c24400
+#define SW_VA_TP_IO_BASE                  0xf1c25000
+#define SW_VA_PMU_IO_BASE                 0xf1c25400
+#define SW_VA_CPUCFG_IO_BASE              0xf1c25c00
+#define SW_VA_UART0_IO_BASE               0xf1c28000
+#define SW_VA_UART1_IO_BASE               0xf1c28400
+#define SW_VA_UART2_IO_BASE               0xf1c28800
+#define SW_VA_UART3_IO_BASE               0xf1c28c00
+#define SW_VA_UART4_IO_BASE               0xf1c29000
+#define SW_VA_UART5_IO_BASE               0xf1c29400
+#define SW_VA_UART6_IO_BASE               0xf1c29800
+#define SW_VA_UART7_IO_BASE               0xf1c29c00
+#define SW_VA_PS20_IO_BASE                0xf1c2a000
+#define SW_VA_PS21_IO_BASE                0xf1c2a400
+#define SW_VA_TWI0_IO_BASE                0xf1c2ac00
+#define SW_VA_TWI1_IO_BASE                0xf1c2b000
+#define SW_VA_TWI2_IO_BASE                0xf1c2b400
+#define SW_VA_TWI3_IO_BASE                0xf1c2b800
+#define SW_VA_CAN0_IO_BASE                0xf1c2bc00
+#define SW_VA_CAN1_IO_BASE                0xf1c2c000 /* sun4i */
+#define SW_VA_TWI4_IO_BASE                0xf1c2c000 /* sun7i */
+#define SW_VA_SCR_IO_BASE                 0xf1c2c400
+#define SW_VA_GPS_IO_BASE                 0xf1c30000
+#define SW_VA_MALI_IO_BASE                0xf1c40000
+#define SW_VA_GMAC_IO_BASE                0xf1c50000
+#define SW_VA_HSTIMER_IO_BASE             0xf1c60000
+#define SW_VA_GIC_IO_BASE                 0xf1c80000
+#define SW_VA_GIC_DIST_IO_BASE            0xf1c81000
+#define SW_VA_GIC_CPU_IO_BASE             0xf1c82000
+#define SW_VA_SCU_IO_BASE                 0xf1c80000
+
+#define SW_VA_TIMER_G_IO_BASE             0xf1c80200
+/* CPU global timer, not used */
+
+#define SW_VA_TIMER_P_IO_BASE             0xf1c80600
+/* CPU private timer, not used */
+
+#define SW_VA_HDMI1_IO_BASE               0xf1ce0000
+#define SW_VA_SRAM_C_IO_BASE              0xf1d00000
+#define SW_VA_DEFE0_IO_BASE               0xf1e00000
+#define SW_VA_DEFE1_IO_BASE               0xf1e20000
+#define SW_VA_DEBE0_IO_BASE               0xf1e60000
+#define SW_VA_DEBE1_IO_BASE               0xf1e40000
+#define SW_VA_MP_IO_BASE                  0xf1e80000
+#define SW_VA_AVG_IO_BASE                 0xf1ea0000
+
+/* memory size */
+#define SW_IO_SIZE                        0x00400000 /* 4MB(Max) */
+#define SW_SRAM_A1_SIZE                   0x00004000 /* 16k */
+#define SW_SRAM_A2_SIZE                   0x00004000 /* 16k */
+#define SW_SRAM_A3_SIZE                   0x00003400 /* 13k */
+#define SW_SRAM_A4_SIZE                   0x00000c00 /* 3k */
+#define SW_SRAM_D_SIZE                    0x00001000 /* 4k */
+#define SW_SRAM_B_SIZE                    0x00010000 /* 64k */
+#define SW_BROM_SIZE                      0x00008000 /* 32k */
+
+/**
+ * Interrupt controller registers
+ *
+ */
+#define SW_INT_VECTOR_REG                 (SW_VA_INT_IO_BASE + 0x00)
+#define SW_INT_BASE_ADR_REG               (SW_VA_INT_IO_BASE + 0x04)
+#define SW_INT_PROTECTION_REG             (SW_VA_INT_IO_BASE + 0x08)
+#define SW_INT_NMI_CTRL_REG               (SW_VA_INT_IO_BASE + 0x0c)
+#define SW_INT_IRQ_PENDING_REG0           (SW_VA_INT_IO_BASE + 0x10)
+#define SW_INT_IRQ_PENDING_REG1           (SW_VA_INT_IO_BASE + 0x14)
+#define SW_INT_IRQ_PENDING_REG2           (SW_VA_INT_IO_BASE + 0x18)
+
+#define SW_INT_FIQ_PENDING_REG0           (SW_VA_INT_IO_BASE + 0x20)
+#define SW_INT_FIQ_PENDING_REG1           (SW_VA_INT_IO_BASE + 0x24)
+#define SW_INT_FIQ_PENDING_REG2           (SW_VA_INT_IO_BASE + 0x28)
+
+#define SW_INT_SELECT_REG0                (SW_VA_INT_IO_BASE + 0x30)
+#define SW_INT_SELECT_REG1                (SW_VA_INT_IO_BASE + 0x34)
+#define SW_INT_SELECT_REG2                (SW_VA_INT_IO_BASE + 0x38)
+
+#define SW_INT_ENABLE_REG0                (SW_VA_INT_IO_BASE + 0x40)
+#define SW_INT_ENABLE_REG1                (SW_VA_INT_IO_BASE + 0x44)
+#define SW_INT_ENABLE_REG2                (SW_VA_INT_IO_BASE + 0x48)
+
+#define SW_INT_MASK_REG0                  (SW_VA_INT_IO_BASE + 0x50)
+#define SW_INT_MASK_REG1                  (SW_VA_INT_IO_BASE + 0x54)
+#define SW_INT_MASK_REG2                  (SW_VA_INT_IO_BASE + 0x58)
+
+#define SW_INT_RESP_REG0                  (SW_VA_INT_IO_BASE + 0x60)
+#define SW_INT_RESP_REG1                  (SW_VA_INT_IO_BASE + 0x64)
+#define SW_INT_RESP_REG2                  (SW_VA_INT_IO_BASE + 0x68)
+
+#define SW_INT_FASTFORCE_REG0             (SW_VA_INT_IO_BASE + 0x70)
+#define SW_INT_FASTFORCE_REG1             (SW_VA_INT_IO_BASE + 0x74)
+#define SW_INT_FASTFORCE_REG2             (SW_VA_INT_IO_BASE + 0x78)
+
+#define SW_INT_SRCPRIO_REG0               (SW_VA_INT_IO_BASE + 0x80)
+#define SW_INT_SRCPRIO_REG1               (SW_VA_INT_IO_BASE + 0x84)
+#define SW_INT_SRCPRIO_REG2               (SW_VA_INT_IO_BASE + 0x88)
+#define SW_INT_SRCPRIO_REG3               (SW_VA_INT_IO_BASE + 0x8c)
+#define SW_INT_SRCPRIO_REG4               (SW_VA_INT_IO_BASE + 0x90)
+#ifdef CONFIG_ARCH_SUN5I
+#define SW_INT_SRCPRIO_REG5               (SW_VA_INT_IO_BASE + 0x94)
+#endif
+
+
+#define PA_VIC_BASE                       0x01c20400
+#define VA_VIC_BASE                       IO_ADDRESS(PA_VIC_BASE)
+#define PIO_BASE                          SW_PA_PORTC_IO_BASE
+
+/**
+*@name DRAM controller register address
+*@{
+*/
+#define SW_DRAM_SDR_CTL_REG               (SW_VA_DRAM_IO_BASE + 0x0C)
+#define SW_DRAM_SDR_DCR                   (SW_VA_DRAM_IO_BASE + 0x04)
+
+/*
+ * other reg defination, not found in spec
+ */
+#define AW_GIC_DIST_BASE                  0x01c81000
+#define AW_GIC_CPU_BASE                   0x01c82000
+
+#define AW_TIMER_G_BASE                   0x01c80200
+/* CPU global timer, not used */
+
+#define AW_TIMER_P_BASE                   0x01c80600
+/* CPU private timer, not used */
+
+/*
+ * CPUCFG
+ */
+#define AW_CPUCFG_P_REG0            0x01a4
+#define CPUX_RESET_CTL(x) (0x40 + (x)*0x40)
+#define CPUX_CONTROL(x)   (0x44 + (x)*0x40)
+#define CPUX_STATUS(x)    (0x48 + (x)*0x40)
+#define AW_CPUCFG_GENCTL            0x0184
+#define AW_CPUCFG_DBGCTL0           0x01e0
+#define AW_CPUCFG_DBGCTL1           0x01e4
+
+#define AW_CPU1_PWR_CLAMP         0x01b0
+#define AW_CPU1_PWROFF_REG        0x01b4
+
+#endif
diff --git a/arch/arm/mach-sunxi/platsmp.c b/arch/arm/mach-sunxi/platsmp.c
new file mode 100644
index 0000000..c76652d
--- /dev/null
+++ b/arch/arm/mach-sunxi/platsmp.c
@@ -0,0 +1,100 @@
+/*
+ *  linux/arch/arm/mach-sun7i/platsmp.c
+ *
+ *  Copyright (C) 2012-2016 Allwinner Ltd.
+ *  All Rights Reserved
+ *
+ * merge it to 3.11 by cini <cinifr@gmail.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+#include <linux/init.h>
+#include <linux/errno.h>
+#include <linux/smp.h>
+#include <linux/io.h>
+#include <linux/delay.h>
+#include <linux/device.h>
+#include <linux/jiffies.h>
+#include <linux/smp.h>
+
+#include <linux/of_address.h>
+#include <linux/of_irq.h>
+#include <linux/of_platform.h>
+
+
+#include <linux/irqchip/arm-gic.h>
+#include <asm/mach-types.h>
+#include <asm/smp_scu.h>
+#include <asm/cacheflush.h>
+#include <asm/smp_plat.h>
+
+#include "platform.h"
+#include <asm/arch_timer.h>
+
+
+/*
+ * for linux/arch/arm/kernel/smp.c:__cpu_up(..)
+ */
+static int sunxi7i_boot_secondary(unsigned int cpu, struct task_struct *idle)
+{
+	long paddr;
+	u32 pwr_reg;
+	if (!sunxi7i_cc_base) {
+		pr_debug("error map cpu configure\n");
+		return -ENOSYS;
+	}
+
+	paddr = virt_to_phys(sun7i_secondary_startup);
+	writel(paddr, sunxi7i_cc_base + AW_CPUCFG_P_REG0);
+	/* step1: Assert nCOREPORESET LOW and hold L1RSTDISABLE LOW.
+		Ensure DBGPWRDUP is held LOW to prevent any external
+		debug access to the processor.
+	*/
+	/* assert cpu core reset */
+	writel(0, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
+	/* L1RSTDISABLE hold low */
+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_GENCTL);
+	pwr_reg &= ~(1<<cpu);
+	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_GENCTL);
+	/* DBGPWRDUP hold low */
+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
+	pwr_reg &= ~(1<<cpu);
+	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
+
+	/* step2: release power clamp */
+	writel(0xff, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x7f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x3f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x1f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x0f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x07, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x03, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x01, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	writel(0x00, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
+	mdelay(10);
+
+	/* step3: clear power-off gating */
+	pwr_reg = readl(sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
+	pwr_reg &= ~(1);
+	writel(pwr_reg, sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
+	mdelay(1);
+
+	/* step4: de-assert core reset */
+	writel(3, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
+
+	/* step5: assert DBGPWRDUP signal */
+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
+	pwr_reg |= (1<<cpu);
+	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
+	return 0;
+}
+
+
+
+struct smp_operations sunxi7i_smp_ops __initdata = {
+	.smp_boot_secondary	= sunxi7i_boot_secondary,
+};
+
+
diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
index e79fb34..88e7ecb 100644
--- a/arch/arm/mach-sunxi/sunxi.c
+++ b/arch/arm/mach-sunxi/sunxi.c
@@ -26,6 +26,9 @@
 #include <asm/mach/map.h>
 #include <asm/system_misc.h>
 
+#include "platform.h"
+
+
 #define SUN4I_WATCHDOG_CTRL_REG		0x00
 #define SUN4I_WATCHDOG_CTRL_RESTART		BIT(0)
 #define SUN4I_WATCHDOG_MODE_REG		0x04
@@ -42,6 +45,14 @@
 #define SUN6I_WATCHDOG1_MODE_ENABLE		BIT(0)
 
 static void __iomem *wdt_base;
+/*
+ * CPU Configure module support
+ * 1: Software reset for smp cpus
+ * 2: Configure for smp cpus including boot.
+ * 3: Three 64-bit idle counters and two 64-bit common counters
+ * it is needed for smp cpus
+ */
+void __iomem *sunxi7i_cc_base; /*CPU Configure Base*/
 
 static void sun4i_restart(enum reboot_mode mode, const char *cmd)
 {
@@ -98,6 +109,11 @@ static struct of_device_id sunxi_restart_ids[] = {
 	{ /*sentinel*/ }
 };
 
+static struct of_device_id sunxi_cc_ids[] = {
+	{ .compatible = "allwinner,sun7i-a20-cpuconfig"},
+	{ /*sentinel*/ }
+};
+
 static void sunxi_setup_restart(void)
 {
 	const struct of_device_id *of_id;
@@ -125,7 +141,6 @@ static void __init sunxi_timer_init(void)
 static void __init sunxi_dt_init(void)
 {
 	sunxi_setup_restart();
-
 	of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
 }
 
@@ -138,7 +153,24 @@ static const char * const sunxi_board_dt_compat[] = {
 	NULL,
 };
 
+
+static void __init sunxi_init_cpuconfig_map(void)
+{
+	struct device_node *np;
+
+	np = of_find_matching_node(NULL, sunxi_cc_ids);
+	if (WARN(!np, "unable to setup cup configure"))
+		return;
+	sunxi7i_cc_base = of_iomap(np, 0);
+	WARN(!sunxi7i_cc_base, "failed to map cup configure base address");
+}
+
+
+early_initcall(sunxi_init_cpuconfig_map);
+
+
 DT_MACHINE_START(SUNXI_DT, "Allwinner A1X (Device Tree)")
+	.smp	= smp_ops(sunxi7i_smp_ops),
 	.init_machine	= sunxi_dt_init,
 	.init_time	= sunxi_timer_init,
 	.dt_compat	= sunxi_board_dt_compat,
-- 
1.8.1.2

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Russell King - ARM Linux]

On Sat, Sep 21, 2013 at 09:05:14PM +0800, Fan Rong wrote:
> @@ -0,0 +1,12 @@
> +#include <linux/linkage.h>
> +#include <linux/init.h>
> +
> +        .section ".text.head", "ax"

Please use a tab instead of 8 spaces.

> +	__CPUINIT

__CPUINIT has been removed.

> +
> +ENTRY(sun7i_secondary_startup)
> +	msr	cpsr_fsxc, #0xd3
> +	mov r0, #0
> +	ldr r1, =0xffffffff

You don't  need to set r0 nor r1 here.

> +	b       secondary_startup
> +ENDPROC(sun7i_secondary_startup)
> diff --git a/arch/arm/mach-sunxi/platform.h b/arch/arm/mach-sunxi/platform.h
> new file mode 100644
> index 0000000..7e127c9
> --- /dev/null
> +++ b/arch/arm/mach-sunxi/platform.h
> @@ -0,0 +1,347 @@
> +/*
> + * arch/arm/plat-sunxi/include/plat/platform.h
> + *
> + * (C) Copyright 2007-2012
> + * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
> + * Benn Huang <benn@allwinnertech.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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA

Just the first paragraph is sufficient.  Also are you sure you want the
"or any later version"?  The kernel is predominantly a GPLv2 only project.

What are the origins of this file - do you have the ability to change this?

> +	/* step3: clear power-off gating */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
> +	pwr_reg &= ~(1);

I think I've said this before.  The parens around '1' are not required.
What do you think you gain by having the additional parens?

> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
> +	mdelay(1);
> +
> +	/* step4: de-assert core reset */
> +	writel(3, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
> +
> +	/* step5: assert DBGPWRDUP signal */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> +	pwr_reg |= (1<<cpu);

The parens around 1<<cpu are not required.  What do you think you gain
by having them?  Coding style also suggests placing a space before and
after the "<<" operation as well.  Again, I've said this in my previous
review.

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 26426 bytes --]

Hi Fan,

On Sat, Sep 21, 2013 at 09:05:14PM +0800, Fan Rong wrote:
> Signed-off-by: Fan Rong <cinifr@gmail.com>

I'd prefer a more verbose commit log here, and for the following patches
as well.

Usually, you detail what you're doing in the commit "title", and the
rest of the commit log would be why you're doing so, the possible
consequences it might have (for example, in your architected timer
related patch, it would be great to have a mention that in order to use
it, you need to upgrade your bootloader, or it will fail to boot).

This will ensure to have a decent history, and we can easily get the
context of a given change.

> ---
>  arch/arm/mach-sunxi/Makefile   |   2 +
>  arch/arm/mach-sunxi/headsmp.S  |  12 ++
>  arch/arm/mach-sunxi/platform.h | 347 +++++++++++++++++++++++++++++++++++++++++
>  arch/arm/mach-sunxi/platsmp.c  | 100 ++++++++++++
>  arch/arm/mach-sunxi/sunxi.c    |  34 +++-
>  5 files changed, 494 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/mach-sunxi/headsmp.S
>  create mode 100644 arch/arm/mach-sunxi/platform.h
>  create mode 100644 arch/arm/mach-sunxi/platsmp.c
> 
> diff --git a/arch/arm/mach-sunxi/Makefile b/arch/arm/mach-sunxi/Makefile
> index 93bebfc..d7f1ef4 100644
> --- a/arch/arm/mach-sunxi/Makefile
> +++ b/arch/arm/mach-sunxi/Makefile
> @@ -1 +1,3 @@
>  obj-$(CONFIG_ARCH_SUNXI) += sunxi.o
> +obj-$(CONFIG_ARCH_SUNXI) += platsmp.o
> +obj-$(CONFIG_ARCH_SUNXI) += headsmp.o
> diff --git a/arch/arm/mach-sunxi/headsmp.S b/arch/arm/mach-sunxi/headsmp.S
> new file mode 100644
> index 0000000..b400602
> --- /dev/null
> +++ b/arch/arm/mach-sunxi/headsmp.S
> @@ -0,0 +1,12 @@
> +#include <linux/linkage.h>
> +#include <linux/init.h>
> +
> +        .section ".text.head", "ax"
> +	__CPUINIT
> +
> +ENTRY(sun7i_secondary_startup)
> +	msr	cpsr_fsxc, #0xd3
> +	mov r0, #0
> +	ldr r1, =0xffffffff

I don't think setting r0 and r1 is needed here.

> +	b       secondary_startup
> +ENDPROC(sun7i_secondary_startup)
> diff --git a/arch/arm/mach-sunxi/platform.h b/arch/arm/mach-sunxi/platform.h
> new file mode 100644
> index 0000000..7e127c9
> --- /dev/null
> +++ b/arch/arm/mach-sunxi/platform.h
> @@ -0,0 +1,347 @@
> +/*
> + * arch/arm/plat-sunxi/include/plat/platform.h
> + *

Please remove this file. We don't need it at all.

> + * (C) Copyright 2007-2012
> + * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
> + * Benn Huang <benn@allwinnertech.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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +extern struct smp_operations sunxi7i_smp_ops;
> +extern void __iomem *sunxi7i_cc_base;
> +void sun7i_secondary_startup(void);
> +
> +#ifndef __SW_PLATFORM_H
> +#define __SW_PLATFORM_H
> +
> +
> +
> +/* Physical Address */
> +#define SW_PA_BROM_START                  0xffff0000
> +#define SW_PA_BROM_END                    0xffff7fff   /* 32KB */
> +
> +#define SW_PA_SRAM_BASE                   0x00000000
> +
> +/* sun7i sram addresses */
> +#define SW_PA_SRAM_A1_BASE                0x00000000
> +#define SW_PA_SRAM_A2_BASE                0x00004000
> +#define SW_PA_SRAM_A3_BASE                0x00008000
> +#define SW_PA_SRAM_A4_BASE                0x0000b400
> +#define SW_PA_SRAM_D_BASE                 0x00010000
> +#define SW_PA_SRAM_B_BASE                 0x00020000
> +
> +#define SW_PA_SDRAM_START                 0x40000000
> +#define SW_PA_IO_BASE                     0x01c00000
> +#define SW_PA_SRAM_IO_BASE                0x01c00000   /* 4KB */
> +#define SW_PA_DRAM_IO_BASE                0x01c01000
> +#define SW_PA_DMAC_IO_BASE                0x01c02000
> +#define SW_PA_NANDFLASHC_IO_BASE          0x01c03000
> +#define SW_PA_TSI_IO_BASE                 0x01c04000
> +#define SW_PA_SPI0_IO_BASE                0x01c05000
> +#define SW_PA_SPI1_IO_BASE                0x01c06000
> +#define SW_PA_MSCC_IO_BASE                0x01c07000
> +#define SW_PA_TVD_IO_BASE                 0x01c08000
> +#define SW_PA_CSI0_IO_BASE                0x01c09000
> +#define SW_PA_TVE_IO_BASE                 0x01c0a000
> +#define SW_PA_EMAC_IO_BASE                0x01c0b000
> +#define SW_PA_TCON0_IO_BASE               0x01c0c000
> +#define SW_PA_TCON1_IO_BASE               0x01c0d000
> +#define SW_PA_VE_IO_BASE                  0x01c0e000
> +#define SW_PA_SDC0_IO_BASE                0x01c0f000
> +#define SW_PA_SDC1_IO_BASE                0x01c10000
> +#define SW_PA_SDC2_IO_BASE                0x01c11000
> +#define SW_PA_SDC3_IO_BASE                0x01c12000
> +#define SW_PA_USB0_IO_BASE                0x01c13000
> +#define SW_PA_USB1_IO_BASE                0x01c14000
> +#define SW_PA_SSE_IO_BASE                 0x01c15000
> +#define SW_PA_HDMI_IO_BASE                0x01c16000
> +#define SW_PA_SPI2_IO_BASE                0x01c17000
> +#define SW_PA_SATA_IO_BASE                0x01c18000
> +#define SW_PA_PATA_IO_BASE                0x01c19000
> +#define SW_PA_ACE_IO_BASE                 0x01c1a000
> +#define SW_PA_TVE1_IO_BASE                0x01c1b000
> +#define SW_PA_USB2_IO_BASE                0x01c1c000
> +#define SW_PA_CSI1_IO_BASE                0x01c1d000
> +#define SW_PA_TZASC_IO_BASE               0x01c1e000
> +#define SW_PA_SPI3_IO_BASE                0x01c1f000
> +#define SW_PA_CCM_IO_BASE                 0x01c20000
> +#define SW_PA_INT_IO_BASE                 0x01c20400
> +#define SW_PA_PORTC_IO_BASE               0x01c20800
> +#define SW_PA_TIMERC_IO_BASE              0x01c20c00
> +#define SW_PA_SPDIF_IO_BASE               0x01c21000
> +#define SW_PA_AC97_IO_BASE                0x01c21400
> +#define SW_PA_IR0_IO_BASE                 0x01c21800
> +#define SW_PA_IR1_IO_BASE                 0x01c21c00
> +#define SW_PA_IIS1_IO_BASE                0x01c22000
> +#define SW_PA_IIS_IO_BASE                 0x01c22400
> +#define SW_PA_LRADC_IO_BASE               0x01c22800
> +#define SW_PA_ADDA_IO_BASE                0x01c22c00
> +#define SW_PA_KEYPAD_IO_BASE              0x01c23000
> +#define SW_PA_TZPC_IO_BASE                0x01c23400
> +#define SW_PA_SID_IO_BASE                 0x01c23800
> +#define SW_PA_SJTAG_IO_BASE               0x01c23c00
> +#define SW_PA_IIS2_IO_BASE                0x01c24400
> +#define SW_PA_TP_IO_BASE                  0x01c25000
> +#define SW_PA_PMU_IO_BASE                 0x01c25400
> +#define SW_PA_CPUCFG_IO_BASE              0x01c25c00
> +#define SW_PA_UART0_IO_BASE               0x01c28000
> +#define SW_PA_UART1_IO_BASE               0x01c28400
> +#define SW_PA_UART2_IO_BASE               0x01c28800
> +#define SW_PA_UART3_IO_BASE               0x01c28c00
> +#define SW_PA_UART4_IO_BASE               0x01c29000
> +#define SW_PA_UART5_IO_BASE               0x01c29400
> +#define SW_PA_UART6_IO_BASE               0x01c29800
> +#define SW_PA_UART7_IO_BASE               0x01c29c00
> +#define SW_PA_PS20_IO_BASE                0x01c2a000
> +#define SW_PA_PS21_IO_BASE                0x01c2a400
> +#define SW_PA_TWI0_IO_BASE                0x01c2ac00
> +#define SW_PA_TWI1_IO_BASE                0x01c2b000
> +#define SW_PA_TWI2_IO_BASE                0x01c2b400
> +#define SW_PA_TWI3_IO_BASE                0x01c2b800
> +#define SW_PA_CAN0_IO_BASE                0x01c2bc00
> +#define SW_PA_CAN1_IO_BASE                0x01c2c000 /* sun4i */
> +#define SW_PA_TWI4_IO_BASE                0x01c2c000 /* sun7i */
> +#define SW_PA_SCR_IO_BASE                 0x01c2c400
> +#define SW_PA_GPS_IO_BASE                 0x01c30000
> +#define SW_PA_MALI_IO_BASE                0x01c40000
> +#define SW_PA_GMAC_IO_BASE                0x01c50000
> +#define SW_PA_HSTIMER_IO_BASE             0x01c60000
> +#define SW_PA_GIC_IO_BASE                 0x01c80000
> +#define SW_PA_GIC_DIST_IO_BASE            0x01c81000
> +#define SW_PA_GIC_CPU_IO_BASE             0x01c82000
> +#define SW_PA_SCU_IO_BASE                 0x01c80000
> +
> +#define SW_PA_TIMER_G_IO_BASE             0x01c80200
> +/* CPU global timer, not used */
> +
> +#define SW_PA_TIMER_P_IO_BASE             0x01c80600
> +/* CPU private timer, not used */
> +
> +#define SW_PA_HDMI1_IO_BASE               0x01ce0000
> +#define SW_PA_SRAM_C_IO_BASE              0x01d00000
> +#define SW_PA_DEFE0_IO_BASE               0x01e00000
> +#define SW_PA_DEFE1_IO_BASE               0x01e20000
> +#define SW_PA_DEBE0_IO_BASE               0x01e60000
> +#define SW_PA_DEBE1_IO_BASE               0x01e40000
> +#define SW_PA_MP_IO_BASE                  0x01e80000
> +#define SW_PA_AVG_IO_BASE                 0x01ea0000
> +#define SW_PA_CPUBIST_IO_BASE             0x3f501000
> +#define SW_PA_BROM_BASE                   0xffff0000
> +
> +/* Virtual Address */
> +#define SW_VA_SRAM_BASE                   0xf0000000	/*16KB*/
> +
> +/* sun7i sram addresses */
> +#define SW_VA_SRAM_A1_BASE                0xf0000000
> +#define SW_VA_SRAM_A2_BASE                0xf0004000
> +#define SW_VA_SRAM_A3_BASE                0xf0008000
> +#define SW_VA_SRAM_A4_BASE                0xf000b400
> +#define SW_VA_SRAM_D_BASE                 0xf0010000
> +#define SW_VA_SRAM_B_BASE                 0xf0020000
> +
> +#define SW_VA_BROM_BASE                   0xf0100000	/*64KB*/
> +#define SW_VA_BROM_START                  0xf0100000
> +#define SW_VA_BROM_END                    0xf0107fff
> +
> +#define SW_VA_IO_BASE                     0xf1c00000
> +#define SW_VA_SRAM_IO_BASE                0xf1c00000   /* 4KB */
> +#define SW_VA_DRAM_IO_BASE                0xf1c01000
> +#define SW_VA_DMAC_IO_BASE                0xf1c02000
> +#define SW_VA_NANDFLASHC_IO_BASE          0xf1c03000
> +#define SW_VA_TSI_IO_BASE                 0xf1c04000
> +#define SW_VA_SPI0_IO_BASE                0xf1c05000
> +#define SW_VA_SPI1_IO_BASE                0xf1c06000
> +#define SW_VA_MSCC_IO_BASE                0xf1c07000
> +#define SW_VA_TVD_IO_BASE                 0xf1c08000
> +#define SW_VA_CSI0_IO_BASE                0xf1c09000
> +#define SW_VA_TVE_IO_BASE                 0xf1c0a000
> +#define SW_VA_EMAC_IO_BASE                0xf1c0b000
> +#define SW_VA_TCON0_IO_BASE               0xf1c0c000
> +#define SW_VA_TCON1_IO_BASE               0xf1c0d000
> +#define SW_VA_VE_IO_BASE                  0xf1c0e000
> +#define SW_VA_SDC0_IO_BASE                0xf1c0f000
> +#define SW_VA_SDC1_IO_BASE                0xf1c10000
> +#define SW_VA_SDC2_IO_BASE                0xf1c11000
> +#define SW_VA_SDC3_IO_BASE                0xf1c12000
> +#define SW_VA_USB0_IO_BASE                0xf1c13000
> +#define SW_VA_USB1_IO_BASE                0xf1c14000
> +#define SW_VA_SSE_IO_BASE                 0xf1c15000
> +#define SW_VA_HDMI_IO_BASE                0xf1c16000
> +#define SW_VA_SPI2_IO_BASE                0xf1c17000
> +#define SW_VA_SATA_IO_BASE                0xf1c18000
> +#define SW_VA_PATA_IO_BASE                0xf1c19000
> +#define SW_VA_ACE_IO_BASE                 0xf1c1a000
> +#define SW_VA_TVE1_IO_BASE                0xf1c1b000
> +#define SW_VA_USB2_IO_BASE                0xf1c1c000
> +#define SW_VA_CSI1_IO_BASE                0xf1c1d000
> +#define SW_VA_TZASC_IO_BASE               0xf1c1e000
> +#define SW_VA_SPI3_IO_BASE                0xf1c1f000
> +#define SW_VA_CCM_IO_BASE                 0xf1c20000
> +#define SW_VA_INT_IO_BASE                 0xf1c20400
> +#define SW_VA_PORTC_IO_BASE               0xf1c20800
> +#define SW_VA_TIMERC_IO_BASE              0xf1c20c00
> +#define SW_VA_SPDIF_IO_BASE               0xf1c21000
> +#define SW_VA_AC97_IO_BASE                0xf1c21400
> +#define SW_VA_IR0_IO_BASE                 0xf1c21800
> +#define SW_VA_IR1_IO_BASE                 0xf1c21c00
> +#define SW_VA_IIS1_IO_BASE                0xf1c22000
> +#define SW_VA_IIS_IO_BASE                 0xf1c22400
> +#define SW_VA_LRADC_IO_BASE               0xf1c22800
> +#define SW_VA_ADDA_IO_BASE                0xf1c22c00
> +#define SW_VA_KEYPAD_IO_BASE              0xf1c23000
> +#define SW_VA_TZPC_IO_BASE                0xf1c23400
> +#define SW_VA_SID_IO_BASE                 0xf1c23800
> +#define SW_VA_SJTAG_IO_BASE               0xf1c23c00
> +#define SW_VA_IIS2_IO_BASE                0xf1c24400
> +#define SW_VA_TP_IO_BASE                  0xf1c25000
> +#define SW_VA_PMU_IO_BASE                 0xf1c25400
> +#define SW_VA_CPUCFG_IO_BASE              0xf1c25c00
> +#define SW_VA_UART0_IO_BASE               0xf1c28000
> +#define SW_VA_UART1_IO_BASE               0xf1c28400
> +#define SW_VA_UART2_IO_BASE               0xf1c28800
> +#define SW_VA_UART3_IO_BASE               0xf1c28c00
> +#define SW_VA_UART4_IO_BASE               0xf1c29000
> +#define SW_VA_UART5_IO_BASE               0xf1c29400
> +#define SW_VA_UART6_IO_BASE               0xf1c29800
> +#define SW_VA_UART7_IO_BASE               0xf1c29c00
> +#define SW_VA_PS20_IO_BASE                0xf1c2a000
> +#define SW_VA_PS21_IO_BASE                0xf1c2a400
> +#define SW_VA_TWI0_IO_BASE                0xf1c2ac00
> +#define SW_VA_TWI1_IO_BASE                0xf1c2b000
> +#define SW_VA_TWI2_IO_BASE                0xf1c2b400
> +#define SW_VA_TWI3_IO_BASE                0xf1c2b800
> +#define SW_VA_CAN0_IO_BASE                0xf1c2bc00
> +#define SW_VA_CAN1_IO_BASE                0xf1c2c000 /* sun4i */
> +#define SW_VA_TWI4_IO_BASE                0xf1c2c000 /* sun7i */
> +#define SW_VA_SCR_IO_BASE                 0xf1c2c400
> +#define SW_VA_GPS_IO_BASE                 0xf1c30000
> +#define SW_VA_MALI_IO_BASE                0xf1c40000
> +#define SW_VA_GMAC_IO_BASE                0xf1c50000
> +#define SW_VA_HSTIMER_IO_BASE             0xf1c60000
> +#define SW_VA_GIC_IO_BASE                 0xf1c80000
> +#define SW_VA_GIC_DIST_IO_BASE            0xf1c81000
> +#define SW_VA_GIC_CPU_IO_BASE             0xf1c82000
> +#define SW_VA_SCU_IO_BASE                 0xf1c80000
> +
> +#define SW_VA_TIMER_G_IO_BASE             0xf1c80200
> +/* CPU global timer, not used */
> +
> +#define SW_VA_TIMER_P_IO_BASE             0xf1c80600
> +/* CPU private timer, not used */
> +
> +#define SW_VA_HDMI1_IO_BASE               0xf1ce0000
> +#define SW_VA_SRAM_C_IO_BASE              0xf1d00000
> +#define SW_VA_DEFE0_IO_BASE               0xf1e00000
> +#define SW_VA_DEFE1_IO_BASE               0xf1e20000
> +#define SW_VA_DEBE0_IO_BASE               0xf1e60000
> +#define SW_VA_DEBE1_IO_BASE               0xf1e40000
> +#define SW_VA_MP_IO_BASE                  0xf1e80000
> +#define SW_VA_AVG_IO_BASE                 0xf1ea0000
> +
> +/* memory size */
> +#define SW_IO_SIZE                        0x00400000 /* 4MB(Max) */
> +#define SW_SRAM_A1_SIZE                   0x00004000 /* 16k */
> +#define SW_SRAM_A2_SIZE                   0x00004000 /* 16k */
> +#define SW_SRAM_A3_SIZE                   0x00003400 /* 13k */
> +#define SW_SRAM_A4_SIZE                   0x00000c00 /* 3k */
> +#define SW_SRAM_D_SIZE                    0x00001000 /* 4k */
> +#define SW_SRAM_B_SIZE                    0x00010000 /* 64k */
> +#define SW_BROM_SIZE                      0x00008000 /* 32k */
> +
> +/**
> + * Interrupt controller registers
> + *
> + */
> +#define SW_INT_VECTOR_REG                 (SW_VA_INT_IO_BASE + 0x00)
> +#define SW_INT_BASE_ADR_REG               (SW_VA_INT_IO_BASE + 0x04)
> +#define SW_INT_PROTECTION_REG             (SW_VA_INT_IO_BASE + 0x08)
> +#define SW_INT_NMI_CTRL_REG               (SW_VA_INT_IO_BASE + 0x0c)
> +#define SW_INT_IRQ_PENDING_REG0           (SW_VA_INT_IO_BASE + 0x10)
> +#define SW_INT_IRQ_PENDING_REG1           (SW_VA_INT_IO_BASE + 0x14)
> +#define SW_INT_IRQ_PENDING_REG2           (SW_VA_INT_IO_BASE + 0x18)
> +
> +#define SW_INT_FIQ_PENDING_REG0           (SW_VA_INT_IO_BASE + 0x20)
> +#define SW_INT_FIQ_PENDING_REG1           (SW_VA_INT_IO_BASE + 0x24)
> +#define SW_INT_FIQ_PENDING_REG2           (SW_VA_INT_IO_BASE + 0x28)
> +
> +#define SW_INT_SELECT_REG0                (SW_VA_INT_IO_BASE + 0x30)
> +#define SW_INT_SELECT_REG1                (SW_VA_INT_IO_BASE + 0x34)
> +#define SW_INT_SELECT_REG2                (SW_VA_INT_IO_BASE + 0x38)
> +
> +#define SW_INT_ENABLE_REG0                (SW_VA_INT_IO_BASE + 0x40)
> +#define SW_INT_ENABLE_REG1                (SW_VA_INT_IO_BASE + 0x44)
> +#define SW_INT_ENABLE_REG2                (SW_VA_INT_IO_BASE + 0x48)
> +
> +#define SW_INT_MASK_REG0                  (SW_VA_INT_IO_BASE + 0x50)
> +#define SW_INT_MASK_REG1                  (SW_VA_INT_IO_BASE + 0x54)
> +#define SW_INT_MASK_REG2                  (SW_VA_INT_IO_BASE + 0x58)
> +
> +#define SW_INT_RESP_REG0                  (SW_VA_INT_IO_BASE + 0x60)
> +#define SW_INT_RESP_REG1                  (SW_VA_INT_IO_BASE + 0x64)
> +#define SW_INT_RESP_REG2                  (SW_VA_INT_IO_BASE + 0x68)
> +
> +#define SW_INT_FASTFORCE_REG0             (SW_VA_INT_IO_BASE + 0x70)
> +#define SW_INT_FASTFORCE_REG1             (SW_VA_INT_IO_BASE + 0x74)
> +#define SW_INT_FASTFORCE_REG2             (SW_VA_INT_IO_BASE + 0x78)
> +
> +#define SW_INT_SRCPRIO_REG0               (SW_VA_INT_IO_BASE + 0x80)
> +#define SW_INT_SRCPRIO_REG1               (SW_VA_INT_IO_BASE + 0x84)
> +#define SW_INT_SRCPRIO_REG2               (SW_VA_INT_IO_BASE + 0x88)
> +#define SW_INT_SRCPRIO_REG3               (SW_VA_INT_IO_BASE + 0x8c)
> +#define SW_INT_SRCPRIO_REG4               (SW_VA_INT_IO_BASE + 0x90)
> +#ifdef CONFIG_ARCH_SUN5I
> +#define SW_INT_SRCPRIO_REG5               (SW_VA_INT_IO_BASE + 0x94)
> +#endif
> +
> +
> +#define PA_VIC_BASE                       0x01c20400
> +#define VA_VIC_BASE                       IO_ADDRESS(PA_VIC_BASE)
> +#define PIO_BASE                          SW_PA_PORTC_IO_BASE
> +
> +/**
> +*@name DRAM controller register address
> +*@{
> +*/
> +#define SW_DRAM_SDR_CTL_REG               (SW_VA_DRAM_IO_BASE + 0x0C)
> +#define SW_DRAM_SDR_DCR                   (SW_VA_DRAM_IO_BASE + 0x04)
> +
> +/*
> + * other reg defination, not found in spec
> + */
> +#define AW_GIC_DIST_BASE                  0x01c81000
> +#define AW_GIC_CPU_BASE                   0x01c82000
> +
> +#define AW_TIMER_G_BASE                   0x01c80200
> +/* CPU global timer, not used */
> +
> +#define AW_TIMER_P_BASE                   0x01c80600
> +/* CPU private timer, not used */
> +
> +/*
> + * CPUCFG
> + */
> +#define AW_CPUCFG_P_REG0            0x01a4
> +#define CPUX_RESET_CTL(x) (0x40 + (x)*0x40)
> +#define CPUX_CONTROL(x)   (0x44 + (x)*0x40)
> +#define CPUX_STATUS(x)    (0x48 + (x)*0x40)
> +#define AW_CPUCFG_GENCTL            0x0184
> +#define AW_CPUCFG_DBGCTL0           0x01e0
> +#define AW_CPUCFG_DBGCTL1           0x01e4
> +
> +#define AW_CPU1_PWR_CLAMP         0x01b0
> +#define AW_CPU1_PWROFF_REG        0x01b4

And move these definitions to platsmp.c.

I'd also like to have some mention that this is for sun7i only, so
naming the registers SUN7I_CPUCFG_* maybe? The A31 has a slightly
different SMP bringup mechanism, that we will need to support as well.

> +#endif
> diff --git a/arch/arm/mach-sunxi/platsmp.c b/arch/arm/mach-sunxi/platsmp.c
> new file mode 100644
> index 0000000..c76652d
> --- /dev/null
> +++ b/arch/arm/mach-sunxi/platsmp.c
> @@ -0,0 +1,100 @@
> +/*
> + *  linux/arch/arm/mach-sun7i/platsmp.c

The path is wrong here. And you probably can remove it actually.

> + *
> + *  Copyright (C) 2012-2016 Allwinner Ltd.

The copyright years are wrong as well.

> + *  All Rights Reserved
> + *
> + * merge it to 3.11 by cini <cinifr@gmail.com>

Use your real name here. And I would probably add your name to the
copyright holders, just aside with allwinner.

> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +#include <linux/init.h>
> +#include <linux/errno.h>
> +#include <linux/smp.h>
> +#include <linux/io.h>
> +#include <linux/delay.h>
> +#include <linux/device.h>
> +#include <linux/jiffies.h>
> +#include <linux/smp.h>
> +
> +#include <linux/of_address.h>
> +#include <linux/of_irq.h>
> +#include <linux/of_platform.h>
> +
> +
> +#include <linux/irqchip/arm-gic.h>
> +#include <asm/mach-types.h>
> +#include <asm/smp_scu.h>
> +#include <asm/cacheflush.h>
> +#include <asm/smp_plat.h>
> +
> +#include "platform.h"
> +#include <asm/arch_timer.h>

There seem to be a lot of unused includes here. You seem to only be
needing linux/smp.h, linux/io.h and platform.h

> +
> +
> +/*
> + * for linux/arch/arm/kernel/smp.c:__cpu_up(..)
> + */
> +static int sunxi7i_boot_secondary(unsigned int cpu, struct task_struct *idle)

sun7i will be enough :)

> +{
> +	long paddr;
> +	u32 pwr_reg;
> +	if (!sunxi7i_cc_base) {
> +		pr_debug("error map cpu configure\n");
> +		return -ENOSYS;
> +	}
> +
> +	paddr = virt_to_phys(sun7i_secondary_startup);
> +	writel(paddr, sunxi7i_cc_base + AW_CPUCFG_P_REG0);
> +	/* step1: Assert nCOREPORESET LOW and hold L1RSTDISABLE LOW.
> +		Ensure DBGPWRDUP is held LOW to prevent any external
> +		debug access to the processor.
> +	*/
> +	/* assert cpu core reset */
> +	writel(0, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
> +	/* L1RSTDISABLE hold low */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> +	pwr_reg &= ~(1<<cpu);
> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> +	/* DBGPWRDUP hold low */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> +	pwr_reg &= ~(1<<cpu);

Use BIT(cpu) here. And you should run scripts/checkpatch.pl on your
patches before sending them.

> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> +
> +	/* step2: release power clamp */
> +	writel(0xff, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x7f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x3f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x1f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x0f, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x07, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x03, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x01, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);
> +	writel(0x00, sunxi7i_cc_base + AW_CPU1_PWR_CLAMP);

You can probably turn it into a loop to simplify it.

> +	mdelay(10);

Ah, and you also need linux/delay.h

> +	/* step3: clear power-off gating */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
> +	pwr_reg &= ~(1);
> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPU1_PWROFF_REG);
> +	mdelay(1);
> +
> +	/* step4: de-assert core reset */
> +	writel(3, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
> +
> +	/* step5: assert DBGPWRDUP signal */
> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> +	pwr_reg |= (1<<cpu);
> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> +	return 0;
> +}
> +
> +
> +

No need for the extra new lines here, one will be enough.

> +struct smp_operations sunxi7i_smp_ops __initdata = {
> +	.smp_boot_secondary	= sunxi7i_boot_secondary,
> +};
> +
> +

Ditto.

> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
> index e79fb34..88e7ecb 100644
> --- a/arch/arm/mach-sunxi/sunxi.c
> +++ b/arch/arm/mach-sunxi/sunxi.c
> @@ -26,6 +26,9 @@
>  #include <asm/mach/map.h>
>  #include <asm/system_misc.h>
>  
> +#include "platform.h"
> +
> +

Ditto.

>  #define SUN4I_WATCHDOG_CTRL_REG		0x00
>  #define SUN4I_WATCHDOG_CTRL_RESTART		BIT(0)
>  #define SUN4I_WATCHDOG_MODE_REG		0x04
> @@ -42,6 +45,14 @@
>  #define SUN6I_WATCHDOG1_MODE_ENABLE		BIT(0)
>  
>  static void __iomem *wdt_base;
> +/*
> + * CPU Configure module support
> + * 1: Software reset for smp cpus
> + * 2: Configure for smp cpus including boot.
> + * 3: Three 64-bit idle counters and two 64-bit common counters
> + * it is needed for smp cpus
> + */
> +void __iomem *sunxi7i_cc_base; /*CPU Configure Base*/
>  
>  static void sun4i_restart(enum reboot_mode mode, const char *cmd)
>  {
> @@ -98,6 +109,11 @@ static struct of_device_id sunxi_restart_ids[] = {
>  	{ /*sentinel*/ }
>  };
>  
> +static struct of_device_id sunxi_cc_ids[] = {
> +	{ .compatible = "allwinner,sun7i-a20-cpuconfig"},
> +	{ /*sentinel*/ }
> +};
> +
>  static void sunxi_setup_restart(void)
>  {
>  	const struct of_device_id *of_id;
> @@ -125,7 +141,6 @@ static void __init sunxi_timer_init(void)
>  static void __init sunxi_dt_init(void)
>  {
>  	sunxi_setup_restart();
> -

This is useless.

>  	of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
>  }
>  
> @@ -138,7 +153,24 @@ static const char * const sunxi_board_dt_compat[] = {
>  	NULL,
>  };
>  
> +
> +static void __init sunxi_init_cpuconfig_map(void)
> +{
> +	struct device_node *np;
> +
> +	np = of_find_matching_node(NULL, sunxi_cc_ids);
> +	if (WARN(!np, "unable to setup cup configure"))
> +		return;
> +	sunxi7i_cc_base = of_iomap(np, 0);
> +	WARN(!sunxi7i_cc_base, "failed to map cup configure base address");
> +}
> +
> +
> +early_initcall(sunxi_init_cpuconfig_map);
> +
> +

I'm wondering, can't this go into the init_cpus SMP callback?

>  DT_MACHINE_START(SUNXI_DT, "Allwinner A1X (Device Tree)")
> +	.smp	= smp_ops(sunxi7i_smp_ops),

Hmm, no, that won't work. The A31 will have different SMP ops.

As part of the work I started on the SMP, I created a new machine for
the A20.

Let me send the patch, and base your work on top of it.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[cinifr]

[-- Attachment #1: Type: text/plain, Size: 96 bytes --]

Thanks. I have modify my patch as you said.
It is in attachments. Maybe I remail the new patch?

[-- Attachment #2: 0001-Add-smp-support-for-Allwinner-A20-sunxi-7i.patch --]
[-- Type: application/octet-stream, Size: 5940 bytes --]

From 4386db830ff29a55b12bdca78dda929862710825 Mon Sep 17 00:00:00 2001
From: Fan Rong <cinifr@gmail.com>
Date: Sun, 22 Sep 2013 00:46:04 +0800
Subject: [PATCH] Add smp support for Allwinner A20(sunxi 7i).

Signed-off-by: Fan Rong <cinifr@gmail.com>
---
 arch/arm/mach-sunxi/Makefile  |  2 +
 arch/arm/mach-sunxi/headsmp.S | 17 +++++++++
 arch/arm/mach-sunxi/platsmp.c | 86 +++++++++++++++++++++++++++++++++++++++++++
 arch/arm/mach-sunxi/sunxi.c   | 31 ++++++++++++++++
 4 files changed, 136 insertions(+)
 create mode 100644 arch/arm/mach-sunxi/headsmp.S
 create mode 100644 arch/arm/mach-sunxi/platsmp.c

diff --git a/arch/arm/mach-sunxi/Makefile b/arch/arm/mach-sunxi/Makefile
index 93bebfc..d7f1ef4 100644
--- a/arch/arm/mach-sunxi/Makefile
+++ b/arch/arm/mach-sunxi/Makefile
@@ -1 +1,3 @@
 obj-$(CONFIG_ARCH_SUNXI) += sunxi.o
+obj-$(CONFIG_ARCH_SUNXI) += platsmp.o
+obj-$(CONFIG_ARCH_SUNXI) += headsmp.o
diff --git a/arch/arm/mach-sunxi/headsmp.S b/arch/arm/mach-sunxi/headsmp.S
new file mode 100644
index 0000000..5899399
--- /dev/null
+++ b/arch/arm/mach-sunxi/headsmp.S
@@ -0,0 +1,17 @@
+/*
+ * SMP support for A20
+ *
+ *  Copyright (C) 2013 Fan Rong <cinifr@gmail.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/linkage.h>
+#include <linux/init.h>
+
+.section ".text.head", "ax" ENTRY(sun7i_secondary_startup)
+msr cpsr_fsxc,
+#0xd3
+b secondary_startup ENDPROC(sun7i_secondary_startup)
diff --git a/arch/arm/mach-sunxi/platsmp.c b/arch/arm/mach-sunxi/platsmp.c
new file mode 100644
index 0000000..5e3e994
--- /dev/null
+++ b/arch/arm/mach-sunxi/platsmp.c
@@ -0,0 +1,86 @@
+/*
+ *  linux/arch/arm/mach-sun7i/platsmp.c
+ *
+ *  Copyright (C) 2013 Fan Rong <cinifr@gmail.com>
+ *  All Rights Reserved
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+#include <linux/init.h>
+#include <linux/errno.h>
+#include <linux/smp.h>
+#include <linux/io.h>
+#include <linux/delay.h>
+#include <linux/smp.h>
+
+extern void __iomem *sunxi7i_cc_base;
+void sun7i_secondary_startup(void);
+
+/*
+ * CPUCFG
+ */
+#define SUN7I_CPUCFG_BOOTADDR	0x01a4
+
+#define SUN7I_CPUCFG_GENCTL	0x0184
+#define SUN7I_CPUCFG_DBGCTL0	0x01e0
+#define SUN7I_CPUCFG_DBGCTL1	0x01e4
+
+#define SUN7I_CPU1_PWR_CLAMP	0x01b0
+#define SUN7I_CPU1_PWROFF_REG	0x01b4
+#define SUN7I_CPUX_RESET_CTL(x)	(0x40 + (x)*0x40)
+
+static int sun7i_boot_secondary(unsigned int cpu, struct task_struct *idle)
+{
+	long paddr;
+	uint32_t pwr_reg;
+	uint32_t j = 0xff << 1;
+	if (!sunxi7i_cc_base) {
+		pr_debug("error map cpu configure\n");
+		return -ENOSYS;
+	}
+	/* Set boot addr */
+	paddr = virt_to_phys(sun7i_secondary_startup);
+	writel(paddr, sunxi7i_cc_base + SUN7I_CPUCFG_BOOTADDR);
+
+	/* Assert cpu core reset */
+	writel(0, sunxi7i_cc_base + SUN7I_CPUX_RESET_CTL(cpu));
+
+	/* Ensure CPU reset also invalidates L1 caches */
+	pwr_reg = readl(sunxi7i_cc_base + SUN7I_CPUCFG_GENCTL);
+	pwr_reg &= ~ BIT(cpu);
+	writel(pwr_reg, sunxi7i_cc_base + SUN7I_CPUCFG_GENCTL);
+
+	/* DBGPWRDUP hold low */
+	pwr_reg = readl(sunxi7i_cc_base + SUN7I_CPUCFG_DBGCTL1);
+	pwr_reg &= ~ BIT(cpu);
+	writel(pwr_reg, sunxi7i_cc_base + SUN7I_CPUCFG_DBGCTL1);
+
+	/* Ramp up power to CPU1 */
+	do {
+		writel(j, sunxi7i_cc_base + SUN7I_CPU1_PWR_CLAMP);
+		j = j >> 1;
+	} while (j != 0);
+
+	mdelay(10);
+
+	pwr_reg = readl(sunxi7i_cc_base + SUN7I_CPU1_PWROFF_REG);
+	pwr_reg &= ~1;
+	writel(pwr_reg, sunxi7i_cc_base + SUN7I_CPU1_PWROFF_REG);
+	mdelay(1);
+
+	/* Release CPU reset */
+	writel(3, sunxi7i_cc_base + SUN7I_CPUX_RESET_CTL(cpu));
+
+	/* Unlock CPU */
+	pwr_reg = readl(sunxi7i_cc_base + SUN7I_CPUCFG_DBGCTL1);
+	pwr_reg |= BIT(cpu);
+	writel(pwr_reg, sunxi7i_cc_base + SUN7I_CPUCFG_DBGCTL1);
+
+	return 0;
+}
+
+struct smp_operations sun7i_smp_ops __initdata = {
+	.smp_boot_secondary = sun7i_boot_secondary,
+};
diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
index e79fb34..a692350 100644
--- a/arch/arm/mach-sunxi/sunxi.c
+++ b/arch/arm/mach-sunxi/sunxi.c
@@ -26,6 +26,8 @@
 #include <asm/mach/map.h>
 #include <asm/system_misc.h>
 
+extern struct smp_operations sun7i_smp_ops;
+
 #define SUN4I_WATCHDOG_CTRL_REG		0x00
 #define SUN4I_WATCHDOG_CTRL_RESTART		BIT(0)
 #define SUN4I_WATCHDOG_MODE_REG		0x04
@@ -42,6 +44,14 @@
 #define SUN6I_WATCHDOG1_MODE_ENABLE		BIT(0)
 
 static void __iomem *wdt_base;
+/*
+ * CPU Configure module support
+ * 1: Software reset for smp cpus
+ * 2: Configure for smp cpus including boot.
+ * 3: Three 64-bit idle counters and two 64-bit common counters
+ * it is needed for smp cpus
+ */
+void __iomem *sunxi7i_cc_base; /*CPU Configure Base*/
 
 static void sun4i_restart(enum reboot_mode mode, const char *cmd)
 {
@@ -98,6 +108,11 @@ static struct of_device_id sunxi_restart_ids[] = {
 	{ /*sentinel*/ }
 };
 
+static struct of_device_id sunxi_cc_ids[] = {
+	{ .compatible = "allwinner,sun7i-a20-cpuconfig"},
+	{ /*sentinel*/ }
+};
+
 static void sunxi_setup_restart(void)
 {
 	const struct of_device_id *of_id;
@@ -138,7 +153,23 @@ static const char * const sunxi_board_dt_compat[] = {
 	NULL,
 };
 
+static int __init sunxi_init_cpuconfig_map(void)
+{
+	struct device_node *np;
+
+	np = of_find_matching_node(NULL, sunxi_cc_ids);
+	if (WARN(!np, "unable to setup cup configure"))
+		return -ENOSYS;
+	sunxi7i_cc_base = of_iomap(np, 0);
+	if (WARN(!sunxi7i_cc_base, "failed to map cup configure base address"))
+		return -ENOSYS;
+	return 0;
+}
+
+early_initcall(sunxi_init_cpuconfig_map);
+
 DT_MACHINE_START(SUNXI_DT, "Allwinner A1X (Device Tree)")
+	.smp	= smp_ops(sun7i_smp_ops),
 	.init_machine	= sunxi_dt_init,
 	.init_time	= sunxi_timer_init,
 	.dt_compat	= sunxi_board_dt_compat,
-- 
1.8.1.2

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Russell King - ARM Linux]

On Sun, Sep 22, 2013 at 12:49:45AM +0800, cinifr wrote:
> Thanks. I have modify my patch as you said.
> It is in attachments. Maybe I remail the new patch?

Please don't attach patches (or if you do, ensure that they have an
inline disposition) as non-inline attachments can't be easily quoted
and replied to.

Thanks.

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 419 bytes --]

On Sun, Sep 22, 2013 at 12:49:45AM +0800, cinifr wrote:
> Thanks. I have modify my patch as you said.
> It is in attachments. Maybe I remail the new patch?

Yes, remail the whole serie and change the prefix to PATCHv3 (you can
use the --subject-prefix option of git format-patch to do so).

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Uwe Kleine-König]

Hello,

On Sat, Sep 21, 2013 at 05:49:00PM -0500, Maxime Ripard wrote:
> On Sun, Sep 22, 2013 at 12:49:45AM +0800, cinifr wrote:
> > Thanks. I have modify my patch as you said.
> > It is in attachments. Maybe I remail the new patch?
> 
> Yes, remail the whole serie and change the prefix to PATCHv3 (you can
> use the --subject-prefix option of git format-patch to do so).
git >= 1.8.2 also support --reroll-count 3 which also changes the filenames
of the patches.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Guenter Roeck]

On 09/21/2013 07:00 AM, Maxime Ripard wrote:

[ ... ]

>> +	/* assert cpu core reset */
>> +	writel(0, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
>> +	/* L1RSTDISABLE hold low */
>> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_GENCTL);
>> +	pwr_reg &= ~(1<<cpu);
>> +	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_GENCTL);
>> +	/* DBGPWRDUP hold low */
>> +	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
>> +	pwr_reg &= ~(1<<cpu);
>
> Use BIT(cpu) here. And you should run scripts/checkpatch.pl on your
> patches before sending them.
>

For the record:

$ scripts/checkpatch.pl --strict allwinner.patch
total: 0 errors, 0 warnings, 0 checks, 527 lines checked

allwinner.patch has no obvious style problems and is ready for submission.

This is on top of "v3.12-rc1-336-gd8524ae".

checkpatch.pl does not complain as long as the number of spaces before
and after an operator is the same. There is a patch pending to change
this with the --strict option, but it will still not complain in 'normal'
operation.

Guenter

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Maxime Ripard]

[-- Attachment #1: Type: text/plain, Size: 1396 bytes --]

Hi Guenter,

On Sun, Sep 22, 2013 at 08:37:23PM -0700, Guenter Roeck wrote:
> On 09/21/2013 07:00 AM, Maxime Ripard wrote:
> 
> [ ... ]
> 
> >>+	/* assert cpu core reset */
> >>+	writel(0, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
> >>+	/* L1RSTDISABLE hold low */
> >>+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> >>+	pwr_reg &= ~(1<<cpu);
> >>+	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> >>+	/* DBGPWRDUP hold low */
> >>+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> >>+	pwr_reg &= ~(1<<cpu);
> >
> >Use BIT(cpu) here. And you should run scripts/checkpatch.pl on your
> >patches before sending them.
> >
> 
> For the record:
> 
> $ scripts/checkpatch.pl --strict allwinner.patch
> total: 0 errors, 0 warnings, 0 checks, 527 lines checked
> 
> allwinner.patch has no obvious style problems and is ready for submission.
> 
> This is on top of "v3.12-rc1-336-gd8524ae".
> 
> checkpatch.pl does not complain as long as the number of spaces before
> and after an operator is the same. There is a patch pending to change
> this with the --strict option, but it will still not complain in 'normal'
> operation.

Hmmm, ok. Somehow, I was convinced that it was triggering a warning.
Thanks for the heads up :)

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

[Guenter Roeck]

On Mon, Sep 23, 2013 at 09:19:17PM +0300, Maxime Ripard wrote:
> Hi Guenter,
> 
> On Sun, Sep 22, 2013 at 08:37:23PM -0700, Guenter Roeck wrote:
> > On 09/21/2013 07:00 AM, Maxime Ripard wrote:
> > 
> > [ ... ]
> > 
> > >>+	/* assert cpu core reset */
> > >>+	writel(0, sunxi7i_cc_base + CPUX_RESET_CTL(cpu));
> > >>+	/* L1RSTDISABLE hold low */
> > >>+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> > >>+	pwr_reg &= ~(1<<cpu);
> > >>+	writel(pwr_reg, sunxi7i_cc_base + AW_CPUCFG_GENCTL);
> > >>+	/* DBGPWRDUP hold low */
> > >>+	pwr_reg = readl(sunxi7i_cc_base + AW_CPUCFG_DBGCTL1);
> > >>+	pwr_reg &= ~(1<<cpu);
> > >
> > >Use BIT(cpu) here. And you should run scripts/checkpatch.pl on your
> > >patches before sending them.
> > >
> > 
> > For the record:
> > 
> > $ scripts/checkpatch.pl --strict allwinner.patch
> > total: 0 errors, 0 warnings, 0 checks, 527 lines checked
> > 
> > allwinner.patch has no obvious style problems and is ready for submission.
> > 
> > This is on top of "v3.12-rc1-336-gd8524ae".
> > 
> > checkpatch.pl does not complain as long as the number of spaces before
> > and after an operator is the same. There is a patch pending to change
> > this with the --strict option, but it will still not complain in 'normal'
> > operation.
> 
> Hmmm, ok. Somehow, I was convinced that it was triggering a warning.
> Thanks for the heads up :)
> 
Yes, me too until I realized that it doesn't :(.

Guenter

[PATCH 2/3] Add cpuconfig nodes in dts for smp configure.

[Fan Rong]

Signed-off-by: Fan Rong <cinifr@gmail.com>
---
 arch/arm/boot/dts/sun7i-a20.dtsi | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi
index 999ff45..f745e0b 100644
--- a/arch/arm/boot/dts/sun7i-a20.dtsi
+++ b/arch/arm/boot/dts/sun7i-a20.dtsi
@@ -20,13 +20,13 @@
 		#address-cells = <1>;
 		#size-cells = <0>;
 
-		cpu@0 {
+		cpu0: cpu@0  {
 			compatible = "arm,cortex-a7";
 			device_type = "cpu";
 			reg = <0>;
 		};
 
-		cpu@1 {
+		cpu1: cpu@1 {
 			compatible = "arm,cortex-a7";
 			device_type = "cpu";
 			reg = <1>;
@@ -167,6 +167,11 @@
 		#size-cells = <1>;
 		ranges;
 
+		cpuconfig: cpuconfig@01c25c00 {
+			compatible = "allwinner,sun7i-a20-cpuconfig";
+			reg = <0x01c25c00 0x400>;
+		};
+
 		pio: pinctrl@01c20800 {
 			compatible = "allwinner,sun7i-a20-pinctrl";
 			reg = <0x01c20800 0x400>;
-- 
1.8.1.2

[PATCH 3/3] Add arch count timer node in dts for Allwinner A20(sunxi 7i).

[Fan Rong]

Signed-off-by: Fan Rong <cinifr@gmail.com>
---
 arch/arm/boot/dts/sun7i-a20.dtsi | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi
index f745e0b..76b8c3f 100644
--- a/arch/arm/boot/dts/sun7i-a20.dtsi
+++ b/arch/arm/boot/dts/sun7i-a20.dtsi
@@ -312,5 +312,15 @@
 			#interrupt-cells = <3>;
 			interrupts = <1 9 0xf04>;
 		};
+
+		timer {
+			compatible ="arm,armv7-timer";
+			interrupts = <1 13 0x308>,
+				     <1 14 0x308>,
+				     <1 11 0x308>,
+				     <1 10 0x308>;
+			clock-frequency = <24000000>;
+		};
+
 	};
 };
-- 
1.8.1.2

[PATCH 0/3]

[Antoniu Miclaus]

The ADMV1014 is a silicon germanium (SiGe), wideband,
microwave downconverter optimized for point to point microwave
radio designs operating in the 24 GHz to 44 GHz frequency range.

Datasheet:
https://www.analog.com/media/en/technical-documentation/data-sheets/ADMV1014.pdf

NOTE:
Currently depends on 64-bit architecture since the input
clock that server as Local Oscillator should support values
in the range 24 GHz to 44 GHz.

We might need some scaling implementation in the clock
framework so that u64 types are supported when using 32-bit
architectures.

Antoniu Miclaus (3):
  iio:frequency:admv1014: add support for ADMV1014
  dt-bindings:iio:frequency: add admv1014 doc
  Documentation:ABI:testing:admv1014: add ABI docs

 .../testing/sysfs-bus-iio-frequency-admv1014  |  23 +
 .../bindings/iio/frequency/adi,admv1014.yaml  |  97 +++
 drivers/iio/frequency/Kconfig                 |  10 +
 drivers/iio/frequency/Makefile                |   1 +
 drivers/iio/frequency/admv1014.c              | 784 ++++++++++++++++++
 5 files changed, 915 insertions(+)
 create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-frequency-admv1014
 create mode 100644 Documentation/devicetree/bindings/iio/frequency/adi,admv1014.yaml
 create mode 100644 drivers/iio/frequency/admv1014.c

-- 
2.34.1

[PATCH 0/3]

[Matthias Kaehlcke]

This series includes a fix for a possible race in qca_suspend() and
some minor refactoring of the same function.


Matthias Kaehlcke (3):
  Bluetooth: hci_qca: Only remove TX clock vote after TX is completed
  Bluetooth: hci_qca: Skip serdev wait when no transfer is pending
  Bluetooth: hci_qca: Refactor error handling in qca_suspend()

 drivers/bluetooth/hci_qca.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

-- 
2.27.0.278.ge193c7cf3a9-goog

[PATCH 0/3]

[Gareth Williams]

This series adds DRM support for the Digital Blocks db9000
LCD controller with RZ/N1 specific changes and updates simple-panel to
include the associated panel. As this has not previously been
documented, also include a yaml file to provide this.

Gareth Williams (3):
  drm/db9000: Add Digital Blocks DB9000 LCD Controller
  drm/db9000: Add bindings documentation for LCD controller
  drm/panel: simple: Add Newhaven ATXL#-CTP panel

 .../devicetree/bindings/display/db9000,du.yaml     |  87 ++
 .../devicetree/bindings/vendor-prefixes.yaml       |   2 +
 drivers/gpu/drm/Kconfig                            |   2 +
 drivers/gpu/drm/Makefile                           |   1 +
 drivers/gpu/drm/digital-blocks/Kconfig             |  13 +
 drivers/gpu/drm/digital-blocks/Makefile            |   3 +
 drivers/gpu/drm/digital-blocks/db9000-du.c         | 953 +++++++++++++++++++++
 drivers/gpu/drm/digital-blocks/db9000-du.h         | 192 +++++
 drivers/gpu/drm/panel/panel-simple.c               |  27 +
 9 files changed, 1280 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/db9000,du.yaml
 create mode 100644 drivers/gpu/drm/digital-blocks/Kconfig
 create mode 100644 drivers/gpu/drm/digital-blocks/Makefile
 create mode 100644 drivers/gpu/drm/digital-blocks/db9000-du.c
 create mode 100644 drivers/gpu/drm/digital-blocks/db9000-du.h

-- 
2.7.4

RE: [PATCH 0/3]

[Gareth Williams]

Hi All,

I noticed some API changes that were not present when I first wrote this driver. 
This will need correcting so I will send out a second version and respond 
to Sam Ravnborg's feedback at the same time. I recommend waiting for that
version before reviewing as this will not function on Linux-next otherwise.

Gareth

On Mon, Apr 27, 2020 at 09:21:49AM +0100, Gareth Williams wrote:
> 
> This series adds DRM support for the Digital Blocks db9000 LCD controller with
> RZ/N1 specific changes and updates simple-panel to include the associated
> panel. As this has not previously been documented, also include a yaml file to
> provide this.
> 
> Gareth Williams (3):
>   drm/db9000: Add Digital Blocks DB9000 LCD Controller
>   drm/db9000: Add bindings documentation for LCD controller
>   drm/panel: simple: Add Newhaven ATXL#-CTP panel
> 
>  .../devicetree/bindings/display/db9000,du.yaml     |  87 ++
>  .../devicetree/bindings/vendor-prefixes.yaml       |   2 +
>  drivers/gpu/drm/Kconfig                            |   2 +
>  drivers/gpu/drm/Makefile                           |   1 +
>  drivers/gpu/drm/digital-blocks/Kconfig             |  13 +
>  drivers/gpu/drm/digital-blocks/Makefile            |   3 +
>  drivers/gpu/drm/digital-blocks/db9000-du.c         | 953
> +++++++++++++++++++++
>  drivers/gpu/drm/digital-blocks/db9000-du.h         | 192 +++++
>  drivers/gpu/drm/panel/panel-simple.c               |  27 +
>  9 files changed, 1280 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/display/db9000,du.yaml
>  create mode 100644 drivers/gpu/drm/digital-blocks/Kconfig
>  create mode 100644 drivers/gpu/drm/digital-blocks/Makefile
>  create mode 100644 drivers/gpu/drm/digital-blocks/db9000-du.c
>  create mode 100644 drivers/gpu/drm/digital-blocks/db9000-du.h
> 
> --
> 2.7.4

[PATCH 0/3]

[Jani Nikula]

Hi Greg, Andrew -

Patch 1 is for DMI, bugfixes in patches 2-3 for i915 and included for
completeness. After a tested-by they should be good for stable. I'll
leave it to Daniel to sort out how the last two get in.

BR,
Jani.

Chris Wilson (1):
  drm/i915: Quirk away phantom LVDS on Intel's D510MO mainboard

Jani Nikula (2):
  dmi: add support for exact DMI matches in addition to substring
    matching
  drm/i915: Quirk away phantom LVDS on Intel's D525MW mainboard

 drivers/firmware/dmi_scan.c       |   12 +++++++++---
 drivers/gpu/drm/i915/intel_lvds.c |   16 ++++++++++++++++
 include/linux/mod_devicetable.h   |    6 ++++--
 3 files changed, 29 insertions(+), 5 deletions(-)

-- 
1.7.9.5

Re: [PATCH 0/3]

[Jani Nikula]

With Greg's address fixed. Please drop the old one from any
replies. Sorry for the noise.

On Thu, 06 Jun 2013, Jani Nikula <jani.nikula@intel.com> wrote:
> Hi Greg, Andrew -
>
> Patch 1 is for DMI, bugfixes in patches 2-3 for i915 and included for
> completeness. After a tested-by they should be good for stable. I'll
> leave it to Daniel to sort out how the last two get in.
>
> BR,
> Jani.
>
> Chris Wilson (1):
>   drm/i915: Quirk away phantom LVDS on Intel's D510MO mainboard
>
> Jani Nikula (2):
>   dmi: add support for exact DMI matches in addition to substring
>     matching
>   drm/i915: Quirk away phantom LVDS on Intel's D525MW mainboard
>
>  drivers/firmware/dmi_scan.c       |   12 +++++++++---
>  drivers/gpu/drm/i915/intel_lvds.c |   16 ++++++++++++++++
>  include/linux/mod_devicetable.h   |    6 ++++--
>  3 files changed, 29 insertions(+), 5 deletions(-)
>
> -- 
> 1.7.9.5
>

-- 
Jani Nikula, Intel Open Source Technology Center

Re: [PATCH 0/3]

[Daniel Vetter]

On Thu, Jun 06, 2013 at 04:59:26PM +0300, Jani Nikula wrote:
> 
> With Greg's address fixed. Please drop the old one from any
> replies. Sorry for the noise.

Oops, replied with the old one still there.

Greg, Andrew: Imo it's best to merge all three patches through the same
tree, so:

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

on the i915 parts of it. If you want I can also slurp them in through the
intel tree, including the new dmi match code.

Thanks, Daniel

> 
> On Thu, 06 Jun 2013, Jani Nikula <jani.nikula@intel.com> wrote:
> > Hi Greg, Andrew -
> >
> > Patch 1 is for DMI, bugfixes in patches 2-3 for i915 and included for
> > completeness. After a tested-by they should be good for stable. I'll
> > leave it to Daniel to sort out how the last two get in.
> >
> > BR,
> > Jani.
> >
> > Chris Wilson (1):
> >   drm/i915: Quirk away phantom LVDS on Intel's D510MO mainboard
> >
> > Jani Nikula (2):
> >   dmi: add support for exact DMI matches in addition to substring
> >     matching
> >   drm/i915: Quirk away phantom LVDS on Intel's D525MW mainboard
> >
> >  drivers/firmware/dmi_scan.c       |   12 +++++++++---
> >  drivers/gpu/drm/i915/intel_lvds.c |   16 ++++++++++++++++
> >  include/linux/mod_devicetable.h   |    6 ++++--
> >  3 files changed, 29 insertions(+), 5 deletions(-)
> >
> > -- 
> > 1.7.9.5
> >
> 
> -- 
> Jani Nikula, Intel Open Source Technology Center

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

Re: [PATCH 0/3]

[Greg Kroah-Hartman]

On Thu, Jun 13, 2013 at 10:22:05AM +0200, Daniel Vetter wrote:
> On Thu, Jun 06, 2013 at 04:59:26PM +0300, Jani Nikula wrote:
> > 
> > With Greg's address fixed. Please drop the old one from any
> > replies. Sorry for the noise.
> 
> Oops, replied with the old one still there.
> 
> Greg, Andrew: Imo it's best to merge all three patches through the same
> tree, so:
> 
> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 
> on the i915 parts of it. If you want I can also slurp them in through the
> intel tree, including the new dmi match code.

Please feel free to take them through your tree, I don't need to take
them.

thanks,

greg k-h

Re: [PATCH 0/3]

[Daniel Vetter]

On Thu, Jun 06, 2013 at 04:53:01PM +0300, Jani Nikula wrote:
> Hi Greg, Andrew -
> 
> Patch 1 is for DMI, bugfixes in patches 2-3 for i915 and included for
> completeness. After a tested-by they should be good for stable. I'll
> leave it to Daniel to sort out how the last two get in.

I'd prefer all to go through the same tree (to avoid tracking them), and
conflicts around lvds quirks will be trivial at most. So no problem for me
if this doesn't go in through drm-next. So

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

on the i915 patches for merging through whatever tree the drm stuff goes
through.
-Daniel

> 
> BR,
> Jani.
> 
> Chris Wilson (1):
>   drm/i915: Quirk away phantom LVDS on Intel's D510MO mainboard
> 
> Jani Nikula (2):
>   dmi: add support for exact DMI matches in addition to substring
>     matching
>   drm/i915: Quirk away phantom LVDS on Intel's D525MW mainboard
> 
>  drivers/firmware/dmi_scan.c       |   12 +++++++++---
>  drivers/gpu/drm/i915/intel_lvds.c |   16 ++++++++++++++++
>  include/linux/mod_devicetable.h   |    6 ++++--
>  3 files changed, 29 insertions(+), 5 deletions(-)
> 
> -- 
> 1.7.9.5
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

[PATCH 0/3]

[Petr Baudis]

The following series implements...

-- 
And on the eigth day, God started debugging.