Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sat, 19 Feb 2022 08:33:11 +0100
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     Desmond Yan <desmond.yan@broadcom.com>
Cc:     Scott Branden <scott.branden@broadcom.com>,
        Arnd Bergmann <arnd@arndb.de>,
        bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 1/1] misc: bcm-vk: add kconfig for supporting viper
 chip
Message-ID: <YhCdNyRxGYmhx21f@kroah.com>
References: <20220218200811.17136-1-desmond.yan@broadcom.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220218200811.17136-1-desmond.yan@broadcom.com>
Precedence: bulk

On Fri, Feb 18, 2022 at 12:08:11PM -0800, Desmond Yan wrote:
> Add kconfig for enabling bcm-vk driver to support the viper chip.
> Default is off and only explicitly configured that the viper chip
> would be discovered by the driver and loaded for usage.
> 
> Tested-by: Desmond Yan <desmond.yan@broadcom.com>
> Signed-off-by: Desmond Yan <desmond.yan@broadcom.com>
> ---
>  drivers/misc/bcm-vk/Kconfig      | 10 ++++++++++
>  drivers/misc/bcm-vk/bcm_vk_dev.c |  2 ++
>  2 files changed, 12 insertions(+)
> 
> diff --git a/drivers/misc/bcm-vk/Kconfig b/drivers/misc/bcm-vk/Kconfig
> index 68a972772b99..43910a378163 100644
> --- a/drivers/misc/bcm-vk/Kconfig
> +++ b/drivers/misc/bcm-vk/Kconfig
> @@ -16,6 +16,16 @@ config BCM_VK
>  
>  	  If unsure, say N.
>  
> +config BCM_VK_VIPER
> +	bool "VK driver for VIPER"
> +	depends on BCM_VK
> +	help
> +	  Turn on to enable using the vk driver for the viper variant.
> +	  Used for development environments where viper uses VK driver
> +	  and VK firmware rather than an alternate firmware and driver.
> +
> +	  If unsure, say N.
> +
>  config BCM_VK_TTY
>  	bool "Enable tty ports on a Broadcom VK Accelerator device"
>  	depends on TTY
> diff --git a/drivers/misc/bcm-vk/bcm_vk_dev.c b/drivers/misc/bcm-vk/bcm_vk_dev.c
> index ad639ee85b2a..41dbfa4fa4a5 100644
> --- a/drivers/misc/bcm-vk/bcm_vk_dev.c
> +++ b/drivers/misc/bcm-vk/bcm_vk_dev.c
> @@ -1633,7 +1633,9 @@ static void bcm_vk_shutdown(struct pci_dev *pdev)
>  
>  static const struct pci_device_id bcm_vk_ids[] = {
>  	{ PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, PCI_DEVICE_ID_VALKYRIE), },
> +#if defined(CONFIG_BCM_VK_VIPER)
>  	{ PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, PCI_DEVICE_ID_VIPER), },
> +#endif

Why do you need a new config option just to add a new device id?  Why
would you ever NOT want this option enabled as there is no other driver
for this hardware?

thanks,

greg k-h