linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* HSI: fix omap ssi driver dependency
@ 2014-06-03 15:14 Arnd Bergmann
  2014-06-03 23:18 ` Sebastian Reichel
  0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2014-06-03 15:14 UTC (permalink / raw)
  To: Sebastian Reichel
  Cc: Pavel Machek, Ivaylo Dimitrov, linux-kernel, linux-omap,
	linux-arm-kernel

The SSI protocol implementation has an incorrect dependency on the OMAP_SSI
driver, which allows SSI to be built-in while the underlying OMAP_SSI
implementation is a loadable module, causing a link error.

This changes the dependency to the simpler 'depends on OMAP_SSI' that also
ensures that SSI-protocol can only be a module if OMAP_SSI is not built-in.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: Sebastian Reichel <sre@kernel.org>
Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Cc: Pavel Machek <pavel@ucw.cz>
---
diff --git a/drivers/hsi/clients/Kconfig b/drivers/hsi/clients/Kconfig
index 71b9f9a..bc60dec 100644
--- a/drivers/hsi/clients/Kconfig
+++ b/drivers/hsi/clients/Kconfig
@@ -15,7 +15,7 @@ config NOKIA_MODEM
 
 config SSI_PROTOCOL
 	tristate "SSI protocol"
-	depends on HSI && PHONET && (OMAP_SSI=y || OMAP_SSI=m)
+	depends on HSI && PHONET && OMAP_SSI
 	help
 	If you say Y here, you will enable the SSI protocol aka McSAAB.
 


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: HSI: fix omap ssi driver dependency
  2014-06-03 15:14 HSI: fix omap ssi driver dependency Arnd Bergmann
@ 2014-06-03 23:18 ` Sebastian Reichel
  0 siblings, 0 replies; 2+ messages in thread
From: Sebastian Reichel @ 2014-06-03 23:18 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Pavel Machek, Ivaylo Dimitrov, linux-kernel, linux-omap,
	linux-arm-kernel

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

Hi Arnd,

On Tue, Jun 03, 2014 at 05:14:01PM +0200, Arnd Bergmann wrote:
> The SSI protocol implementation has an incorrect dependency on the OMAP_SSI
> driver, which allows SSI to be built-in while the underlying OMAP_SSI
> implementation is a loadable module, causing a link error.
> 
> This changes the dependency to the simpler 'depends on OMAP_SSI' that also
> ensures that SSI-protocol can only be a module if OMAP_SSI is not built-in.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Thanks, applied:

https://git.kernel.org/cgit/linux/kernel/git/sre/linux-hsi.git/commit/?h=for-next&id=518e1624f56615a88c91bc79c7b4b4d9eb2419e5

-- Sebastian

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-06-03 23:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-03 15:14 HSI: fix omap ssi driver dependency Arnd Bergmann
2014-06-03 23:18 ` Sebastian Reichel

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).