search for: tegra30

...t); > + mc_writel(mc, val, hr_client->ctrl); > + mc_readl(mc, hr_client->ctrl); > + > + /* poll till the flush is done */ > + if (enable) { > + do { > + udelay(10); This should probably be usleep_range(10, 20) or something. Would it be difficult to implement this for Tegra30 and Tegra114? Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20150106/9c6f4da2/attachment.sig>

...ike we don't need to lock around the polling loop here since we're merely reading a status register. We would only need to lock around accesses to the control register to make sure two processes can't step on each others's toes. > > Would it be difficult to implement this for Tegra30 and Tegra114? > No. But I have to check the detail in Tegra30 and Tegra114. And the biggest > problem is I don't have the boards to test. I can help with testing. Though that raises the question of how this can be tested. It seems like this feature is used to make sure that all outstandi...

...(enable) { > > + do { > > + udelay(10); > > This should probably be usleep_range(10, 20) or something. Maybe no. We might need some spin lock here to ensure only one flushing operation requested and no race could happen. > > Would it be difficult to implement this for Tegra30 and Tegra114? No. But I have to check the detail in Tegra30 and Tegra114. And the biggest problem is I don't have the boards to test. Thanks, Vince

Hi, This series includes some pieces of fixes to complete the GK20A power on/off sequences and add the suspend/resume support. The patches 1/11 - 4/11 are based on the linux-next-20141219. The patches 5/11 - 11/11 are based on the branch "linux-3.19" of Ben Skeggs's tree (http://cgit.freedesktop.org/~darktama/nouveau). Thanks, Vince Vince Hsu (4): (linux-next-20141219) ARM: