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=-6.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 74D3FC83003 for ; Wed, 29 Apr 2020 08:17:56 +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 230932083B for ; Wed, 29 Apr 2020 08:17:56 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="eFJq9g78"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="keE4AtNX"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="RMY1ao3D" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 230932083B 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+infradead-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.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=LhQlzUqmYrbrSBk39dn3WtF4w1F8z9LrjI5v1MFrJnA=; b=eFJq9g78bM4m14dbwL/PU9xdH Jw/e9B3Y2/w7Ak1trbHyf3aWwye2LCSvulZb8CZ2XIfu/IIUQhXrzYO9ZsNuqGrmfLn4vm9rdp+8i NxFXQ02OzcLI2Q6X/32JGaBixwZrQt84BEoH0nlOsR4+sxZQMX70lT+CPCVuOXaDO5W3QsW3Ddhjq F7+8qwLnAHH7ueK9fAJ0KIk6MUSxwEIQJGewxWwXK8LSVFcmrvv4WPcFW6Lhub22Mq9GGx72w84T8 ZCDtfeECQVWdt5MlPFE7BaehQwdeXSh+Kkc1ku/zxucTqCIyJeQwAfiaAZjLUHIH7DuYRyktw5fMd VxBC4MAIQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jThuf-00055Y-LT; Wed, 29 Apr 2020 08:17:49 +0000 Received: from new1-smtp.messagingengine.com ([66.111.4.221]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jThuc-00052B-AH for linux-arm-kernel@lists.infradead.org; Wed, 29 Apr 2020 08:17:48 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id DDA5458027F; Wed, 29 Apr 2020 04:17:35 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Wed, 29 Apr 2020 04:17:35 -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:in-reply-to; s=fm2; bh=9P7/0UPj4dCl+a9BcbVbxC2GgjY drzjTRocMwAg6NXE=; b=keE4AtNXGrlJztPQMSvbYzA7Oe3i6UvofEq6umdij9u Pux+ERdLZV7JTep+VMsZG5IsrOh444fgFLZl9YyywdvLKAfQyoPgjut0SQXoMkNg MHGXl9b7+6Y0BoQr2g6R/VG+4/h/XDkwBMhIKyzGp+wvh/jzyjGKEi5KHohISFaw zDyQU/FXhBE4V54O1LPLiylO4qSIM3zwtnp+CWayu2KXxTCe70//Ddo02l1ODMLd HqMbfr3mIQjQ+R6LTUEL1OCUpwU8FHNX4HBhpxj0i6iKbtSXP/daE4QTvhnsH3Nk ABybXh/saM33t73l/2NT5oiZIoMXeXOWHEW3anNfH6w== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc: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=fm2; bh=9P7/0U Pj4dCl+a9BcbVbxC2GgjYdrzjTRocMwAg6NXE=; b=RMY1ao3DHXhAEwoo64/9W/ Unc27nWmyVqTxWpZ5Kj/qOMTZBmTT1oygte96nP8AWbiNNARfWT1m89qqNQZvzOt VhHjcjw2etl5XyHGqS5Jpf7wsUt3ib3fvRdSmGeAtbw/amfNFZochFNcB2iVOgWl yRZZE4o8nQ96TECjUl9PmH6e5NTt0mXNW+OJaZlB/MR3MEWLKdE2pf9+V319E613 wZjfVXqBR9Czu78uTwO0tx1wPomQ8ZZFc7Crru3/nnd5aGWKVIQTfq8HRUAFkd5H m1vCyW9OhIzTe4tXiNmsx4efwtSjoVDDVitG8IfFPZz7tsiCx2DZYw47D0HRtR1g == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrieefgddtvdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvuffkfhggtggujgesghdtreertddtudenucfhrhhomhepofgrgihimhgv ucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucfkphepledtrd ekledrieekrdejieenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhl fhhrohhmpehmrgigihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 2F9863280063; Wed, 29 Apr 2020 04:17:32 -0400 (EDT) Date: Wed, 29 Apr 2020 10:17:29 +0200 From: Maxime Ripard To: Chen-Yu Tsai Subject: Re: [linux-sunxi] Re: Audio sound card name [was [PATCH 4/7] arm64: dts: allwinner: a64: Add HDMI audio] Message-ID: <20200429081729.qa3gqtl5sof2jhem@gilmour.lan> References: <20200426120442.11560-1-peron.clem@gmail.com> <20200426120442.11560-5-peron.clem@gmail.com> <20200428080020.35qcuylwq2ylmubu@gilmour.lan> <20200428160417.6q5oab2guaumhhwi@gilmour.lan> <031ee5d3-8a30-82ee-76db-c0e8a1073046@arm.com> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200429_011746_496876_998C272E X-CRM114-Status: GOOD ( 25.98 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree , Linux-ALSA , Takashi Iwai , Jernej Skrabec , Liam Girdwood , Rob Herring , linux-kernel , Marcus Cooper , linux-sunxi , Mark Brown , =?utf-8?B?Q2zDqW1lbnQgUMOpcm9u?= , Jaroslav Kysela , Robin Murphy , linux-arm-kernel Content-Type: multipart/mixed; boundary="===============2939203594578239520==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2939203594578239520== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="gt2wxqg5mx7iouyn" Content-Disposition: inline --gt2wxqg5mx7iouyn Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Apr 29, 2020 at 02:24:00PM +0800, Chen-Yu Tsai wrote: > On Wed, Apr 29, 2020 at 1:11 AM Robin Murphy wrote: > > > > On 2020-04-28 5:49 pm, Cl=E9ment P=E9ron wrote: > > > Hi Mark, Rob, > > > > > > On Tue, 28 Apr 2020 at 18:04, Maxime Ripard wrote: > > >> > > >> On Tue, Apr 28, 2020 at 10:54:00AM +0200, Cl=E9ment P=E9ron wrote: > > >>> Hi Maxime, > > >>> > > >>> On Tue, 28 Apr 2020 at 10:00, Maxime Ripard wro= te: > > >>>> > > >>>> On Sun, Apr 26, 2020 at 02:04:39PM +0200, Cl=E9ment P=E9ron wrote: > > >>>>> From: Marcus Cooper > > >>>>> > > >>>>> Add a simple-soundcard to link audio between HDMI and I2S. > > >>>>> > > >>>>> Signed-off-by: Jernej Skrabec > > >>>>> Signed-off-by: Marcus Cooper > > >>>>> Signed-off-by: Cl=E9ment P=E9ron > > >>>>> --- > > >>>>> arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 21 ++++++++++++= +++++++ > > >>>>> 1 file changed, 21 insertions(+) > > >>>>> > > >>>>> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch= /arm64/boot/dts/allwinner/sun50i-a64.dtsi > > >>>>> index e56e1e3d4b73..08ab6b5e72a5 100644 > > >>>>> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > > >>>>> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > > >>>>> @@ -102,6 +102,25 @@ > > >>>>> status =3D "disabled"; > > >>>>> }; > > >>>>> > > >>>>> + hdmi_sound: hdmi-sound { > > >>>>> + compatible =3D "simple-audio-card"; > > >>>>> + simple-audio-card,format =3D "i2s"; > > >>>>> + simple-audio-card,name =3D "allwinner,hdmi"; > > >>>> > > >>>> I'm not sure what the usual card name should be like though. I wou= ld assume that > > >>>> this should be something specific enough so that you're able to di= fferentiate > > >>>> between boards / SoC so that the userspace can choose a different = configuration > > >>>> based on it? > > >>> > > >>> I really don't know what we should use here, > > >>> I just have a look at other SoC: > > >>> rk3328: "HDMI" > > >>> rk3399: "hdmi-sound" > > >>> r8a774c0-cat874: "CAT874 HDMI sound" > > >>> > > >>> But maybe it's time to introduce proper name: > > >>> What about : > > >>> pat > > >>> sun50i-h6-hdmi > > >> > > >> It's pretty much what we've been using for the other sound cards we = have, so it > > >> makes sense to me. > > > > > > I have a question regarding the simple-audio-card,name. > > > In this patch, I would like to introduce a simple-audio-card for the > > > Allwinner A64 HDMI. > > > > > > What should be the preferred name for this sound card? > > > "sun50i-a64-hdmi" ? "allwinner, sun50i-a64-hdmi" ? > > > > I can at least speak for RK3328, and the reasoning there was that as the > > user looking at what `aplay -l` says, I don't give a hoot about what the > > SoC may be called, I see two cards and I want to know, with the least > > amount of uncertainty, which one will make the sound come out of the > > port that's labelled "HDMI" on the box ;) >=20 > I agree. The user really doesn't care what SoC the system uses. The only > real requirement is to be able to tell which output the card is related > to, i.e. is it onboard or an external DAC, is it analog or HDMI, etc.. Yeah, but it's exactly the point. If we also end up with "HDMI" as our card name, then the userspace has no w= ay to tell anymore if it's running from an rk3328 or an allwinner SoC, or somethi= ng else entirely. And therefore it cannot really configure anything to work ou= t of the box anymore. Maxime --gt2wxqg5mx7iouyn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXqk4GQAKCRDj7w1vZxhR xdYuAQCSWAhkZ+elqEcbpxFkCF5lqsPWcmn8x+bXetvScA7cMgEAqsSiFvNk9/P6 g63BaHgYF/RoI8GqAS53kmmdYLmT1gs= =tWdH -----END PGP SIGNATURE----- --gt2wxqg5mx7iouyn-- --===============2939203594578239520== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2939203594578239520==--