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

From: Vadim Fedorenko
Date: Wed Sep 04 2024 - 07:11:54 EST


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 <martyn.welch@xxxxxxxxxxxxx>
Reviewed-by: Vadim Fedorenko <vadim.fedorenko@xxxxxxxxx>

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

Thanks,
Vadim