Fixed audio on hisi-gen2 as well, user frame depth needed to be confi…

…gured
OpenIPC · Aug 18, 2024 · 029c200 · 029c200
1 parent e0f4aff
commit 029c200
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -20,9 +20,9 @@ In spite of these design choices, Divinus boasts numerous features that cater to
 |-------------------------|:------------:|:-------------:|:-----------:|:-----------:|:------------------:|
 | CV181x[^1]              | ↻            | ↻            | ↻           | ↻           | ↻                 |
 | GM813x[^2]              | ✔️           | ✔️           | ✔️          | ✔️          | ✗                 |
-| Hi3516AV100[^3]         | ↻            | ✔️           | ✔️          | ✔️          | ✔️                |
+| Hi3516AV100[^3]         | ✔️           | ✔️           | ✔️          | ✔️          | ✔️                |
 | Hi3516CV100[^4]         | ↻            | ↻            | ↻           | ↻           | ↻                 |
-| Hi3516CV200[^5]         | ↻            | ✔️           | ✔️          | ✔️          | ✔️                |
+| Hi3516CV200[^5]         | ✔️           | ✔️           | ✔️          | ✔️          | ✔️                |
 | Hi3516CV300[^6]         | ✔️           | ✔️           | ✔️          | ✔️          | ✔️                |
 | Hi3516CV500[^7]         | ✔️           | ✔️           | ✔️          | ✔️          | ✔️                |
 | Hi3516EV200[^8]         | ✔️           | ✔️           | ✔️          | ✔️          | ✔️                |

diff --git a/src/hal/hisi/v2_aud.h b/src/hal/hisi/v2_aud.h
@@ -52,6 +52,10 @@ typedef struct {
     char isSysBound;
 } v2_aud_efrm;
 
+typedef struct {
+    unsigned int userFrmDepth;
+} v2_aud_para;
+
 typedef struct {
     void *handle;
 
@@ -61,6 +65,7 @@ typedef struct {
 
     int (*fnDisableChannel)(int device, int channel);
     int (*fnEnableChannel)(int device, int channel);
+    int (*fnSetChannelParam)(int device, int channel, v2_aud_para *param);
 
     int (*fnFreeFrame)(int device, int channel, v2_aud_frm *frame, v2_aud_efrm *encFrame);
     int (*fnGetFrame)(int device, int channel, v2_aud_frm *frame, v2_aud_efrm *encFrame, int millis);
@@ -90,6 +95,10 @@ static int v2_aud_load(v2_aud_impl *aud_lib) {
         hal_symbol_load("v2_aud", aud_lib->handle, "HI_MPI_AI_EnableChn")))
         return EXIT_FAILURE;
 
+    if (!(aud_lib->fnSetChannelParam = (int(*)(int device, int channel, v2_aud_para *param))
+        hal_symbol_load("v2_aud", aud_lib->handle, "HI_MPI_AI_SetChnParam")))
+        return EXIT_FAILURE;
+
     if (!(aud_lib->fnFreeFrame = (int(*)(int device, int channel, v2_aud_frm *frame, v2_aud_efrm *encFrame))
         hal_symbol_load("v2_aud", aud_lib->handle, "HI_MPI_AI_ReleaseFrame")))
         return EXIT_FAILURE;

diff --git a/src/hal/hisi/v2_hal.c b/src/hal/hisi/v2_hal.c
@@ -93,6 +93,12 @@ int v2_audio_init(int samplerate)
     if (ret = v2_aud.fnEnableDevice(_v2_aud_dev))
         return ret;
 
+    {
+        v2_aud_para para;
+        para.userFrmDepth = 30;
+        if (ret = v2_aud.fnSetChannelParam(_v2_aud_dev, _v2_aud_chn, &para))
+            return ret;
+    }
     if (ret = v2_aud.fnEnableChannel(_v2_aud_dev, _v2_aud_chn))
         return ret;