commit 1345fdf4dbba7ba0a99b5f3376ab49f35fd1db2e
author Chris Fries <cfries@google.com> Sat May 05 13:04:42 2018 -0500
committer juyuchen <juyuchen@google.com> Tue May 08 06:00:05 2018 +0800
tree 774c0d20ace0a3cb16d96f95f28d235965fe606b
parent 7780106b569a2daa71ea28b48e20ca4314b07501

audio: cirrus: Don't enable to write calibration files

Let's use a flag to determine whether write to /persist/audio at boottime or not, if the cal file is missing.

Bug: 78460200
Test: manual audio test on B1.

Change-Id: I77559e0132356db336a7f662432a28af8e268121
Signed-off-by: juyuchen <juyuchen@google.com>

hal/audio_extn/cirrus_playback.c

diff --git a/hal/audio_extn/cirrus_playback.c b/hal/audio_extn/cirrus_playback.c
index 1fd8984..21dbc10 100644
--- a/hal/audio_extn/cirrus_playback.c
+++ b/hal/audio_extn/cirrus_playback.c
@@ -282,11 +282,13 @@
         if (ret < 0)
             goto exit;
 
+#ifdef ENABLED_CIRRUS_WRITE_CAL_FILE
         cal_file = fopen(CRUS_CAL_FILE, "wb");
         if (cal_file == NULL) {
-            ALOGE("%s: Cannot create Cirrus SP calibration file (%s), write cal value to dsp",
+            ALOGE("%s: Cannot create Cirrus SP calibration file (%s)",
                   __func__, strerror(errno));
-            goto write_dsp;
+            ret = -EINVAL;
+            goto exit;
         }
 
         ret = fwrite(&result, sizeof(result), 1, cal_file);
@@ -303,9 +305,9 @@
 
         ALOGI("%s: Cirrus calibration file successfully written",
               __func__);
+#endif
     }
 
-write_dsp:
     header.size = sizeof(header);
     header.module_id = CRUS_MODULE_ID_TX;
     header.param_id = 0;