* [PATCH v2] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig
@ 2017-10-02 4:18 Anup Patel
2017-10-05 2:44 ` Bjorn Andersson
0 siblings, 1 reply; 3+ messages in thread
From: Anup Patel @ 2017-10-02 4:18 UTC (permalink / raw)
To: Ohad Ben-Cohen, Bjorn Andersson
Cc: linux-remoteproc, linux-kernel, Anup Patel
Currently, RPMSG_VIRTIO can only be enabled if some other kconfig
option selects it. This does not allow it to be enabled for
virtualized systems where Virtio RPMSG is available over Virtio
MMIO or PCI transport.
This patch updates RPMSG_VIRTIO kconfig option so that we can
enable the VirtIO RPMSG driver via menuconfig or defconfig.
Signed-off-by: Anup Patel <anup@brainfault.org>
---
Changes since v1:
- Add depends on HAS_DMA to avoid build failures on
archs (such as um) with NO_DMA=y. For most archs,
HAS_DMA=y so having depends on HAS_DMA is fine.
drivers/rpmsg/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/rpmsg/Kconfig b/drivers/rpmsg/Kconfig
index 0fe6eac..65a9f6b 100644
--- a/drivers/rpmsg/Kconfig
+++ b/drivers/rpmsg/Kconfig
@@ -47,7 +47,8 @@ config RPMSG_QCOM_SMD
platforms.
config RPMSG_VIRTIO
- tristate
+ tristate "Virtio RPMSG bus driver"
+ depends on HAS_DMA
select RPMSG
select VIRTIO
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig
2017-10-02 4:18 [PATCH v2] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig Anup Patel
@ 2017-10-05 2:44 ` Bjorn Andersson
2017-10-05 4:00 ` Anup Patel
0 siblings, 1 reply; 3+ messages in thread
From: Bjorn Andersson @ 2017-10-05 2:44 UTC (permalink / raw)
To: Anup Patel; +Cc: Ohad Ben-Cohen, linux-remoteproc, linux-kernel
On Sun 01 Oct 21:18 PDT 2017, Anup Patel wrote:
> Currently, RPMSG_VIRTIO can only be enabled if some other kconfig
> option selects it. This does not allow it to be enabled for
> virtualized systems where Virtio RPMSG is available over Virtio
> MMIO or PCI transport.
>
> This patch updates RPMSG_VIRTIO kconfig option so that we can
> enable the VirtIO RPMSG driver via menuconfig or defconfig.
>
> Signed-off-by: Anup Patel <anup@brainfault.org>
> ---
This part looks good (and really I think this is the right thing to do).
But turning RPMSG_VIRTIO into a user selectable item makes it invalid to
"select RPMSG_VIRTIO" from drivers/remoteproc/Kconfig.
Can you please as part of this change remove those selects and as a
separate patch add CONFIG_RPMSG_VIRTIO=m to
arch/arm/configs/multi_v7_defconfig?
Thanks,
Bjorn
>
> Changes since v1:
> - Add depends on HAS_DMA to avoid build failures on
> archs (such as um) with NO_DMA=y. For most archs,
> HAS_DMA=y so having depends on HAS_DMA is fine.
>
> drivers/rpmsg/Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/rpmsg/Kconfig b/drivers/rpmsg/Kconfig
> index 0fe6eac..65a9f6b 100644
> --- a/drivers/rpmsg/Kconfig
> +++ b/drivers/rpmsg/Kconfig
> @@ -47,7 +47,8 @@ config RPMSG_QCOM_SMD
> platforms.
>
> config RPMSG_VIRTIO
> - tristate
> + tristate "Virtio RPMSG bus driver"
> + depends on HAS_DMA
> select RPMSG
> select VIRTIO
>
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig
2017-10-05 2:44 ` Bjorn Andersson
@ 2017-10-05 4:00 ` Anup Patel
0 siblings, 0 replies; 3+ messages in thread
From: Anup Patel @ 2017-10-05 4:00 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Ohad Ben-Cohen, linux-remoteproc, linux-kernel@vger.kernel.org List
On Thu, Oct 5, 2017 at 8:14 AM, Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
> On Sun 01 Oct 21:18 PDT 2017, Anup Patel wrote:
>
>> Currently, RPMSG_VIRTIO can only be enabled if some other kconfig
>> option selects it. This does not allow it to be enabled for
>> virtualized systems where Virtio RPMSG is available over Virtio
>> MMIO or PCI transport.
>>
>> This patch updates RPMSG_VIRTIO kconfig option so that we can
>> enable the VirtIO RPMSG driver via menuconfig or defconfig.
>>
>> Signed-off-by: Anup Patel <anup@brainfault.org>
>> ---
>
> This part looks good (and really I think this is the right thing to do).
> But turning RPMSG_VIRTIO into a user selectable item makes it invalid to
> "select RPMSG_VIRTIO" from drivers/remoteproc/Kconfig.
>
> Can you please as part of this change remove those selects and as a
> separate patch add CONFIG_RPMSG_VIRTIO=m to
> arch/arm/configs/multi_v7_defconfig?
Sure, I will update the patch as-per your suggestion and also
send separate patch to add CONFIG_RPMSG_VIRTIO=m in
arch/arm/configs/multi_v7_defconfig.
Regards,
Anup
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-05 4:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-02 4:18 [PATCH v2] rpmsg: Allow RPMSG_VIRTIO to be enabled via menuconfig or defconfig Anup Patel
2017-10-05 2:44 ` Bjorn Andersson
2017-10-05 4:00 ` Anup Patel
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).