From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 779DAC63798 for ; Thu, 22 Jul 2021 08:00:07 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 465AC61283 for ; Thu, 22 Jul 2021 08:00:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 465AC61283 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=MBzbDomBQCl8pKd5GBpWxFi3bhDplIk4P8KYWXbV008=; b=2F3pRBcH5J+uIq EhBL1XUBtJTiu8U95A9i8IQJ7tOpVfQwDZBwtRlV5mSk3XDGu3F+4Y/8T2iZzW3cGEdmfHEvpE/MZ /gQcz2fHx1hTLibIOje3l8gZPaPSp0vJoKIcsZywWkvv8VCNfmtYf1nR3AZWXXg9NLJ8FOL240C5i G+s58Y4R1ELBChlRvRrkNnJ4hE4w1/5CIESX0Hy43sEgwQ90WcOH+1TPK7k+/Om/EcoG3hkszI+YE NMfUxw7Vu0BMstuMPUhBdhFvbvzmDFLsOGmL4CfJsZzebLH3nz5P+2OS3SHrYZhnbdBU4iqw1Vkgf CVR0uiJPqmVXrx055TvQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6Tas-000bsx-Pp; Thu, 22 Jul 2021 07:58:11 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6Tap-000bs2-7T for linux-arm-kernel@lists.infradead.org; Thu, 22 Jul 2021 07:58:08 +0000 Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailnew.nyi.internal (Postfix) with ESMTP id 5141A5817AC; Thu, 22 Jul 2021 03:58:06 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Thu, 22 Jul 2021 03:58:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:content-transfer-encoding:in-reply-to; s=fm3; bh=k kCGDZ4mp1QSn53GXPFjfkumzgPnIst+r1z4DaNYle4=; b=B+Jms/YWtvM00vl0i OoWuB6C0ciNyzgWCcoNJUtnwcDf5VFK1VBULqZoUANTA27vz85OCuH++4UJHpcZX Yv5+u4Y44uPCcaxFVce8usaw2p7rbjTni2+NhFkdtLZackes5zi1IpLxHG109CoI RRGUDvqnvtkmIsWsI2Iv5qUICnDEPwcGXB9UOe2+cD0zzUh8+b6KMzs3Z6/UtZTx bJAig6IKmnQy8hR9fX+AjZUcvG6JzZf6h2/NrBTJi7jjryJNI0Y5EpREwOxlf3tT jkXp8tWOGgaN+QrOgxUJylKW7XaYCbOOwxHYbQjWdygmdm8BTqU/MJtSSv/ERFp0 bHy5w== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=kkCGDZ4mp1QSn53GXPFjfkumzgPnIst+r1z4DaNYl e4=; b=Qa8nY2a7JffMOPw2c0BYaZKyr8/OeYJempoB3LGboH6v1yBXBaWe+dyCb iCGFe5jZEifVHPglag1E/KYgbsqazUbXmdtkZ+7zAGG5C7vXe6zJrW7eeOgc7viW anrl3Gn1iAtjrVKVPYBoBGmaItRgcYQ5KePqIHfuN18rIiGW/+upOyOQYGqA22ov MjXnlF+bRh+8QJiUOkZxSeVsggBo084f8mAtGw8NMPHNDnxyF72BbUhGaovgldmA T3sHDxNhz/a+4NrFEYGYknsO7t3bxc1EPpZEMMViQj+JK1RRkNvMXCr8YPGVVz6r x5vBP5D3e3lBM8bdQANGGXFAiHLGg== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrfeehgdduvdegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtugfgjgesthhqredttddtvdenucfhrhhomhepofgrgihi mhgvucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucggtffrrg htthgvrhhnpefgjeettdejgffgffdvteeutdehtdehgeehueetkeefgefhtdetjeekledu gedvudenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe hmrgigihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 22 Jul 2021 03:58:04 -0400 (EDT) Date: Thu, 22 Jul 2021 09:58:03 +0200 From: Maxime Ripard To: Samuel Holland Cc: Chen-Yu Tsai , Jernej Skrabec , devicetree@vger.kernel.org, Rob Herring , Frank Rowand , linux-arm-kernel@lists.infradead.org, linux-sunxi@googlegroups.com, alsa-devel@alsa-project.org, Liam Girdwood , Mark Brown Subject: Re: [PATCH 02/54] ASoC: dt-bindings: Convert Bluetooth SCO Link binding to a schema Message-ID: <20210722075803.42ltzog3sltbeivh@gilmour> References: <20210721140424.725744-1-maxime@cerno.tech> <20210721140424.725744-3-maxime@cerno.tech> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210722_005807_363152_39C3F6B3 X-CRM114-Status: GOOD ( 21.13 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Samuel, On Thu, Jul 22, 2021 at 12:35:33AM -0500, Samuel Holland wrote: > On 7/21/21 9:03 AM, Maxime Ripard wrote: > > Bluetooth SCO Link are supported by Linux with a matching device tree > > binding. > > > > Now that we have the DT validation in place, let's convert the device > > tree bindings for that driver over to a YAML schema. > > > > The value expected for #sound-dai-cells wasn't documented though, and > > the users were inconsistent. The example didn't list it, and across the > > 4 users we have in tree: > > - 1 had a cells value of 1, but using only 0 as argument > > - 1 had a cells value of 0, > > - 2 didn't have this property at all, behaving as if it was 0, > > > > It seems like the consensus seems to be that it should be 0, so let's > > enforce it. > > The driver has two DAIs: "bt-sco-pcm" and "bt-sco-pcm-wb". If > #sound-dai-cells is 0, only the first DAI can be referenced from a > device tree. So to declare support for wideband PCM, or explicitly > declare a lack of support for it, #sound-dai-cells must be 1. Yeah, I knew there was something else to it :) I'll fix it for the next iteration. Thanks! Maxime _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel