From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: re: mmc: mediatek: add HS400 support Date: Fri, 11 Mar 2016 00:19:16 +0300 Message-ID: <20160310211916.GA10813@mwanda> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from userp1040.oracle.com ([156.151.31.81]:20454 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932216AbcCJVTa (ORCPT ); Thu, 10 Mar 2016 16:19:30 -0500 Content-Disposition: inline Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: chaotian.jing@mediatek.com Cc: linux-mmc@vger.kernel.org Hello Chaotian Jing, The patch 6397b7f5f405: "mmc: mediatek: add HS400 support" from Oct 27, 2015, leads to the following static checker warning: drivers/mmc/host/mtk-sd.c:1320 msdc_tune_response() error: XXX potentially using uninitialized 'cmd_err'. drivers/mmc/host/mtk-sd.c 1306 static int msdc_tune_response(struct mmc_host *mmc, u32 opcode) 1307 { 1308 struct msdc_host *host = mmc_priv(mmc); 1309 u32 rise_delay = 0, fall_delay = 0; 1310 struct msdc_delay_phase final_rise_delay, final_fall_delay; 1311 u8 final_delay, final_maxlen; 1312 int cmd_err; ^^^^^^^ 1313 int i; 1314 1315 sdr_clr_bits(host->base + MSDC_IOCON, MSDC_IOCON_RSPL); 1316 for (i = 0 ; i < PAD_DELAY_MAX; i++) { 1317 sdr_set_field(host->base + MSDC_PAD_TUNE, 1318 MSDC_PAD_TUNE_CMDRDLY, i); 1319 mmc_send_tuning(mmc, opcode, &cmd_err); ^^^^^^^^ If kmalloc fails then it's not initialized. 1320 if (!cmd_err) ^^^^^^^ 1321 rise_delay |= (1 << i); 1322 } regards, dan carpenter