Re: [PATCH 4/4] bus: add BCM6362 on-chip WLAN SHIM bridge driver

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The `BCM6362_WLAN_SHIM` Kconfig entry selects `BCMA` and `BCMA_HOST_SOC` without inheriting their hardware dependencies (`HAS_IOMEM` and `HAS_DMA`), leading to unmet direct dependency warnings under `COMPILE_TEST`.
--

commit 1aef741e07978e7d0f4c1dd259cf899630d9e65e
Author: Alessio Ferri <alessio.ferri@mythread.it>

bus: add BCM6362 on-chip WLAN SHIM bridge driver

This commit adds a bridge driver to bring up the BCM6362 on-chip WLAN SHIM
block. It configures clocks and toggles resets, then creates a brcm,bus-axi
child platform device for the bcma driver to enumerate.

> diff --git a/drivers/bus/Kconfig b/drivers/bus/Kconfig
> index 3181d8aa32a3e..e992a34c52308 100644
> --- a/drivers/bus/Kconfig
> +++ b/drivers/bus/Kconfig
> @@ -29,6 +29,19 @@ config ARM_INTEGRATOR_LM
>  	  Say y here to enable support for the ARM Logic Module bus
>  	  found on the ARM Integrator AP (Application Platform)
>  
> +config BCM6362_WLAN_SHIM
> +	tristate "BCM6362 on-chip WLAN SHIM bridge"
> +	depends on BMIPS_GENERIC || COMPILE_TEST
> +	depends on OF
> +	select BCMA
> +	select BCMA_HOST_SOC

[Severity: Low]
Since BCMA depends on HAS_IOMEM and HAS_DMA (via BCMA_POSSIBLE), and
BCMA_HOST_SOC depends on HAS_IOMEM, will selecting them here without
inheriting those hardware dependencies cause unmet direct dependency
warnings?

When COMPILE_TEST is enabled on architectures lacking these capabilities
(such as UML), forcing these symbols on with select might trigger Kconfig
warnings or build failures.

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260529-add-bcm6362-wlan-v1-0-722242777f58@gmail.com?part=4