From 66976c46ba000f14fabdea576a95e38bb2bee7c2 Mon Sep 17 00:00:00 2001 From: fangpeina Date: Tue, 29 Oct 2024 17:08:28 +0800 Subject: [PATCH] drivers/input: fix compliation warn in aw86225 input/aw86225.c: In function 'aw86225_ram_work_routine': input/aw86225.c:1976:27: warning: 'rtp_file.data' may be used uninitialized [-Wmaybe-uninitialized] 1976 | struct aw86225_firmware rtp_file; | ^~~~~~~~ In function 'aw86225_ram_loaded', inlined from 'aw86225_ram_work_routine' at input/aw86225.c:1982:3: input/aw86225.c:1445:17: warning: 'rtp_file.size' may be used uninitialized [-Wmaybe-uninitialized] 1445 | for (i = 2; i < cont->size; i++) | ~~^~~~~~~~~~~~ input/aw86225.c: In function 'aw86225_ram_work_routine': input/aw86225.c:1976:27: note: 'rtp_file.size' was declared here 1976 | struct aw86225_firmware rtp_file; Signed-off-by: fangpeina --- drivers/input/aw86225.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/input/aw86225.c b/drivers/input/aw86225.c index bb0e463837..24fb868fb8 100644 --- a/drivers/input/aw86225.c +++ b/drivers/input/aw86225.c @@ -1418,7 +1418,12 @@ static void aw86225_ram_loaded(FAR struct aw86225_firmware *cont, load_cont++; #endif ierr("%s enter\n", __func__); - aw86225_request_firmware(cont, g_aw86225_ram_name); + ret = aw86225_request_firmware(cont, g_aw86225_ram_name); + if (ret < 0) + { + ierr("failed to request firmware %s\n", g_aw86225_ram_name); + return; + } if (!cont) {