drivers: sensor: fcx_mldx5 issue 74761

[jilaypandya]

This commit fixes #74761

[jeppenodgaard]

LGTM

[jeppenodgaard]

First check seems redundant now. Why keep it?

[jilaypandya]

First check seems redundant now. Why keep it?

coz one of the values in the array is zero and i think in this case, no memcpy operation shall take place.

static const uint8_t fcx_mldx5_cmds_data_len[] = {
        ********************************
	[FCX_MLDX5_CMD_RESET] = 0,
        ********************************
};

[jeppenodgaard]

First check seems redundant now. Why keep it?

coz one of the values in the array is zero and i think in this case, no memcpy operation shall take place.

static const uint8_t fcx_mldx5_cmds_data_len[] = {
        **************************
	[FCX_MLDX5_CMD_RESET] = 0,
        **************************
};

Of course. Thanks for the explanation

[MaureenHelm]

Please explain in the commit message why you are making this change. It's ok to reference a GitHub Issue number, but that in itself isn't sufficient.