All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <jdelvare@suse.de>
To: linux-remoteproc@vger.kernel.org
Cc: Ohad Ben-Cohen <ohad@wizery.com>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	Dmitry Tarnyagin <dmitry.tarnyagin@lockless.no>,
	Suman Anna <s-anna@ti.com>
Subject: remoteproc/ste: Hide config option by default
Date: Thu, 1 Dec 2016 09:49:55 +0100	[thread overview]
Message-ID: <20161201094955.4c4eefe3@endymion> (raw)

Back in July 2014 I asked around what was the intended target
platform for the STE Modem remoteproc driver, so that I could add the
proper hardware dependency to its config option. The answer I got was
that there was no known publicly available hardware needing it and it
was unlikely that there ever would.

Still there were objections to deleting the driver, which I do not
really understand, but I do respect. But as there is no point in
presenting the config option by default, let's hide it unless
build-testing.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Ohad Ben-Cohen <ohad@wizery.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Dmitry Tarnyagin <dmitry.tarnyagin@lockless.no>
Cc: Suman Anna <s-anna@ti.com>
---
As suggested by Linus Walleij over 2 years ago, sorry for not
following up back then:
  https://lkml.org/lkml/2014/7/25/218

The driver could also be moved to staging/.

 drivers/remoteproc/Kconfig |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

--- linux-4.9-rc7.orig/drivers/remoteproc/Kconfig	2016-11-08 15:01:12.247924911 +0100
+++ linux-4.9-rc7/drivers/remoteproc/Kconfig	2016-12-01 09:23:57.379218183 +0100
@@ -34,11 +34,12 @@ config OMAP_REMOTEPROC
 config STE_MODEM_RPROC
 	tristate "STE-Modem remoteproc support"
 	depends on HAS_DMA
+	depends on COMPILE_TEST
 	select REMOTEPROC
 	default n
 	help
 	  Say y or m here to support STE-Modem shared memory driver.
-	  This can be either built-in or a loadable module.
+	  Note that there is no known available system needing this driver.
 	  If unsure say N.
 
 config WKUP_M3_RPROC


-- 
Jean Delvare
SUSE L3 Support

             reply	other threads:[~2016-12-01  8:49 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-01  8:49 Jean Delvare [this message]
2016-12-02 10:08 ` remoteproc/ste: Hide config option by default Linus Walleij
2016-12-02 21:56   ` Bjorn Andersson

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=20161201094955.4c4eefe3@endymion \
    --to=jdelvare@suse.de \
    --cc=bjorn.andersson@linaro.org \
    --cc=dmitry.tarnyagin@lockless.no \
    --cc=linus.walleij@linaro.org \
    --cc=linux-remoteproc@vger.kernel.org \
    --cc=ohad@wizery.com \
    --cc=s-anna@ti.com \
    /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.