[Coverity CID :219556] Self assignment in drivers/espi/host_subs_npcx.c

[zephyrbot]

Static code scan issues found in file:

https:/zephyrproject-rtos/zephyr/tree/bd97359a5338b2542d19011b6d6aa1d8d1b9cc3f/drivers/espi/host_subs_npcx.c

Category: Incorrect expression
Function: npcx_host_init_subs_core_domain
Component: Drivers
CID: 219556

Details:

zephyr/drivers/espi/host_subs_npcx.c

Line 1001 in bd97359

inst_shm->SMC_STS = inst_shm->SMC_STS;

Please fix or provide comments in coverity using the link:

https://scan9.coverity.com/reports.htm#v32951/p12996.

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.

[MulinChao]

All bit fields of SMC_STS is RW/1C. It's common for MCU to clear all pending bits during initialization. If solving 'Self assignment ' issue is necessary, the ideas I have to replace inst_shm->SMC_STS = inst_shm->SMC_STS; are:
option 1:
inst_shm->SMC_STS = 0x7B /* bit 7 & 2 are reserved */
option 2:
uint8_t tmp = inst_shm->SMC_STS; inst_shm->SMC_STS = tmp;

Any further comment on this issue is appreciated.