On Mon, Aug 07, 2017 at 04:00:06PM +0800, Kai-Heng Feng wrote: > On Mon, Aug 7, 2017 at 3:51 PM, Mika Westerberg > <mika.westerb...@linux.intel.com> wrote: > > At this point we should find out that the ICM is already running and the > > function never calls pci2cio_write(). > > I guess you mean this code section: > > /* Check if the ICM firmware is already running */ > val = ioread32(nhi->iobase + REG_FW_STS); > if (val & REG_FW_STS_ICM_EN) > return 0;
Yes, that's correct.