From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755384AbeDZMLQ (ORCPT ); Thu, 26 Apr 2018 08:11:16 -0400 Received: from mx2.suse.de ([195.135.220.15]:59630 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753951AbeDZMLP (ORCPT ); Thu, 26 Apr 2018 08:11:15 -0400 Date: Thu, 26 Apr 2018 14:11:12 +0200 Message-ID: From: Takashi Iwai To: "Arnd Bergmann" Cc: "kbuild test robot" , , , "Arvind Yadav" , "Bhumika Goyal" , "Liam Girdwood" , , "Guneshwor Singh" , , , "Naveen M" , , "Vinod Koul" , "Mark Brown" , "Ingo Molnar" , "Baolin Wang" , "Dan Carpenter" , "Jaroslav Kysela" , "Takashi Sakamoto" , "Fabian Frederick" , "SF Markus Elfring" , "Linux Kernel Mailing List" Subject: Re: [PATCH 8/8] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control In-Reply-To: References: <201804261905.bMSzVUh7%fengguang.wu@intel.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 26 Apr 2018 13:31:17 +0200, Arnd Bergmann wrote: > > On Thu, Apr 26, 2018 at 1:25 PM, kbuild test robot wrote: > > Hi Arnd, > > > > I love your patch! Perhaps something to improve: > > > > [auto build test WARNING on v4.17-rc2] > > [cannot apply to sound/for-next asoc/for-next arm-soc/for-next next-20180426] > > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > > > url: https://github.com/0day-ci/linux/commits/Baolin-Wang/Fix-year-2038-issue-for-sound-subsystem/20180426-010145 > > reproduce: > > # apt-get install sparse > > make ARCH=x86_64 allmodconfig > > make C=1 CF=-D__CHECK_ENDIAN__ > > > > > > sparse warnings: (new ones prefixed by >>) > > > > sound/firewire/motu/motu-pcm.c:35:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:35:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:36:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:36:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:65:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:65:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:66:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:66:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:92:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:92:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:93:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:93:36: sparse: expression using sizeof(void) > >>> sound/firewire/motu/motu-pcm.c:204:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:223:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:238:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:254:50: sparse: restricted snd_pcm_state_t degrades to integer > > From what I can tell, these are all existing driver problems, no idea why we now > have one more warning than before. The only marked one is new, and others are already present in the current code. You can ignore these sparse warnings. thanks, Takashi From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH 8/8] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control Date: Thu, 26 Apr 2018 14:11:12 +0200 Message-ID: References: <201804261905.bMSzVUh7%fengguang.wu@intel.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Arnd Bergmann Cc: kbuild test robot , kbuild-all@01.org, alsa-devel@alsa-project.org, Arvind Yadav , Bhumika Goyal , Liam Girdwood , gudishax.kranthikumar@intel.com, Guneshwor Singh , hardik.t.shah@intel.com, jeeja.kp@intel.com, Naveen M , subhransu.s.prusty@intel.com, Vinod Koul , Mark Brown , Ingo Molnar , Baolin Wang , Dan Carpenter , Jaroslav Kysela , Takashi Sakamoto , Fabian Frederick , SF Markus Elfring List-Id: alsa-devel@alsa-project.org On Thu, 26 Apr 2018 13:31:17 +0200, Arnd Bergmann wrote: > > On Thu, Apr 26, 2018 at 1:25 PM, kbuild test robot wrote: > > Hi Arnd, > > > > I love your patch! Perhaps something to improve: > > > > [auto build test WARNING on v4.17-rc2] > > [cannot apply to sound/for-next asoc/for-next arm-soc/for-next next-20180426] > > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > > > url: https://github.com/0day-ci/linux/commits/Baolin-Wang/Fix-year-2038-issue-for-sound-subsystem/20180426-010145 > > reproduce: > > # apt-get install sparse > > make ARCH=x86_64 allmodconfig > > make C=1 CF=-D__CHECK_ENDIAN__ > > > > > > sparse warnings: (new ones prefixed by >>) > > > > sound/firewire/motu/motu-pcm.c:35:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:35:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:36:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:36:29: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:65:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:65:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:66:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:66:32: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:92:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:92:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:93:36: sparse: expression using sizeof(void) > > sound/firewire/motu/motu-pcm.c:93:36: sparse: expression using sizeof(void) > >>> sound/firewire/motu/motu-pcm.c:204:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:223:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:238:50: sparse: restricted snd_pcm_state_t degrades to integer > > sound/firewire/motu/motu-pcm.c:254:50: sparse: restricted snd_pcm_state_t degrades to integer > > From what I can tell, these are all existing driver problems, no idea why we now > have one more warning than before. The only marked one is new, and others are already present in the current code. You can ignore these sparse warnings. thanks, Takashi