On 01/11/17 05:19PM, Julien Thierry <[email protected]> wrote: > -----Original Message----- > From: Julien Thierry [mailto:[email protected]] > Sent: Wednesday, November 01, 2017 5:19 PM > To: Qiang Zhao <[email protected]>; [email protected] > Cc: [email protected]; [email protected] > Subject: Re: [Patch v10] QE: remove PPCisms for QE > > Hi Zhao, > > I just noticed a small nit. > > > /* wait for the QE_CR_FLG to clear */ > > - ret = spin_event_timeout((in_be32(&qe_immr->cp.cecr) & QE_CR_FLG) > == 0, > > - 100, 0); > > + ret = -EIO; > > + for (i = 0; i < 100; i++) { > > + if ((ioread32be(&qe_immr->cp.cecr) & QE_CR_FLG) == 0) { > > + ret = 0; > > + break; > > + } > > + udelay(1); > > + } > > + > > /* On timeout (e.g. failure), the expression will be false (ret == 0), > > otherwise it will be true (ret == 1). */ > > nit: > The comment here is no longer valid, on timeout ret == -EIO and on success 0. > It > should probably be removed to avoid confusion.
So thank you for you reminder, I ignored it, and will fix in the next version. BR -Qiang Zhao
