From: Tony Lindgren <tony@atomide.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org, arm@kernel.org,
linux-omap@vger.kernel.org
Subject: Re: [PATCH 02/14] ARM: omap2: fix am43xx dependency on l2x0 cache
Date: Mon, 16 Jun 2014 02:41:56 -0700 [thread overview]
Message-ID: <20140616094156.GA17845@atomide.com> (raw)
In-Reply-To: <1402675276-538682-3-git-send-email-arnd@arndb.de>
* Arnd Bergmann <arnd@arndb.de> [140613 09:02]:
> Commit d941f86fad41b ("ARM: l2c: AM43x: add L2 cache support") enabled
> the L2 cache support for the am43xx SoC, but caused a build regression
> when the driver for that cache controller is disabled:
>
> arch/arm/mach-omap2/built-in.o: In function `am43xx_init_early':
> :(.init.text+0xb20): undefined reference to `omap_l2_cache_init'
>
> This did not happen for OMAP4, which has the same call, but enables
> the l2x0 driver unconditionally. We could do the same thing for
> am43xx, but it seems better to allow turning it off and make the
> code work in either case.
>
> This adds an inline wrapper for omap_l2_cache_init for the disabled
> case, and removes the 'select' from OMAP4 so it becomes a user
> visible option.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Cc: linux-omap@vger.kernel.org
Acked-by: Tony Lindgren <tony@atomide.com>
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 02/14] ARM: omap2: fix am43xx dependency on l2x0 cache
Date: Mon, 16 Jun 2014 02:41:56 -0700 [thread overview]
Message-ID: <20140616094156.GA17845@atomide.com> (raw)
In-Reply-To: <1402675276-538682-3-git-send-email-arnd@arndb.de>
* Arnd Bergmann <arnd@arndb.de> [140613 09:02]:
> Commit d941f86fad41b ("ARM: l2c: AM43x: add L2 cache support") enabled
> the L2 cache support for the am43xx SoC, but caused a build regression
> when the driver for that cache controller is disabled:
>
> arch/arm/mach-omap2/built-in.o: In function `am43xx_init_early':
> :(.init.text+0xb20): undefined reference to `omap_l2_cache_init'
>
> This did not happen for OMAP4, which has the same call, but enables
> the l2x0 driver unconditionally. We could do the same thing for
> am43xx, but it seems better to allow turning it off and make the
> code work in either case.
>
> This adds an inline wrapper for omap_l2_cache_init for the disabled
> case, and removes the 'select' from OMAP4 so it becomes a user
> visible option.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Cc: linux-omap at vger.kernel.org
Acked-by: Tony Lindgren <tony@atomide.com>
next prev parent reply other threads:[~2014-06-16 9:42 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-13 16:01 [PATCH 00/14] arm-soc fixes for 3.16 review Arnd Bergmann
2014-06-13 16:01 ` Arnd Bergmann
2014-06-13 16:01 ` [PATCH 01/14] ARM: mvebu: fix randconfig builds for pmsu driver Arnd Bergmann
2014-06-17 14:00 ` Thomas Petazzoni
2014-06-17 15:03 ` Arnd Bergmann
2014-06-17 15:17 ` Thomas Petazzoni
2014-06-17 15:20 ` Arnd Bergmann
2014-06-13 16:01 ` [PATCH 02/14] ARM: omap2: fix am43xx dependency on l2x0 cache Arnd Bergmann
2014-06-13 16:01 ` Arnd Bergmann
2014-06-16 9:41 ` Tony Lindgren [this message]
2014-06-16 9:41 ` Tony Lindgren
2014-06-13 16:01 ` [PATCH 03/14] ARM: keystone requires ARM_PATCH_PHYS_VIRT Arnd Bergmann
2014-06-13 16:23 ` Santosh Shilimkar
2014-06-18 14:39 ` Rob Herring
2014-06-18 15:16 ` Arnd Bergmann
2014-06-18 15:49 ` Nicolas Pitre
2014-06-13 16:01 ` [PATCH 04/14] bus/arm-cci: add dependency on OF && CPU_V7 Arnd Bergmann
2014-06-13 16:01 ` [PATCH 05/14] remoteproc: da8xx: don't select CMA on no-MMU Arnd Bergmann
2014-06-13 16:01 ` [PATCH 06/14] ARM: samsung: make SAMSUNG_DMADEV optional Arnd Bergmann
2014-06-13 16:01 ` [PATCH 07/14] ARM: mvebu: DT: fix OpenBlocks AX3-4 RAM size Arnd Bergmann
2014-06-13 16:01 ` [PATCH 08/14] ARM: integrator: fix section mismatch problem Arnd Bergmann
2014-06-13 16:01 ` [PATCH 09/14] ARM: Remove ARCH_HAS_CPUFREQ config option Arnd Bergmann
2014-06-13 16:01 ` [PATCH 10/14] misc: vexpress: fix error handling vexpress_syscfg_regmap_init() Arnd Bergmann
2014-06-13 16:01 ` [PATCH 11/14] ARM: exynos: cleanup kconfig option display Arnd Bergmann
2014-06-13 16:01 ` Arnd Bergmann
2014-06-13 16:01 ` [PATCH 12/14] ARM: use menuconfig for sub-arch menus Arnd Bergmann
2014-06-13 16:01 ` [PATCH 13/14] ARM: exynos: move sysram info to exynos.c Arnd Bergmann
2014-06-13 16:01 ` [PATCH 14/14] ARM: EXYNOS: Fix compilation warning Arnd Bergmann
2014-06-13 20:40 ` [PATCH 00/14] arm-soc fixes for 3.16 review Jason Cooper
2014-06-13 20:40 ` Jason Cooper
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=20140616094156.GA17845@atomide.com \
--to=tony@atomide.com \
--cc=arm@kernel.org \
--cc=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
/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.