diff mbox

[1/2] ARM: npcm: add CONFIG_ARCH_MULTI_V7 dependency

Message ID CAK8P3a2=0ZUqyhmKVa4u_VvOg4rtd6vUWdpRojfsoP3OSvGyeQ@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Arnd Bergmann March 12, 2018, 1:37 p.m. UTC
On Mon, Mar 12, 2018 at 2:28 PM, Tomer Maimon <tmaimon77@gmail.com> wrote:
> Hi Brendan,
>
> According to the last mail I have with Arnd can you modify the Kconfig
> as follow:
>
> +menuconfig ARCH_NPCM
> +       bool "Nuvoton NPCM Architecture"
> +       depends on ARCH_MULTI_V7
> +       select PINCTRL
> +
> +if ARCH_NPCM
> +
> +config ARCH_NPCM7XX
> +       bool "Support for NPCM7xx BMC (Poleg)"
> +       depends on ARCH_MULTI_V7
> +       select PINCTRL_NPCM7XX
> +       select NPCM7XX_TIMER
> +       select ARCH_REQUIRE_GPIOLIB
> +       select CACHE_L2X0
> +       select ARM_GIC
> +       select HAVE_ARM_TWD if SMP
> +       select HAVE_ARM_SCU if SMP
> +       select ARM_ERRATA_764369 if SMP
> +       select ARM_ERRATA_720789
> +       select ARM_ERRATA_754322
> +       select ARM_ERRATA_794072
> +       select PL310_ERRATA_588369
> +       select PL310_ERRATA_727915
> +       select MFD_SYSCON
> +       help
> +         General support for NPCM7xx BMC (Poleg).
> +
> +         Nuvoton NPCM7xx BMC based on the Cortex A9.
> +
> +endif

One more change is needed to avoid a link error without CONFIG_SMP:


Can one of you send me a pull request or the full patch series on top
of the version I have in arm-soc?

      Arnd

Comments

Tomer Maimon March 12, 2018, 3:15 p.m. UTC | #1
Hi Arnd,

I will send tomorrow new version with some modification we need to do
once we use NPCM7XX and not NPCM750.

Brendan,

Is it O.K.?



On 12 March 2018 at 15:37, Arnd Bergmann <arnd@arndb.de> wrote:
> On Mon, Mar 12, 2018 at 2:28 PM, Tomer Maimon <tmaimon77@gmail.com> wrote:
>> Hi Brendan,
>>
>> According to the last mail I have with Arnd can you modify the Kconfig
>> as follow:
>>
>> +menuconfig ARCH_NPCM
>> +       bool "Nuvoton NPCM Architecture"
>> +       depends on ARCH_MULTI_V7
>> +       select PINCTRL
>> +
>> +if ARCH_NPCM
>> +
>> +config ARCH_NPCM7XX
>> +       bool "Support for NPCM7xx BMC (Poleg)"
>> +       depends on ARCH_MULTI_V7
>> +       select PINCTRL_NPCM7XX
>> +       select NPCM7XX_TIMER
>> +       select ARCH_REQUIRE_GPIOLIB
>> +       select CACHE_L2X0
>> +       select ARM_GIC
>> +       select HAVE_ARM_TWD if SMP
>> +       select HAVE_ARM_SCU if SMP
>> +       select ARM_ERRATA_764369 if SMP
>> +       select ARM_ERRATA_720789
>> +       select ARM_ERRATA_754322
>> +       select ARM_ERRATA_794072
>> +       select PL310_ERRATA_588369
>> +       select PL310_ERRATA_727915
>> +       select MFD_SYSCON
>> +       help
>> +         General support for NPCM7xx BMC (Poleg).
>> +
>> +         Nuvoton NPCM7xx BMC based on the Cortex A9.
>> +
>> +endif
>
> One more change is needed to avoid a link error without CONFIG_SMP:
>
> diff --git a/arch/arm/mach-npcm/Makefile b/arch/arm/mach-npcm/Makefile
> index c7a1316d27c1..34d51f9f207c 100644
> --- a/arch/arm/mach-npcm/Makefile
> +++ b/arch/arm/mach-npcm/Makefile
> @@ -1,3 +1,4 @@
>  AFLAGS_headsmp.o               += -march=armv7-a
>
> -obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o platsmp.o headsmp.o
> +obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o
> +obj-$(CONFIG_SMP)              += platsmp.o headsmp.o
>
> Can one of you send me a pull request or the full patch series on top
> of the version I have in arm-soc?
>
>       Arnd
Tomer Maimon March 13, 2018, 1:35 p.m. UTC | #2
Hi Arnd,

I sent now Full patch for the NPCM BMC's V12,

Is it O.K.? or do you want me to send the same patch to ARM: npcm:
drop extraneous 'select' statements as well?

Thanks

Tomer


On 12 March 2018 at 17:15, Tomer Maimon <tmaimon77@gmail.com> wrote:
> Hi Arnd,
>
> I will send tomorrow new version with some modification we need to do
> once we use NPCM7XX and not NPCM750.
>
> Brendan,
>
> Is it O.K.?
>
>
>
> On 12 March 2018 at 15:37, Arnd Bergmann <arnd@arndb.de> wrote:
>> On Mon, Mar 12, 2018 at 2:28 PM, Tomer Maimon <tmaimon77@gmail.com> wrote:
>>> Hi Brendan,
>>>
>>> According to the last mail I have with Arnd can you modify the Kconfig
>>> as follow:
>>>
>>> +menuconfig ARCH_NPCM
>>> +       bool "Nuvoton NPCM Architecture"
>>> +       depends on ARCH_MULTI_V7
>>> +       select PINCTRL
>>> +
>>> +if ARCH_NPCM
>>> +
>>> +config ARCH_NPCM7XX
>>> +       bool "Support for NPCM7xx BMC (Poleg)"
>>> +       depends on ARCH_MULTI_V7
>>> +       select PINCTRL_NPCM7XX
>>> +       select NPCM7XX_TIMER
>>> +       select ARCH_REQUIRE_GPIOLIB
>>> +       select CACHE_L2X0
>>> +       select ARM_GIC
>>> +       select HAVE_ARM_TWD if SMP
>>> +       select HAVE_ARM_SCU if SMP
>>> +       select ARM_ERRATA_764369 if SMP
>>> +       select ARM_ERRATA_720789
>>> +       select ARM_ERRATA_754322
>>> +       select ARM_ERRATA_794072
>>> +       select PL310_ERRATA_588369
>>> +       select PL310_ERRATA_727915
>>> +       select MFD_SYSCON
>>> +       help
>>> +         General support for NPCM7xx BMC (Poleg).
>>> +
>>> +         Nuvoton NPCM7xx BMC based on the Cortex A9.
>>> +
>>> +endif
>>
>> One more change is needed to avoid a link error without CONFIG_SMP:
>>
>> diff --git a/arch/arm/mach-npcm/Makefile b/arch/arm/mach-npcm/Makefile
>> index c7a1316d27c1..34d51f9f207c 100644
>> --- a/arch/arm/mach-npcm/Makefile
>> +++ b/arch/arm/mach-npcm/Makefile
>> @@ -1,3 +1,4 @@
>>  AFLAGS_headsmp.o               += -march=armv7-a
>>
>> -obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o platsmp.o headsmp.o
>> +obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o
>> +obj-$(CONFIG_SMP)              += platsmp.o headsmp.o
>>
>> Can one of you send me a pull request or the full patch series on top
>> of the version I have in arm-soc?
>>
>>       Arnd
Arnd Bergmann March 13, 2018, 4:23 p.m. UTC | #3
On Tue, Mar 13, 2018 at 2:35 PM, Tomer Maimon <tmaimon77@gmail.com> wrote:
> Hi Arnd,
>
> I sent now Full patch for the NPCM BMC's V12,
>
> Is it O.K.? or do you want me to send the same patch to ARM: npcm:
> drop extraneous 'select' statements as well?

The patch series you just sent contains a new version of the patches
I already applied. What I need instead is a patch series that
does the changes on top, since I don't rebase the my next/* branches.

      Arnd
diff mbox

Patch

diff --git a/arch/arm/mach-npcm/Makefile b/arch/arm/mach-npcm/Makefile
index c7a1316d27c1..34d51f9f207c 100644
--- a/arch/arm/mach-npcm/Makefile
+++ b/arch/arm/mach-npcm/Makefile
@@ -1,3 +1,4 @@ 
 AFLAGS_headsmp.o               += -march=armv7-a

-obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o platsmp.o headsmp.o
+obj-$(CONFIG_ARCH_NPCM750)     += npcm7xx.o
+obj-$(CONFIG_SMP)              += platsmp.o headsmp.o