On Thu, Sep 25, 2025 at 03:15:46PM +0300, Patrisious Haddad wrote:

On 9/25/2025 2:54 PM, Jason Gunthorpe wrote:

quoted

On Thu, Sep 25, 2025 at 02:48:33PM +0300, Tariq Toukan wrote:

quoted

+static void mlx5_iowrite64_copy(struct mlx5_wc_sq *sq, __be32 mmio_wqe[16],
+				size_t mmio_wqe_size, unsigned int offset)
+{
+#if defined(CONFIG_KERNEL_MODE_NEON) && defined(CONFIG_ARM64)

IS_ENABLED() not defined()

I just wonder why, Is there a preference in the driver from like
aesthetic/convention point of view?
Since here it technically doesnt matter - IS_ENABLED have no functional
difference from defined since these are boolean configs not *tristate*ones
(cant be loaded as module).

I think it is an aesthetic convention to avoid defined(CONFIG_*) as
the reasoning it is not tristate is a bit tricky.

Jason