Re: [PATCH] misc: Kconfig: exclude mrvl-cn10k-dpi compilation for 32-bit systems

[Jeff Johnson]

On 7/17/24 09:37, Vamsi Attunuru wrote:
> Upon adding CONFIG_ARCH_THUNDER & CONFIG_COMPILE_TEST dependency,
> compilation errors arise on 32-bit ARM with writeq() & readq() calls
> which are used for accessing 64-bit values.
>
> Since DPI hardware only works with 64-bit register accesses, using
> CONFIG_64BIT dependency to skip compilation on 32-bit systems.
>
> Fixes: a5e43e2d202d ("misc: Kconfig: add a new dependency for MARVELL_CN10K_DPI")
> Signed-off-by: Vamsi Attunuru <vattunuru@xxxxxxxxxxx>
> Tested-by: Nathan Chancellor <nathan@xxxxxxxxxx>
> Tested-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx>
> ---
>   drivers/misc/Kconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index f3bb75384627..41c3d2821a78 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -588,7 +588,7 @@ config NSM
>   config MARVELL_CN10K_DPI
>   	tristate "Octeon CN10K DPI driver"
>   	depends on PCI
> -	depends on ARCH_THUNDER || COMPILE_TEST
> +	depends on ARCH_THUNDER || (COMPILE_TEST && 64BIT)
>   	help
>   	  Enables Octeon CN10K DMA packet interface (DPI) driver which
>   	  intializes DPI hardware's physical function (PF) device's


BTW it looks like you accidentally submitted Makefile.rej, can you 
submit a patch to remove it?


commit 5f67eef6dff39421215e9134f1eaae51b67a73b7
Author: Vamsi Attunuru <vattunuru@xxxxxxxxxxx>
Date:   Sat Jul 6 08:30:09 2024 -0700

    misc: mrvl-cn10k-dpi: add Octeon CN10K DPI administrative driver

...

diff --git a/drivers/misc/Makefile.rej b/drivers/misc/Makefile.rej
new file mode 100644
index 000000000000..a6aaed13f950
--- /dev/null
+++ b/drivers/misc/Makefile.rej
@@ -0,0 +1,7 @@
+--- drivers/misc/Makefile
++++ drivers/misc/Makefile
+@@ -69,3 +69,4 @@ obj-$(CONFIG_TMR_INJECT)     += xilinx_tmr_inject.o
+ obj-$(CONFIG_TPS6594_ESM)     += tps6594-esm.o
+ obj-$(CONFIG_TPS6594_PFSM)    += tps6594-pfsm.o
+ obj-$(CONFIG_NSM)             += nsm.o
++obj-$(CONFIG_MARVELL_CN10K_DPI)       += mrvl_cn10k_dpi.o