Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935470AbcCQJyx (ORCPT ); Thu, 17 Mar 2016 05:54:53 -0400 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:39202 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932084AbcCQJyv (ORCPT ); Thu, 17 Mar 2016 05:54:51 -0400 Date: Thu, 17 Mar 2016 09:54:38 +0000 From: Mark Brown To: Harry Pan Cc: LKML , Harry Pan , lgirdwood@gmail.com, perex@perex.cz, tiwai@suse.com, alsa-devel@alsa-project.org Message-ID: <20160317095438.GD2566@sirena.org.uk> References: <1458127071-14417-1-git-send-email-harry.pan@intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="YpRP0zjcYagsg5gy" Content-Disposition: inline In-Reply-To: <1458127071-14417-1-git-send-email-harry.pan@intel.com> X-Cookie: Walk softly and carry a megawatt laser. User-Agent: Mutt/1.5.24 (2015-08-30) X-SA-Exim-Connect-IP: 2a01:348:6:8808:fab::3 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH] ASoC: dapm: Do not traverse widget hooks to snd-soc-dummy X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1223 Lines: 38 --YpRP0zjcYagsg5gy Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Mar 16, 2016 at 07:17:51PM +0800, Harry Pan wrote: > Conflicts: > sound/soc/soc-dapm.c Don't include noise like this in upstream submissions. > + if (!strcmp(cmpnt->name, "snd-soc-dummy")) > + return 0; > + This doesn't make much sense and is going to be very fragile. We should either make the dummy component look like other components or make the code cope with them as they stand, that way we don't have random undocumented special cases scattered through the code. Probably it's better to make the dummy component look like others. --YpRP0zjcYagsg5gy Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJW6n7dAAoJECTWi3JdVIfQk+4IAIABmLQIsUWP7/6db5+ky6MM jr7Fh7/d42wfjKw2TS744tugV1plO7SPZogWOz6uE1FiKuqO2UntBlCx+1BOy1Z2 zd31Ibf/HZaX4T+BvPUdRci/TVsn7wGYLXSI5mBiOnaAP3ILQqEvJd0x3KFwjadD IiaKVOqGNZ6iFAGW8Lw97s35n28ZU2kNgi/a0McoWf4hyu4fsRZvxiwcZgvLrZCg unpz+kXGRiGtzmVut84RbnaNbz3q0GLw0M0sK5/5ygfH56KSAhlrIpJGPVKiTS6z ZsuLBrbIltpQSbOchXv1HRyO4xYqMSpgyhEpVlt8uhuJBMkMC7A5oKkvqU+2yYU= =Dsg9 -----END PGP SIGNATURE----- --YpRP0zjcYagsg5gy--