Subject: Re: [PATCH net-next v3] net: enetc: Replace ifdef with IS_ENABLED

From: Vadim Fedorenko <[email-masked]>
Date: Wed, 4 Sep 2024 12:11:31 +0100

On 04/09/2024 11:51, Martyn Welch wrote:
> The enetc driver uses ifdefs when checking whether
> CONFIG_FSL_ENETC_PTP_CLOCK is enabled in a number of places. This works
> if the driver is compiled in but fails if the driver is available as a
> kernel module. Replace the instances of ifdef with use of the IS_ENABLED
> macro, that will evaluate as true when this feature is built as a kernel
> module and follows the kernel's coding style.
> 
> Signed-off-by: Martyn Welch <[email-masked]>
> Reviewed-by: Vadim Fedorenko <[email-masked]>

If there will be v4, please, put yours SoB as the last tag after all
other tags.

Thanks,
Vadim