Return-path: Received: from sabertooth02.qualcomm.com ([65.197.215.38]:13653 "EHLO sabertooth02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755380Ab3FAMRV convert rfc822-to-8bit (ORCPT ); Sat, 1 Jun 2013 08:17:21 -0400 From: Kalle Valo To: Geert Uytterhoeven CC: "John W. Linville" , , , , Subject: Re: [PATCH] ath6kl: Unify sg_sz and buf_sz in ath6kl_sdio_alloc_prep_scat_req() References: <1366804954-20491-1-git-send-email-geert@linux-m68k.org> Date: Sat, 1 Jun 2013 15:17:14 +0300 In-Reply-To: <1366804954-20491-1-git-send-email-geert@linux-m68k.org> (Geert Uytterhoeven's message of "Wed, 24 Apr 2013 14:02:34 +0200") Message-ID: <87fvx2nhyt.fsf@kamboji.qca.qualcomm.com> (sfid-20130601_141727_131034_7CC1E3C4) MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: Geert Uytterhoeven writes: > sg_sz and buf_sz are initialized and used in a mutual exclusive way. > However, some versions of gcc are not smart enough to see this: > > drivers/net/wireless/ath/ath6kl/sdio.c: In function ‘ath6kl_sdio_alloc_prep_scat_req’: > drivers/net/wireless/ath/ath6kl/sdio.c:338: warning: ‘sg_sz’ may be used uninitialized in this function > > Unify the sg_sz and buf_sz variables into a single size variable to kill > the compiler warning. > > Signed-off-by: Geert Uytterhoeven Thanks, applied. Sorry for taking so long. -- Kalle Valo