Received: by 10.223.185.116 with SMTP id b49csp1034796wrg; Fri, 16 Feb 2018 11:12:06 -0800 (PST) X-Google-Smtp-Source: AH8x2251cgWdAEUc1cx/AN/24+geETQ+uqa/I23jLrkqVF9wleA/jEMIwYzhd2Q8H/4apjnseU32 X-Received: by 2002:a17:902:5814:: with SMTP id m20-v6mr6717857pli.389.1518808326662; Fri, 16 Feb 2018 11:12:06 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1518808326; cv=none; d=google.com; s=arc-20160816; b=WE9cmhmOwRh3x2jWMTf2k4STyyVeBI6M6xOYIiUcjbO3FRe+DyG/8EF2lEwPYLa3uf b93jElwSm36eWYNJQIlrbPpSUCPogIISok0vFrqvtMP0dr6Z35pEtne23Z9zUcVuexfb FspECaT9WZQVySsM+7CF4kqOZnW+kX7YLXpkNRWQTg3Vh1L/UFnzWYbk7gICnxBRU96g cHrW23fvUZfk6qPUFV0VhWFxMEQ2wedPhiF2x9yQUTzD8XDWjN2NN8Kj0Xyfqs+4k9Mb YVZ95JOrmwoKptj3aWykj0xPZOoKIkDqpnh+0QoxXcUE15d+yxVCIxRc2F/7ysi+hb1P qjyg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=hi0nMYzwjsy6Gx2DCG7i9TRjJq0eqTB7mSD8PVBGDcM=; b=IA5h17XszmnOwDut+p4Rxwtq+tSojmkrCzsbm0S6nYjcmcO11qENc+J87/YIutzbNL W8ZG67TD53yTfiLRTmAQwSY7IXZVJa2cyEwsn0fb0gLXrbDT8Jk+fpnMcmnkRew8BCu/ CBKKwkgfQjgyWmQ3Bx8hpmatTOqxTtD4yJ01byVKssqZXZU9KGQbBpc/8uNJhzg547c4 KlPXZUkZ+vaTLmyoUHcyFmBKCtXWniFmDomDAxb4CXzOU8Sq38FwhAzlIj3mvy7dbSL2 3lEEiSpJ5IvNkcCfs+QtVO7BBu6udquNmc4QpOkbW0Rk5XAW5/tayfBPDyDzChODAETQ EY+A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=collabora.co.uk Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e1-v6si6374497ple.154.2018.02.16.11.11.52; Fri, 16 Feb 2018 11:12:06 -0800 (PST) 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; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=collabora.co.uk Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1032783AbeBPNvT (ORCPT + 99 others); Fri, 16 Feb 2018 08:51:19 -0500 Received: from bhuna.collabora.co.uk ([46.235.227.227]:33474 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1032300AbeBPNvR (ORCPT ); Fri, 16 Feb 2018 08:51:17 -0500 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: sre) with ESMTPSA id 4B572260688 Date: Fri, 16 Feb 2018 14:51:13 +0100 From: Sebastian Reichel To: Mark Brown Cc: Liam Girdwood , Rob Herring , Tony Lindgren , Jaroslav Kysela , Takashi Iwai , alsa-devel@alsa-project.org, linux-omap@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCHv4 2/4] ASoC: codec: cpcap: new codec Message-ID: <20180216135113.akicejkodhbjpp2w@earth.universe> References: <20180214220741.28306-1-sebastian.reichel@collabora.co.uk> <20180214220741.28306-3-sebastian.reichel@collabora.co.uk> <20180216113901.GC5886@sirena.org.uk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="qnpszhyifoopthl6" Content-Disposition: inline In-Reply-To: <20180216113901.GC5886@sirena.org.uk> User-Agent: NeoMutt/20171215 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --qnpszhyifoopthl6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Mark, On Fri, Feb 16, 2018 at 11:39:01AM +0000, Mark Brown wrote: > On Wed, Feb 14, 2018 at 11:07:39PM +0100, Sebastian Reichel wrote: >=20 > > +config SND_SOC_CPCAP > > + tristate "Motorola CPCAP codec" > > + depends on MFD_CPCAP > > + default MFD_CPCAP > > + >=20 > We don't have default lines like this for other MFDs... Is there a reason not to add them? It looks very useful to me: 99% of people are not interested in the MFD. They don't enable it and are not affected at all. They do not even see extra configuration entry due to the dependency. 0.9% of people have the MFD in their system and want their hardware to 'just work'. With this line it's enough to enable the MFD and the audio subsystem, so less knowledge and configuration is required. 0.1% of people have special requirements and want the MFD and generic audio support, but no support for the MFD audio codec. There is a high chance, that they know how to configure it properly :) > > + SOC_ENUM("Hifi Left Phase Playback Switch", > > + cpcap_hifi_l_phase_inv_enum), > > + SOC_ENUM("Ext Left Phase Playback Switch", > > + cpcap_ext_l_phase_inv_enum), >=20 > If this is a Switch control it should be a simple on/off binary control. > If it's an enumeration it shouldn't have Switch at the end of the name > since that's a keyword interpreted by userspace. Either option seems > fine in this case, if it's a switch it should be "Foo Invert Switch" or > similar. Ok. -- Sebastian --qnpszhyifoopthl6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAlqG4c8ACgkQ2O7X88g7 +prytxAApQfU9evuavpOT0hbpERznCy64y2UYXCdwCBq8vG900eNN/Cxcepl5Sln eq9HNjceGrYgLb83ynHNL9I4enp7Rf7wMIAu6NLPRYzuNgoABLWWOlLoiEf+dWzE kojXaxu9uFdi50+kGy723cxXFzXGddUhEHic5L9GOKt/BawV76ZWX7DkGa5HKgO3 L2UPSv68U1GdOV1zYYFvAa5TBu+klr8OHwG1v/EQ4H3F+8R59QiFRhotfRjah8d6 +z+xRA1wLlyh+h8nbVRM5Q1V9964n79zkE67U3LsZ9j8xdeO8njW2L0Fwdkcl+G4 +7o7VZSRTWP5FftXacvFQ9LqxYLi7G84CG/1XU0+630kdfdNUpYz7rIBlhjBU52u qJJkVIHsSCeJwMoRGhkrlsIxjiYQa6JgNS9cqdEWerJv33q6wyhMcRWC79VIcYB8 Y5nSPjym8Ph+brG0B5TApUtm/2/yMTZLp0rG8s8x+7BjSAXqF19SVhVux6XzE6BG haStiWk9V+lpWaxM3gcyVcPYgcsRn2FDmjw9jRHQ5LCFUlAwEw9nB+V8qnJhDHo2 lpTas0MFwOs5yM2tUerhivD7MttWNXfdrvlE6A4SDOlFBy0u3kwGTcu3T0mjgCyt aAuSo828I/3GLQY3I6EwdS+e0LrjRls6vtBP70s+pwa9JgnA8qo= =Swer -----END PGP SIGNATURE----- --qnpszhyifoopthl6--