Re: [PATCH] virtio: vdpa: explain the dependency of SNET_VDPA on HWMON

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 20, 2022 at 5:07 PM Alvaro Karsz <alvaro.karsz@xxxxxxxxxxxxx> wrote:
>
> Add a comment in Kconfig explaining the
> "depends on .. && (HWMON || HWMON=n)" part.
>
> This patch should be applied on top of the following patch:
>
> virtio: vdpa: new SolidNET DPU driver,
> by Alvaro Karsz alvaro.karsz@xxxxxxxxxxxxx
>
> Signed-off-by: Alvaro Karsz <alvaro.karsz@xxxxxxxxxxxxx>

Acked-by: Jason Wang <jasowang@xxxxxxxxxx>

Thanks

> ---
>  drivers/vdpa/Kconfig | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/drivers/vdpa/Kconfig b/drivers/vdpa/Kconfig
> index 79625c7cc46..7396e92b485 100644
> --- a/drivers/vdpa/Kconfig
> +++ b/drivers/vdpa/Kconfig
> @@ -89,6 +89,14 @@ config ALIBABA_ENI_VDPA
>   config SNET_VDPA
>         tristate "SolidRun's vDPA driver for SolidNET"
>         depends on PCI_MSI && PCI_IOV && (HWMON || HWMON=n)
> +
> +       # This driver MAY create a HWMON device.
> +       # Depending on (HWMON || HWMON=n) ensures that:
> +       # If HWMON=n the driver can be compiled either as a module or built-in.
> +       # If HWMON=y the driver can be compiled either as a module or built-in.
> +       # If HWMON=m the driver is forced to be compiled as a module.
> +       # By doing so, IS_ENABLED can be used instead of IS_REACHABLE
> +
>         help
>           vDPA driver for SolidNET DPU.
>           With this driver, the VirtIO dataplane can be
> --
> 2.32.0
>

_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization



[Index of Archives]     [KVM Development]     [Libvirt Development]     [Libvirt Users]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux