Received: by 10.223.185.116 with SMTP id b49csp1035807wrg; Fri, 16 Feb 2018 11:13:09 -0800 (PST) X-Google-Smtp-Source: AH8x225OM+cUQncca92yx5DKDDNg1CzWH5qL9Q/SQYa+LlNLSxprfDJZ2fzd+Buoyrpd9/VNgoy8 X-Received: by 10.98.227.8 with SMTP id g8mr7030192pfh.200.1518808388969; Fri, 16 Feb 2018 11:13:08 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1518808388; cv=none; d=google.com; s=arc-20160816; b=vzkAyHiJdcqND8iKh+1YO4Y7Gj90mAJPIuDyyiIv6BcgHR0mGVC74ffjRV/c7w6y/o Z7BeYC+DYw08o9X0VioBinjE7QGMizkYgEizMaIQIO2ATNAxDv6wclldDNhZTEbCT+WZ +SgEijC2K9fEXDNzRZbkHE/2HqkwONQLLbdau4UR74AYBYVWEisMVk6loG9szTaqQjUx 0rXTFOMrqDz4Jqvl4gidd4DJk2MXopaGu8ax3kr/pPYUcs1H4ZePv+EahPPXLPtAlh3Q m5md+Bw+r1Bw7H1OAGOwOJ2DSDd1X5PBY77PSYxg/91Gx844W/25bjzNTCjJvubLcSDp POjQ== 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=TNpiFrovWfxMb4pSlKeGEeu1hJPby/sVLg0I4fupXy4=; b=rwujdFqcB5Isg9eLkCuHfazsXH3wVjQeJTqWr7GIYb7N07AuksymZphqKRXTW7AiFU dbNaeeVIGI7JEuUCW0XRKekUuDpv+naV5AplF2DrhJ0SQk9JxZROKUd7TxZvksuLZrwL TS829dZ6ZZgJY7IP4XZdRMKQnfs5e0tpE/ck1fq3t1p3TETbLztdgQB84CECWuBkXP/h FdZOwj0XFjBDyZ+7nrx+iGM5EhV3A4bLuFWpIoDhdK6FqADvHCCrDaKvh9of2fngHs2v JT+Lrv+nmipNqSSfs41ghyElXNYt5TLiNLpTYcIaE6aVcbIx8PmJrGl6ep7nB8BnJ8LH XWvA== 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 o2-v6si487773pls.366.2018.02.16.11.12.45; Fri, 16 Feb 2018 11:13:08 -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 S1033958AbeBPOMn (ORCPT + 99 others); Fri, 16 Feb 2018 09:12:43 -0500 Received: from bhuna.collabora.co.uk ([46.235.227.227]:33576 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1033893AbeBPOMl (ORCPT ); Fri, 16 Feb 2018 09:12:41 -0500 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: sre) with ESMTPSA id A88582676E8 Date: Fri, 16 Feb 2018 15:12:37 +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 1/4] dt-bindings: sound: add motorola,cpcap-audio-codec Message-ID: <20180216141237.rd75sbix7bopi7zu@earth.universe> References: <20180214220741.28306-1-sebastian.reichel@collabora.co.uk> <20180214220741.28306-2-sebastian.reichel@collabora.co.uk> <20180216113008.GB5886@sirena.org.uk> <20180216132537.lkd4wzfg7uuoyx7k@earth.universe> <20180216134448.GI5886@sirena.org.uk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="uyih4se4bmnl7bld" Content-Disposition: inline In-Reply-To: <20180216134448.GI5886@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 --uyih4se4bmnl7bld Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Feb 16, 2018 at 01:44:48PM +0000, Mark Brown wrote: > On Fri, Feb 16, 2018 at 02:25:38PM +0100, Sebastian Reichel wrote: > > On Fri, Feb 16, 2018 at 11:30:08AM +0000, Mark Brown wrote: >=20 > > > Why are we adding a separate DT node with no content for this? This = is > > > a single chip, we already know that the CODEC part is there from the = DT > > > telling us that the chip is there and what we decide is part of the > > > CODEC is going to depend on what the OS running on the system is doin= g. >=20 > > While it looks empty in the DT binding file, it's actually not empty > > once some standard properties are added to support audio-graph-card. >=20 > This tells me you're missing something in the binding defining the > DAIs and... Well it is described by the following document: Documentation/devicetree/bindings/sound/audio-graph-card.txt Previous revision of the codec also worked perfectly fine with the simple card binding, which does the DAI stuff differently: Documentation/devicetree/bindings/sound/simple-card.txt A quick check of the other codecs suggested, that none of them descibes the graph based binding style. AFAIUI they could be used with it, though. So if you have a suggestion for a better binding document I can adopt this in the next version. > > A real world example looks like this: >=20 > > &cpcap { > > audio-codec { > > compatible =3D "motorola,cpcap-audio-codec"; > > #sound-dai-cells =3D <1>; >=20 > ...that still doesn't require a compatible here. I agree, that it's not required. Also the node is not required. Everything could be dumped into the main node. Many things are not required, but they make implementations easier and help in regards to DT readability and consistency. Having the compatible means, that all sub-functions _can_ be handled equally by the operating system. Not having the compatible means you _always_ need special handling for the audio codec. This basically makes the codec node different for the simple purpose of "because it is not strictly required". If we have a compatible node, other operating systems can still decide to ignore it, right? -- Sebastian --uyih4se4bmnl7bld Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAlqG5tIACgkQ2O7X88g7 +prEZg/+Ifk7bp+ccQ3mD97OqKP8xZAjxemJ0PTgQttwYPd68AUV2M3XkLfxDWRb lWMhhxP2plciikgTS4ehDUFUG37fabNf+VpKPsu753NJDgcD+JNETf+h/grZfj4b SYBcFc/oLOTb+KdyNk+nXMYsPlFQ9yDXAB9axLO+Hs52WV4w6wnFCUSiw+XYJMX3 ZzhkNzBjdnr8eL37t72eg1N5/+u33yiw8fJb2+Hz+Qa6nvRiq8ksDHFXG714RJiM Xv6Ss/iskWqRMAvr/1hVNfyCF0DB0kYeY2fb7bI2h6QiSKwAQBc41/XCHZSoh2ZX gIiED5k4tjcx3rhas3NSPaEZeCeJzGtqTOVDJAoGwwpMsUV9o5vB/bojNe3T9ggQ C3uX/I7UmjVrXX2N6gubo12q3JKM3SCD/EhH+pJ3oMj0ToTYuRkIyO2gLVjfllYU rjvlqT+u661k2jr68KNbjuUgKMlaEkHNmeQipPDDyo2xdr0nOnzyb0642nx8/OvJ wWS/gp/0epzJZWgSV4sd6P1BUeiplTz68WXtqw4nzQ6qAT+ePTEWXFqz7GyoRgQE hB6Rxu1n0Ur+KfsHnO2Z2UJJxXt3RD5pQIVD79XQqOubDb+Y4qtW+BceunaZ6RY2 Dv0f//cIWhvKN0lRZRBge2z2V6VOFT3/MEexX0LnfnuuhR8D4Qs= =r5ow -----END PGP SIGNATURE----- --uyih4se4bmnl7bld--