| Message ID | 1474361249-31064-7-git-send-email-matt.redfearn@imgtec.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Hello. On 9/20/2016 11:47 AM, Matt Redfearn wrote: > The MIPS remote processor driver (CONFIG_MIPS_RPROC) provides a more > standard mechanism for using one or more VPs as coprocessors running > separate firmware. > > Here we deprecate this mechanism before it is removed. > > Signed-off-by: Matt Redfearn <matt.redfearn@imgtec.com> > --- > > Changes in v2: None > > arch/mips/Kconfig | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig > index 2094cbcea0d4..071fc4585265 100644 > --- a/arch/mips/Kconfig > +++ b/arch/mips/Kconfig > @@ -2263,7 +2263,7 @@ comment "MIPS R2-to-R6 emulator is only available for UP kernels" > depends on SMP && CPU_MIPSR6 > > config MIPS_VPE_LOADER > - bool "VPE loader support." > + bool "VPE loader support. (DEPRECATED)" I think the period should be after (DEPRECATED), if at all. [...] MBR,Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-remoteproc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Sep 20, 2016 at 09:47:29AM +0100, Matt Redfearn wrote: > The MIPS remote processor driver (CONFIG_MIPS_RPROC) provides a more > standard mechanism for using one or more VPs as coprocessors running > separate firmware. > > Here we deprecate this mechanism before it is removed. The world will be a better place once this is removed. I receive the occasional minor cleanup or robopatch (coccinelle or similar) for the VPE loader but I have no indication this is actually being used by anybody, so is thee any reason why not to delete it right away? Ralf -- To unsubscribe from this list: send the line "unsubscribe linux-remoteproc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Ralf, On 20/09/16 14:19, Ralf Baechle wrote: > On Tue, Sep 20, 2016 at 09:47:29AM +0100, Matt Redfearn wrote: > >> The MIPS remote processor driver (CONFIG_MIPS_RPROC) provides a more >> standard mechanism for using one or more VPs as coprocessors running >> separate firmware. >> >> Here we deprecate this mechanism before it is removed. > The world will be a better place once this is removed. Indeed :-) > > I receive the occasional minor cleanup or robopatch (coccinelle or similar) > for the VPE loader but I have no indication this is actually being used > by anybody, so is thee any reason why not to delete it right away? I'd like to get the remote processor implementation in, then delete the VPE loader in the next kernel revision, if that's ok with you. Once it is removed, we should also be able to remove the CMP SMP implementation, and other bits of infrastructure that'll no longer be needed. Thanks, Matt > > Ralf -- To unsubscribe from this list: send the line "unsubscribe linux-remoteproc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig index 2094cbcea0d4..071fc4585265 100644 --- a/arch/mips/Kconfig +++ b/arch/mips/Kconfig @@ -2263,7 +2263,7 @@ comment "MIPS R2-to-R6 emulator is only available for UP kernels" depends on SMP && CPU_MIPSR6 config MIPS_VPE_LOADER - bool "VPE loader support." + bool "VPE loader support. (DEPRECATED)" depends on SYS_SUPPORTS_MULTITHREADING && MODULES select CPU_MIPSR2_IRQ_VI select CPU_MIPSR2_IRQ_EI @@ -2272,6 +2272,9 @@ config MIPS_VPE_LOADER Includes a loader for loading an elf relocatable object onto another VPE and running it. + Unless you have a specific need, you should use CONFIG_MIPS_RPROC + instead of this. + config MIPS_VPE_LOADER_CMP bool default "y"
The MIPS remote processor driver (CONFIG_MIPS_RPROC) provides a more standard mechanism for using one or more VPs as coprocessors running separate firmware. Here we deprecate this mechanism before it is removed. Signed-off-by: Matt Redfearn <matt.redfearn@imgtec.com> --- Changes in v2: None arch/mips/Kconfig | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)