Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp693043rdb; Fri, 22 Dec 2023 02:03:55 -0800 (PST) X-Google-Smtp-Source: AGHT+IHNhAQUPGqxan2fwXqJW4e0dFCcWFQNMmH2/UFB7e5VWXsyvKtbIYuIkaDiBm06mUlbjavF X-Received: by 2002:a05:6808:11c9:b0:3ba:1417:434 with SMTP id p9-20020a05680811c900b003ba14170434mr1188987oiv.101.1703239435187; Fri, 22 Dec 2023 02:03:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703239435; cv=none; d=google.com; s=arc-20160816; b=RShZ04+OcLP0nN9O031kgB7pSfaS5hI0kFZIe4dYNGapmUOH1Z9CBvUdIBBPkR+GCS 9KO7liBKn+e6EJT7olnEUtANeURnT7ygSHZWnba38JFypnI/YJ5aLjPsF8eWU23JgFGS oZCMffOifz1P+klsgj67/W0hx2BhKSKi3IQYFE17BvuRfPdkZ5icH5lOhb5FjujipSdk PW43UCZCJiflgV9LNTNGtu0Dd2TthTTnDkn/wdWKPEmf+Yh4nj8xpWxJPMx7A1kJHwvO yOyHAeIfcukx2/a4Y4A9K4jzu/nR8D6EHun2kxHUhZN7wOjb8TZCxXK5ENJmtqGpNAx0 M3HQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id; bh=ylqVeXLi+AYp9IN/VpVbEX7FMaoTZQFV+mkzH0v1MNY=; fh=8nfJAFwKgnSgfQ5K/oPQRcwxDKCPHb5FHyAVv7CAWs4=; b=oXpGFOrC/eCwqMOu+gBPEMVE+XSAhkmHqoiBGsl2KE/L7qUCiL7P+uQZv3f4X5FsyV XuxoJzkRMrfnXzfETV5LhgJ+Ia674gesZOKN67LUMY4K9C9QPkNxJ9DlWKe95nG/nYhq nOQe2ucAvhge3aRYAmrszlOTBR5DQfeU/9mjohlxDDha44EdR1nLLWG5XhRvX3oFg89I giIi98XjA0wq6SxLh+UtWsJKocwaWoDbEUlZ4MyGzqa9GXXVx2yoVwlqQpVJTZThNp9v 0dsfhEqcvLb6vxhOvIoNXgM8f7shwGilJWWnWqi6wRkgPvzqEg0oBYsoHpSQatnhb7TN FAnQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-9574-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-9574-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id bz7-20020ad44c07000000b0067f82185c5csi3259732qvb.606.2023.12.22.02.03.55 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 22 Dec 2023 02:03:55 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-9574-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-9574-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-9574-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id E79A11C2235D for ; Fri, 22 Dec 2023 10:03:54 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 040EF11189; Fri, 22 Dec 2023 10:03:44 +0000 (UTC) X-Original-To: linux-kernel@vger.kernel.org Received: from ex01.ufhost.com (ex01.ufhost.com [61.152.239.75]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9E085156C3; Fri, 22 Dec 2023 10:03:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=starfivetech.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=starfivetech.com Received: from EXMBX165.cuchost.com (unknown [175.102.18.54]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "EXMBX165", Issuer "EXMBX165" (not verified)) by ex01.ufhost.com (Postfix) with ESMTP id 4C6C324E2DC; Fri, 22 Dec 2023 18:03:37 +0800 (CST) Received: from EXMBX061.cuchost.com (172.16.6.61) by EXMBX165.cuchost.com (172.16.6.75) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Fri, 22 Dec 2023 18:03:37 +0800 Received: from [192.168.125.131] (113.72.145.47) by EXMBX061.cuchost.com (172.16.6.61) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Fri, 22 Dec 2023 18:03:36 +0800 Message-ID: <11c932a8-5596-4186-9c9d-ec8ca5a6ea35@starfivetech.com> Date: Fri, 22 Dec 2023 17:55:14 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 1/2] dt-bindings: ASoC: Add Cadence I2S controller for StarFive JH8100 SoC Content-Language: en-US To: Mark Brown , Conor Dooley CC: Liam Girdwood , Claudiu Beznea , Jaroslav Kysela , "Takashi Iwai" , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Walker Chen , , , , References: <20231221033223.73201-1-xingyu.wu@starfivetech.com> <20231221033223.73201-2-xingyu.wu@starfivetech.com> <20231221-saddlebag-tricolor-d02a17d66795@spud> From: Xingyu Wu In-Reply-To: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-ClientProxiedBy: EXCAS066.cuchost.com (172.16.6.26) To EXMBX061.cuchost.com (172.16.6.61) X-YovoleRuleAgent: yovoleflag On 2023/12/21 21:58, Mark Brown wrote: > On Thu, Dec 21, 2023 at 01:53:00PM +0000, Conor Dooley wrote: >> On Thu, Dec 21, 2023 at 11:32:22AM +0800, Xingyu Wu wrote: > >> > + cdns,i2s-max-channels: >> > + description: | >> > + Number of I2S max stereo channels supported by the hardware. >> > + $ref: /schemas/types.yaml#/definitions/uint32 >> > + minimum: 1 >> > + maximum: 8 > >> Mark, is there no common property for this kind of thing? That said, >> there's one device here so the number is known at present. >> Another note, this property is not required, so it should have a >> default. > > I wouldn't expect this to be a property in the first place, as currently > presented this is specific to a single instance of the IP in a single > SoC. In general this is something that is obvious from the compatible > and doesn't need a property, it's only plausibly useful for Cadence and > Designware which is a very short list of vendors. The Cadence I2S can support 8 channels. But on the JH8100 SoC, two instances of this just provide 4 channels to use, one just provides 2 channels, and the other one can provide 8 channels. Should I use the property name of 'jh8100,i2s-max-channels' instead for some special instances on the JH8100 SoC? Best regards, Xingyu Wu