Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
From:   Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To:     Daniel Baluta <daniel.baluta@oss.nxp.com>,
        "broonie@kernel.org" <broonie@kernel.org>
CC:     "daniel.baluta@gmail.com" <daniel.baluta@gmail.com>,
        "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "iuliana.prodan@nxp.com" <iuliana.prodan@nxp.com>,
        "shengjiu.wang@nxp.com" <shengjiu.wang@nxp.com>,
        Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Subject: RE: [RFC PATCH] ASoC: simple-card: Use dai_id from node description
Thread-Topic: [RFC PATCH] ASoC: simple-card: Use dai_id from node description
Thread-Index: AQHaGXSgqKqilaeU5kqopIa/H3nty7CCTJSg
Date:   Sun, 19 Nov 2023 23:28:53 +0000
Message-ID: <TYCPR01MB10914A8D3BA6BC9387EC2DC3CD4B5A@TYCPR01MB10914.jpnprd01.prod.outlook.com>
References: <20231117163900.766996-1-daniel.baluta@oss.nxp.com>
In-Reply-To: <20231117163900.766996-1-daniel.baluta@oss.nxp.com>
Accept-Language: ja-JP, en-US
Content-Language: ja-JP
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?98sfXxzSLd4uXCCqI8r6ybovU1cso6Idt8BM1S3tSYL/n1VSLBEqqIA0SnUL?=
 =?us-ascii?Q?A3r18q9V3S3po4Appjly+119hYZCc0sP3+0WpfdAQ/dF0Uphciv30POkpx4X?=
 =?us-ascii?Q?p8nsIuZPWtlVWLBNOibTxocLxeiwvBQiHdPcs0uFT7ENPesmpeZykqc7x8Xd?=
 =?us-ascii?Q?v3J4T7dQUSNicG4nF2vHX77tvaEcGWGJoaY23DefNDeURO8q0llXQOtpiXuZ?=
 =?us-ascii?Q?KudLoLAsZXl7t3XnZbrDkMZubIx/BelUl5rfBM0xfXUYVUEPfFzhzDK9K8aJ?=
 =?us-ascii?Q?/1KxrB1tjej6zy2uD41APYg8PaTh+Ag6ezWnW+YUGD7s0ZErNm18BIblHGF9?=
 =?us-ascii?Q?L5zjcamsdCh8fKl/iS1iXjSaYmtLDnTHDaxBmV/31lkDEZTMjkAAVTJeiIVl?=
 =?us-ascii?Q?SyAzNIOdrdFJOvcn4TWC36tUfdw+FSiHEcR+SU3Rz+QSWWT6hvWwfq6+pjqZ?=
 =?us-ascii?Q?d8A8eK1P2A2ebF+poF85cLEF7ReF2xdeJUUDY3DGc0BoRmN7W6XCw7Ls7nRT?=
 =?us-ascii?Q?Hhrh/GutzJOfsCF1kLsrT7Eclqt8GwkQGuC3T7JJUOv3Nq8lp3cp/cV4DtzV?=
 =?us-ascii?Q?OQJTarbdCIMYarJGdkYdbzPJTvbnpp/fzNGVlJa1ATt/Wu6NrRsQUdo1DCVZ?=
 =?us-ascii?Q?AfzQXIctNpAmpGVB1CG+pC2gDIri6ltjvCtSzgaAgz2XHxJ6Cf3C3BO3s/VA?=
 =?us-ascii?Q?8w758V/1nQfzQpoVPW/YiEL2bmNofh/HUJwmCa1xTvn6wtfcfAEYhHh/nrim?=
 =?us-ascii?Q?rx7/IP/6xDg8BnQMhGtUEWvcmPLqSz0WbBTG4ENYaZFtDxBuM9vC8mHFS8hC?=
 =?us-ascii?Q?yQLFIl8wYIDHlE4OAZIpiA64NV/OkObz9M0Q4ETVIIxzSXRICm2bnjo+8oMW?=
 =?us-ascii?Q?zRLKiz2VHKV1pWrDSVcT0dM7UhqQJUkwiVu2b/keJNK9jnF3DnABUoxjQHBh?=
 =?us-ascii?Q?5/v4CsHey2GbyHZb9sNtnyXS8YeGjx+1ikyzFBMKbdCCDSdzY5ls1Q4YdEb7?=
 =?us-ascii?Q?kqYLOS3cxoXAW6MT4VtV+UQMSd4tcnYFfUfLF1raLINqEaHWG5jTbzbi7O8r?=
 =?us-ascii?Q?dI1aYq0YJB93Hl9J2ng/SiUZPp43eU0pTEDpGwgHUmjh8LcfUQgfb9mvBWeJ?=
 =?us-ascii?Q?q4sMVJBkw+y3z5jLDkZjGgxTQxiA/4+AMRwniGzlHua3e7ypyeZGoZNJhFTV?=
 =?us-ascii?Q?CcpL+3nhlozjr616uQYFqm9qZfei7dnlFyyqcbmZjeq7jQvJ6kelxIULWjb7?=
 =?us-ascii?Q?G/HqaGR3YhMm3JG8vUHefmGYfI7M+svL8EDoqXjv4IE5RfuxGXVhvIkOIp6O?=
 =?us-ascii?Q?iKhaPA2408uyDTh7zOaDL/xeuTAZV4FCCt2wbfkP2xULYH+/5QNvk4xqWK50?=
 =?us-ascii?Q?QSKOhqUNrk1EdIqEcZpC7SiBMLwuobmp4nlqdZhj09/YS6nYNIyKem3ecCaV?=
 =?us-ascii?Q?GAGTgavOmLzKIS7pcqDcCKQqSuRMS+3ymaKiEj+35Y//FY3WHEJlmAc2Us66?=
 =?us-ascii?Q?BsNNuxZCB3rPahyelEfnGd3hL3vcX1yS98YQzrwmf2OaReOgkTed6NyNNv6A?=
 =?us-ascii?Q?Tazgw4WWChn24To4Ab/UzjU6K5p+YYk6wXo0ZgkJ48zLHuoLOhLmmwBUEi1F?=
 =?us-ascii?Q?EohyUZYPPzKUBmS8V4XMGoNjOh6nJEi+Rp5akCIrTqsIrDpafGnXyBjGlzDH?=
 =?us-ascii?Q?sJRdqw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: renesas.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB10914.jpnprd01.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ecb2f27e-ef62-41f2-60bd-08dbe9574b1a
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Nov 2023 23:28:53.3941
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 53d82571-da19-47e4-9cb4-625a166a4a2a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: FMg+MHUUkR9p57awbm/20ouhzFuGolMf/odHS/IOSvCu0RZnLxglH00VfQTFmeGI6OcbPxCIaujz7TCCcBdaIWoSRL1Ma/HVVpmBUcIbRBagIhFcmYPZPweFPq0D4MNp
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYAPR01MB6220
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Sun, 19 Nov 2023 15:45:03 -0800 (PST)

Hi Daniel

Thank you for your patch.

> We can specify DAI id using reg property. When dts
> node has only 1 DAI simple-card always assumes that DAI id is 0.
>=20
> But this is not correct in the case of SOF for example which adds DAIs
> staticaly (See definition of snd_soc_dai_driver in sound/soc/sof/imx/imx8=
m.c)
>=20
> Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
> ---
(snip)
> -	args.args_count	=3D (of_graph_get_endpoint_count(node) > 1);
> +	args.args_count	=3D (of_graph_get_endpoint_count(node) >=3D 1);

Hmm... I'm not sure how it works for existing drivers.
I'm busy this week, so I will check it next week.

Thanks