Skip to contentSkip to conversation

remoteproc/ste: Delete unused driver

PATCH In review Jean Delvare 4 participants 4 messages Dec 5, 2016 – Dec 7, 2016 Remote processor (remoteproc)

lore
Share & tools
lore

Jean Delvare

author
#0 lore raw

PATCH 1/1 remoteproc/ste: Delete unused driver

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.

So I think it's time to delete this driver to lower the maintenance burden.

Signed-off-by Jean Delvare <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Ohad Ben-Cohen <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Suman Anna <[email protected]>
---
 drivers/remoteproc/Kconfig           |   10 -
 drivers/remoteproc/Makefile          |    1 
 drivers/remoteproc/ste_modem_rproc.c |  342 -----------------------------------
 3 files changed, 353 deletions(-)
drivers/remoteproc/Kconfig +0 -10
@@ -31,16 +31,6 @@ config OMAP_REMOTEPROC
 	  It's safe to say n here if you're not interested in multimedia
 	  offloading or just want a bare minimum kernel.
 
-config STE_MODEM_RPROC
-	tristate "STE-Modem remoteproc support"
-	depends on HAS_DMA
-	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.
-	  If unsure say N.
-
 config WKUP_M3_RPROC
 	tristate "AMx3xx Wakeup M3 remoteproc support"
 	depends on SOC_AM33XX || SOC_AM43XX
drivers/remoteproc/Makefile +0 -1
@@ -8,7 +8,6 @@ remoteproc-y				+= remoteproc_debugfs.o
 remoteproc-y				+= remoteproc_virtio.o
 remoteproc-y				+= remoteproc_elf_loader.o
 obj-$(CONFIG_OMAP_REMOTEPROC)		+= omap_remoteproc.o
-obj-$(CONFIG_STE_MODEM_RPROC)	 	+= ste_modem_rproc.o
 obj-$(CONFIG_WKUP_M3_RPROC)		+= wkup_m3_rproc.o
 obj-$(CONFIG_DA8XX_REMOTEPROC)		+= da8xx_remoteproc.o
 obj-$(CONFIG_QCOM_MDT_LOADER)		+= qcom_mdt_loader.o
drivers/remoteproc/ste_modem_rproc.c deleted +0 -342

342 lines not on this page · raw message

Signature · 2 lines
Jean Delvare
SUSE L3 Support
Acked-by on v1 1/1
#1 lore raw
Quoted 6 lines · Jean Delvare

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.

Quoted 8 lines

So I think it's time to delete this driver to lower the maintenance burden.

Signed-off-by Jean Delvare <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Ohad Ben-Cohen <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Suman Anna <[email protected]>
Acked-by Linus Walleij <[email protected]>

I put Dmitry in CC, last chance to protest :)

Yours,
Linus Walleij

#2 lore raw
Quoted 17 lines · Linus Walleij
Quoted 6 lines · Jean Delvare

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.

Quoted 8 lines

So I think it's time to delete this driver to lower the maintenance burden.

Signed-off-by Jean Delvare <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Ohad Ben-Cohen <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Suman Anna <[email protected]>
Quoted 3 lines
Acked-by Linus Walleij <[email protected]>

I put Dmitry in CC, last chance to protest :)

+ Lee and Loic as well, would be nice to get an Ack from either one of them.

regards
Suman

Acked-by
#3 lore raw
Quoted 18 lines · Suman Anna
Quoted 17 lines · Linus Walleij
Quoted 14 lines · Jean Delvare

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.

So I think it's time to delete this driver to lower the maintenance burden.

Signed-off-by Jean Delvare <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Ohad Ben-Cohen <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Suman Anna <[email protected]>
Quoted 5 lines
Quoted 3 lines
Acked-by Linus Walleij <[email protected]>

I put Dmitry in CC, last chance to protest :)

+ Lee and Loic as well, would be nice to get an Ack from either one of them.

I confirm, no public hardware available.
Thanks for the clean up.

Acked-by Loic Pallardy <[email protected]>

regards,
Loic

Quoted 4 lines

regards
Suman