From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Date: Fri, 15 Apr 2016 12:09:23 +0000 Subject: Re: [PATCH 2/2] net-ath9k_htc: Replace a variable initialisation by an assignment in ath9k_htc_set_c Message-Id: <87ega7qdjg.fsf@kamboji.qca.qualcomm.com> List-Id: References: <566ABCD9.1060404@users.sourceforge.net> <5686C390.5060600@users.sourceforge.net> <5686C47E.1040906@users.sourceforge.net> In-Reply-To: (Julian Calaby's message of "Fri, 8 Apr 2016 11:40:45 +1000") MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Julian Calaby Cc: ath9k-devel@lists.ath9k.org, linux-wireless , netdev , QCA ath9k Development , LKML , SF Markus Elfring , kernel-janitors , Julia Lawall Julian Calaby writes: > Hi Kalle, > > On Sat, Jan 2, 2016 at 5:25 AM, SF Markus Elfring > wrote: >> From: Markus Elfring >> Date: Fri, 1 Jan 2016 19:09:32 +0100 >> >> Replace an explicit initialisation for one local variable at the beginning >> by a conditional assignment. >> >> Signed-off-by: Markus Elfring > > This looks sane to me. > > Reviewed-by: Julian Calaby Before I commit I'll just change the commit title to: ath9k_htc: Replace a variable initialisation by an assignment in ath9k_htc_set_channel() -- Kalle Valo