commit 74fab24be8994bb5bb8d1aa8828f50e16bb38346
author Cheng-Yi Chiang <cychiang@chromium.org> Tue Aug 13 07:44:30 2019
committer Douglas Anderson <dianders@chromium.org> Mon Aug 26 16:19:55 2019
tree ec323fd84b4aae6d11a54fb8272f1fcfae8ccab4
parent de9362443dedfb962b650233ab4e4f6e40fea193

FROMLIST: ASoC: rockchip: rockchip_max98090: Set period size to 240

From stress testing of arecord, we found that period size
greater than ~900 will bring pl330 to DYING state and
can not recover within 100 iterations.
The result is that arecord will stuck and get I/O error,
and issue can not be recovered until reboot.

This issue does not happen when period size is small.
Set constraint of period size to 240 to prevent such issue.
With the constraint, there will be no issue after 2000 iterations.

We can revert this patch once the root cause is found
in rockchip's pl330 implementation.

Signed-off-by: Cheng-Yi Chiang <cychiang@chromium.org>
(am from https://patchwork.kernel.org/patch/11091403/)
(also found at https://lkml.kernel.org/r/20190813074430.191791-1-cychiang@chromium.org)

BUG=chromium:990218
TEST=run alsa_conformance_test and arecord in a loop for
     2000 iterations. The script is attached in
     crbug.com/990218#c9

Original Change-Id: Ic3e54798e4aeaa862b2e8eebcbbcef4e51ccae19
Original Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1746363
Original Reviewed-by: Sean Paul <seanpaul@chromium.org>
Original Reviewed-by: Tzung-Bi Shih <tzungbi@chromium.org>
Original Commit-Queue: Cheng-Yi Chiang <cychiang@chromium.org>
Original Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>
(cherry picked from commit 48425b33b106fad4c9e9fdd984a0683e7c679cc6)

Change-Id: I545959f99a2d99bd2f099e97c0aa6c0f4adf9a9e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1753496
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Tested-by: Douglas Anderson <dianders@chromium.org>