Hi Bjorn, Thanks a lot for your comments! > -----Original Message----- > From: Bjorn Helgaas [mailto:helgaas@xxxxxxxxxx] > Sent: 2019年3月11日 22:08 > To: Z.q. Hou <zhiqiang.hou@xxxxxxx> > Cc: linux-pci@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx; l.subrahmanya@xxxxxxxxxxxxxx; > shawnguo@xxxxxxxxxx; Leo Li <leoyang.li@xxxxxxx>; > lorenzo.pieralisi@xxxxxxx; catalin.marinas@xxxxxxx; > will.deacon@xxxxxxx; M.h. Lian <minghuan.lian@xxxxxxx>; Xiaowei Bao > <xiaowei.bao@xxxxxxx>; Mingkai Hu <mingkai.hu@xxxxxxx> > Subject: Re: [PATCHv4 10/28] PCI: mobiveil: fix the INTx process error > > On Mon, Mar 11, 2019 at 09:31:16AM +0000, Z.q. Hou wrote: > > From: Hou Zhiqiang <Zhiqiang.Hou@xxxxxxx> > > > > In the loop block, there is not code change the loop key, this patch > > updated the loop key by re-read the INTx status register. > > > > This patch also change to clear the handled INTx status. > > > > Note: Need MV to test this fix. > > > > Fixes: 9af6bcb11e12 ("PCI: mobiveil: Add Mobiveil PCIe Host Bridge IP > > driver") > > The "Fixes:" line should be all on one line, without a newline in the middle, > even if it exceeds 80 columns. That's just to make it easier for programs to > parse the logs. Will fix in v5. Thanks, Zhiqiang