Received: by 10.223.185.116 with SMTP id b49csp6274754wrg; Thu, 8 Mar 2018 04:55:23 -0800 (PST) X-Google-Smtp-Source: AG47ELt7llMHKIyLCD3f9h4j2PrtTD8xKMGDbG+2nsz83tgN2BP6R+0tv4XbdrquRCEp1bs1qRx8 X-Received: by 2002:a17:902:501:: with SMTP id 1-v6mr23854893plf.283.1520513723421; Thu, 08 Mar 2018 04:55:23 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1520513723; cv=none; d=google.com; s=arc-20160816; b=NYyKRRg3CVy1oAwxOL8QDfOFcZJE+C9MHr/cUvxnriyrwN06zFmXqQldOJMcw/IsNs 7zRE7iK2ZADDXbqieweLfEk8CrP1T35qfEAWBMytoLyOutcb1+XhkDPVVCyMtQRrFFPE Vdei+YTviBYHcQsr4xy+h455wrf1Anp5BVmFCNZf0ur3L7rRfM+aKD8ihkbMXUns+tN0 cYZNs69AexzcuFGne/WMBthTXrOGJZvjCS9NMX6jXnxBR2lBxEkk+8A7tf8meOZoJQY8 l8zaMQ+WUZIcp1AKemch8sH7Wt9nEF8zblO2Rf3Fa1pwHHucw3R1SvbfXc3J5oRbvR7g nohQ== 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=K8LWD1LFjMgZ+n577R1ZTzwLfgY6QW8scIwq6eXEdw8=; b=BwPpKRuolTV6RtgWJrAAZxt+hbaUq4e6XhO8xH/+ea93RCtnl/vOGv0xT/PXbLEpSr 1IxW3q51t0tE4ijVrS1qKK3E13QnzNlMSXJE2Iv1i5a+2Jw+QSQ64+rCRIxuRimRTLeG HD8DJe6gmp7AN3RZ26WrJjTwxrNUfp89v3f5jj6bmfP2LlcelFRyl9ZEPUSoO6K8AHf5 DHIfnIqL2mtsWIB/ox9RSHGGQB95Tif6lZIAreaSrLVclWN4/pa4SyUkCHOmse3jNLUV KNbccvQ92Pfmq/WRelQxuW6QllKq59JrKb5KiZLSOnKCdElwuxkx7DOsa3PrNS/2j6BX e5tQ== 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 a3si12921967pgc.455.2018.03.08.04.55.08; Thu, 08 Mar 2018 04:55:23 -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 S1755625AbeCHMyO (ORCPT + 99 others); Thu, 8 Mar 2018 07:54:14 -0500 Received: from bhuna.collabora.co.uk ([46.235.227.227]:59210 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751355AbeCHMyN (ORCPT ); Thu, 8 Mar 2018 07:54:13 -0500 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: sre) with ESMTPSA id A12A627640C Date: Thu, 8 Mar 2018 13:54:08 +0100 From: Sebastian Reichel To: Lee Jones , Rob Herring , Mark Brown Cc: Liam Girdwood , 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, kernel@collabora.com Subject: Re: [PATCHv5 3/5] mfd: motorola-cpcap: Add audio-codec support Message-ID: <20180308125408.45jyq2poz7najqze@earth.universe> References: <20180223200254.25685-1-sebastian.reichel@collabora.co.uk> <20180223200254.25685-4-sebastian.reichel@collabora.co.uk> <20180307163211.rytfli5tb47yhtug@dell> <20180308094652.qg4atjw5c3hayaz3@earth.universe> <20180308095315.mpcmx2ob6yhsnrm6@dell> <20180308102757.jyi7uo566n6nuct5@earth.universe> <20180308104831.fflq2arj5dxgntia@dell> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="6dfljmtajcc6f3jx" Content-Disposition: inline In-Reply-To: <20180308104831.fflq2arj5dxgntia@dell> User-Agent: NeoMutt/20171215 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --6dfljmtajcc6f3jx Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Thu, Mar 08, 2018 at 10:48:31AM +0000, Lee Jones wrote: > On Thu, 08 Mar 2018, Sebastian Reichel wrote: > > On Thu, Mar 08, 2018 at 09:53:15AM +0000, Lee Jones wrote: > > > On Thu, 08 Mar 2018, Sebastian Reichel wrote: > > > > On Wed, Mar 07, 2018 at 04:32:11PM +0000, Lee Jones wrote: > > > > > On Fri, 23 Feb 2018, Sebastian Reichel wrote: > > > > > > +static const struct mfd_cell cpcap_mfd_devices[] =3D { > > > >=20 > > > > [...] > > > >=20 > > > > > > + }, { > > > > > > + .name =3D "cpcap-led", > > > > > > + .id =3D 4, > > > > > > + .of_compatible =3D "motorola,cpcap-led-cp", > > > > > > + }, { > > > > > > + .name =3D "cpcap-codec", > > > > > > + } > > > > > > +}; > > > > >=20 > > > > > With none of the entries containing platform_data /me wonders why= you > > > > > can't still use devm_of_platform_populate()? > > > >=20 > > > > Because devm_of_platform_populate works with compatible properties = and > > > > cpcap-codec does not have one after I removed it for Mark. > > >=20 > > > Sorry, I missed that conversation. Why was it removed? > >=20 > > I had it in PATCHv1-PATCHv4. It was removed, since Mark didn't want > > to have it in the DT ABI. >=20 > Right, but why? Is it not a hardware device? I think converting from > devm_of_platform_populate() for one sub-device is a bit drastic. This must be answered by Mark. Personally I think it makes more sense to have the compatible, since all other cpcap sub-devices have them and it should be consistent IMHO. I changed it to avoid bikeshedding. The previous discussion was here: https://patchwork.kernel.org/patch/102200= 35/ -- Sebastian --6dfljmtajcc6f3jx Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAlqhMmwACgkQ2O7X88g7 +prfYg/9FYwcfRW/eU9T8Df/aIaWZ58mBd2CIwpxUSksodKBaSKWJfWIr+p9/Mvp kra3a4NVF+DdE7rAn9IyVp5jgbwkf0vdhvzekakHVpT0jhxaZr+bWCa0q96B2FWR P3jGB7J1Ic0jnGP0Iwb2yZdub++oTVEBf6tDELzWLBzkVJctQJwGdrlReIMxHYsN Rw5LOLd9OiWuWWYNpL5Qu4r7SYWx7flkMC8oD5DlEQyUakvCc8Ls+eciNdrenwnw UQTYtkhmorYuWvd+C5sx3OaZmBdUAnuAMBKSjvo9u4pnz9NO+FMrm4MHyLiM3O0m MFlA6EdeKZcavff/46HCZvdAMYJqBFSqDRTTqYekPU4mN3G+wHRnn4i39DgIlFI9 +eX7DH8yq/ZALV3QTcyXNADExyAuJqsuBvlCxLgDnVgP78+Ne/3/t3DS5156VJ0O /djJRUreDR9W2Bd9KvYB1/c5/aqgqlqnE95GNhZ7m1gYKSzzLvcpLDxX3bRVuPgh 5OjzNvHY5Wh1f0IXZYJwX3G6eMmDED0v2iUUpqeCvr+CgLqpHwz35SoVRa1ZUl7L t4Th3NxxSuTTSJT4xH3wcJ1uFwZ9GRiOEGJ5uxa7YEuCBlZo2ZjTwXO+8zgu329x tSUmK/fdqg1YhJR5WHV0HG+Sujh5z/8q7+jRWODvT68jbCqcVTI= =Zpsm -----END PGP SIGNATURE----- --6dfljmtajcc6f3jx--