From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Henningsson Date: Mon, 06 Feb 2012 08:06:11 +0000 Subject: Re: [alsa-devel] [patch] ALSA: hda/realtek - cleanup a condition Message-Id: <4F2F89F3.4060301@canonical.com> List-Id: References: <20120206075218.GA14304@elgon.mountain> In-Reply-To: <20120206075218.GA14304@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Takashi Iwai , alsa-devel@alsa-project.org, kernel-janitors@vger.kernel.org On 02/06/2012 08:52 AM, Dan Carpenter wrote: > Sparse complains that "spec->multiout.dac_nids" is a pointer. > > sound/pci/hda/patch_realtek.c:2321:37: error: incompatible types for operation (>) > sound/pci/hda/patch_realtek.c:2321:37: left side has type unsigned short const [usertype] *dac_nids > sound/pci/hda/patch_realtek.c:2321:37: right side has type int > > I think the intent here was just to check if it was non-NULL, but please > double check. It could also have meant to be "if (spec->multiout.num_dacs > 0)", but either way, it seems very unlikely that the condition is false. -- David Henningsson, Canonical Ltd. http://launchpad.net/~diwic