Return-path: Received: from smtprelay0056.hostedemail.com ([216.40.44.56]:40982 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752755AbcHNMXX (ORCPT ); Sun, 14 Aug 2016 08:23:23 -0400 Message-ID: <1471177398.5201.9.camel@perches.com> (sfid-20160814_142343_730421_CA61D269) Subject: Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue From: Joe Perches To: Johannes Berg , sunbing , Jes Sorensen Cc: Larry.Finger@lwfinger.net, gregkh@linuxfoundation.org, linux-wireless@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, sunbing.linux@gmail.com Date: Sun, 14 Aug 2016 05:23:18 -0700 In-Reply-To: <1471176951.5903.6.camel@sipsolutions.net> References: <1470924695-14931-1-git-send-email-sunbing@redflag-linux.com> <7FC388BC-FD87-44C6-87D3-AC041F1393DB@redflag-linux.com> <1471176423.5201.7.camel@perches.com> <1471176951.5903.6.camel@sipsolutions.net> Content-Type: text/plain; charset="ISO-8859-1" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sun, 2016-08-14 at 14:15 +0200, Johannes Berg wrote: > > > > It's a sparse defect. > > > > Try again after patching sparse with Jes' patch: > > http://marc.info/?l=linux-sparse&m=147091200720267&w=3 > Mine, not Jes's ;-) Right, sorry 'bout that. > But there's another patch going into the kernel to fix it: > > https://www.ozlabs.org/~akpm/mmotm/broken-out/byteswap-dont-use-__builtin_bswap-with-sparse.patch Thanks. Maybe this test should be sparse version checked after sparse is updated.