Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: linux-next: build warnings after merge of the
 wireless-drivers-next tree
To:     Luciano Coelho <luciano.coelho@intel.com>,
        Kalle Valo <kvalo@codeaurora.org>,
        Stephen Rothwell <sfr@canb.auug.org.au>
Cc:     Wireless <linux-wireless@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Sara Sharon <sara.sharon@intel.com>,
        Kees Cook <keescook@google.com>
References: <20181130120555.1d35eb6b@canb.auug.org.au>
 <87lg5b5glw.fsf@kamboji.qca.qualcomm.com>
 <20181130165006.60b2545f@canb.auug.org.au>
 <87h8fz57ty.fsf@kamboji.qca.qualcomm.com>
 <1773a3329bcd1bde2a22c267e4c582b0149a307e.camel@intel.com>
From:   "Gustavo A. R. Silva" <gustavo@embeddedor.com>
Message-ID: <8c9a4e83-35c9-6078-8b15-f1b7895c8923@embeddedor.com>
Date:   Mon, 3 Dec 2018 09:15:23 -0600
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.2.1
MIME-Version: 1.0
In-Reply-To: <1773a3329bcd1bde2a22c267e4c582b0149a307e.camel@intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 12/3/18 6:38 AM, Luciano Coelho wrote:
> On Fri, 2018-11-30 at 09:43 +0200, Kalle Valo wrote:
>> Stephen Rothwell <sfr@canb.auug.org.au> writes:
>>
>>> Hi Kalle,
>>>
>>> On Fri, 30 Nov 2018 06:33:47 +0200 Kalle Valo <kvalo@codeaurora.org
>>>> wrote:
>>>> I take it that -Wimplict-fallthrough is not enabled by default
>>>> yet? So
>>>> Dave and Linus won't see these warnings?
>>>
>>> Correct.
>>>
>>>> What I'm planning is to send a pull request to Dave today and fix
>>>> these
>>>> warnings in the next pull request. Does that sound good?
>>>
>>> That sounds fine.
>>
>> Great, thanks for the confirmation. I'll then send the pull request
>> today.
> 
> Just FYI, I enabled this by default in our internal tree so we will
> notice when it happens before sending the patch upstream.
> 

That's really great.

Thanks, Luca
--
Gustavo