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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id BC756C0219B for ; Fri, 7 Feb 2025 08:55:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:From:References:To:Subject: Date:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=WgD8+/mEXIQa6DcUPSOwh6SKC+n/sLRmZRDV/VRvl38=; b=z/XbE7c1RVSAEyyrQjwszFBIj9 1iv35Sj4BKzVFk+ebmaH0vfLG32pRNpj1p9MXmruXdk6s+YneV7RC7k1acvJ7SdbZ2WtYumr7t6fR 7ik34S2/E3dHMsIOhQjlDjl0yP+B0tmPIQiNcEBk35v6BGM8wTGGawKfReGboqTa9BvMJKxFCsDCp V8d+XXs7j/bKgkPXpQnJ/HGMyrfZT57fSy7MK0XMfSpg0/isBq0SApFH6fpd6be8oiMAOxGzVNHB4 73pXycl+OuP7zVOPkh73cuvJWAZbsq3LruUxmKhi7g1/HIbu2bRJssAUevvaXRpxTRb9GY7qTzBoV VIOQcXYg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tgK8d-00000008tgJ-1kLN; Fri, 07 Feb 2025 08:55:03 +0000 Received: from mail-am6eur05on2060a.outbound.protection.outlook.com ([2a01:111:f403:2612::60a] helo=EUR05-AM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tgK0u-00000008sGF-2Qsw; Fri, 07 Feb 2025 08:47:05 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ghdUFgSsSDLSJs0R6Fl0jXcgzaCPwXBIIFQ8suc9KMCcj2lKr1yCFocPwNpg9GOGDnYhglxcU+F3eARiorctTKQou8DTEQwuodWNdvl67GyWrHP2JhRmAaDuqUkEo1l/YK7Dvf4uPqME/R9FmmBtVXP4rpvukMcbPUbH3BHhgayj62VRUFrFYwBpU23lTIEfhNNx+hKScUibCaT+YiLcYPSDg03LCF/WAl88KhCc66RyYP3is67Dgfv/+7UfFyKaQ2R18Z7H3dH8PX1jTiqmzToRTzf/bV1pZLyUv7ZJElhIWBeazC0tLPPsaQKRRfL17mH8wr05vBAKJYW2adPbOw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=SH46aVv9/VOfH/xEb11ZtoX4HVKmA1nTMrYeld7JIEM=; b=lbyl+9sgxDiq3MVhARcSwCyRsYZmDglAeZmKtHh+rDkLzbgpduQqpLtYkrr2JCQhnh3oJXz3aRPzxLu7TRreooF/pq3NNtwCcSru6PBMuViiwB3M4t0jOgcQADW6KGuV94N6TSM+deG5rHuGaV3tuq+IPmsTmjXWd9sZERvj2xMNyxvsf/amj5CEcJNqtXZcauExi55TRTWaS7oeVhuee8+iFBW10CWGKEOwO5uj113Jgvr2MMZXZ7ui7GcYrF6JAJQjec5FYQQehX53pdy6p4ctIdYwKTVkEbTMDlAiQ5/UnG3uLhpCFFmpsZHF45TxL2caPVEh6+L2NcBGFRO13g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cherry.de; dmarc=pass action=none header.from=cherry.de; dkim=pass header.d=cherry.de; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cherry.de; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=SH46aVv9/VOfH/xEb11ZtoX4HVKmA1nTMrYeld7JIEM=; b=EX1HGSnUVI6QkRgM5BbTc5vq2pWA7uTDBWGiig4Q2N6+JxsAR+5GkvcBmSspfyLkW+BmeWDzlro27FKZaxlNmNZctLFrp37ECx6CIYAqEVfaTph16Ph6RYUqseAHKa8U5V9UkJckdzbEdBLV9O24P8AncgxVqRiBtm+c0pXINlU= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=cherry.de; Received: from AS8PR04MB8897.eurprd04.prod.outlook.com (2603:10a6:20b:42c::20) by VI1PR04MB9833.eurprd04.prod.outlook.com (2603:10a6:800:1da::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8398.25; Fri, 7 Feb 2025 08:46:58 +0000 Received: from AS8PR04MB8897.eurprd04.prod.outlook.com ([fe80::35f6:bc7d:633:369a]) by AS8PR04MB8897.eurprd04.prod.outlook.com ([fe80::35f6:bc7d:633:369a%3]) with mapi id 15.20.8398.021; Fri, 7 Feb 2025 08:46:55 +0000 Message-ID: Date: Fri, 7 Feb 2025 09:46:53 +0100 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 1/3] drm/bridge: synopsys: Add audio support for dw-hdmi-qp To: Detlev Casanova , linux-kernel@vger.kernel.org References: <20250203171925.126309-1-detlev.casanova@collabora.com> <20250203171925.126309-2-detlev.casanova@collabora.com> <13709044.uLZWGnKmhe@earth> Content-Language: en-US From: Quentin Schulz In-Reply-To: <13709044.uLZWGnKmhe@earth> X-ClientProxiedBy: VI1PR0102CA0005.eurprd01.prod.exchangelabs.com (2603:10a6:802::18) To AS8PR04MB8897.eurprd04.prod.outlook.com (2603:10a6:20b:42c::20) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8PR04MB8897:EE_|VI1PR04MB9833:EE_ X-MS-Office365-Filtering-Correlation-Id: a85854e6-d1a6-45ba-a52f-08dd4753f9dc X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|10070799003|366016|1800799024|376014|7416014; X-Microsoft-Antispam-Message-Info: =?utf-8?B?d0p2ckVjRU9ZRmRIK05UM2pWT1EwL245UFNjK1RrbDNsZWNYN3FQTVJobk1s?= =?utf-8?B?Y01UdGVBdmdVUk00b0dFd052K3hwN3NCQlhpUHRZaDkxV2pPdXlCZTJPVktF?= =?utf-8?B?Uk5sUWMweG1qcUNhb084SGxzWURqZUl4VXhHVnpYV2hGWEV0MFhwYVpDK2E3?= =?utf-8?B?Y0dOQStwZ21UbHRxUjJ1UzVmTHFDdXA4cEptL01qajdpNTdTeURyd05BUHJh?= =?utf-8?B?RVZRKzdYWUlsYnVCNmRYRENjTUhITkxwaDhZTEhVWUtqWmc0S1NhT1lrVTYz?= =?utf-8?B?Y0xmV0JEblJrbDRYbk9qUE5ibkRQdXJ0T1JyWkRHRkdxekpyUTBHVlFpa1Bj?= =?utf-8?B?Uzg3cWM0dEZXQXVKUHZUYXVUYUFaYkhGWW5CbGlmczlMcWZaVnZIVnkyeC9k?= =?utf-8?B?elM4cGJzd2E1RFBwcUFnREtYL2lsZk9NOFVEdXhSSlV2K0tkQnEweFVBMHZu?= =?utf-8?B?WFhkRXFzUThrSktnNEJ5cUlkZlpFczlyb0I0bFFJdEY2elRZSndRYkpUUEt3?= =?utf-8?B?TGVhK3BKRUZtUU9QaVRjRk8xMXdtN1BxWGowU0hIRjZodjkxVVpIQXhoQjR1?= =?utf-8?B?aTA0Lzg1bU95U0ZIRy96M1l5aXFXL0NDQkhIQ2RvUjhieXVMVkNPNVJBSW4z?= =?utf-8?B?NnRTcm9rZTlzOVlabkZicXIrZEg2NWVSbDdUeDRVTzFiQzNmM2pQNmVZVHFp?= =?utf-8?B?UGR5ZEN6Umxxc0NvazNvalBucVRIRFRNcEtXdEFtV3dicm5YbXQ4RG9HVjR5?= =?utf-8?B?dkhrczR1aWtFdFdmYlVUV3NMSTlKWXhMYUc3amRMVlUxamlQTGRaZnJPbW5S?= =?utf-8?B?cmxpQkRWMm0yWGlWd0RRa0FOdENhRkxvdlora3R2cngzOTVsWHhrb284Z2lq?= =?utf-8?B?Qm5XQmF4a2I4M0F6NC9ZZ0ZvOEFjN0x2TEhnaUdkeVQ2SkV0RXd5dW1OVDVp?= =?utf-8?B?aXo0elUrR2VpeHArUVVYaU80TnNKSWtZd08vUEtJUzlaci9tM3B3NHpkVEp4?= =?utf-8?B?dHd2cjRFMVFwUjdnWjJVZ2h0YkpSdHhIQksraWw0bDltdm9pMFRnZGdPZUkz?= =?utf-8?B?NDZQUjJlL2JkUnZWSzBXVXB2bklOMG02eTlCOGFpR0VJWERWRDV1Q1RCNnF5?= =?utf-8?B?ZVAyMzN6dmFtcm5xRmJxY0NLYklIdXNRdm93L2RmcnpSUUQ5SGYydjFxMUZ5?= =?utf-8?B?UEsyaGo5ODhsZHBvTUZOc2Y4OFdndDNQb2F2WjJvTTVLVy9tR215YnRBUXRU?= =?utf-8?B?ZDV5blVVZFdJM0U3aEQwU0MweDUwK1kwSU1lcER2amNac0t4SWN0S0djeFBG?= =?utf-8?B?NTE2eFlGY1MvRjdwRUV5V2ROMFBPZjVhOGcrRjlkOUlkcmpSVjFUeWhDSk8x?= =?utf-8?B?SmZVZWdkVXpJdk4rZUNrYkh6YmNsbzMvVDJwTStrQnhadHRPdGJyUCtCWXZB?= =?utf-8?B?a3ByMDRVTzN3cTRDbGxYNnl6b3c1MG5TcDJBNEZBeUI0bXpXeUpIM3BycDAx?= =?utf-8?B?Z0tmVlpkTzFMaW16MitXMGRWOUF2cEVHUTFXaHgvUTNBdjVCZUExVk1uaFpq?= =?utf-8?B?UWkwZFZ2T09sWUxRYnplOVRxanVhYlptUnRHUGxFZE5rb1hsdFRUVXMvZlNm?= =?utf-8?B?dThiUm1vc0xsVFRMb3NSRVhZcnV6a2tEMjNacUR3ZXB2U29NQ24xcWpFNVpP?= =?utf-8?B?ZFlFT0l1d1dPZlJpMko3TUxCb25jRUdvR3RSN3E2YU4wMythVXRjZ3dKUXZD?= =?utf-8?B?OHp3WkRsRWJKUnE3RmhqTDE1dGQwTXY5bW1BWEliV3l3aE9BN1JlQ1dreHM1?= =?utf-8?B?WGtBWHRjSnl4RmltRk1yZ0NONXByK0hJWGt6QjhycERTQUtqZEdHL3E3cE5h?= =?utf-8?Q?qo9Gc/XKQ5Kle?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS8PR04MB8897.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(10070799003)(366016)(1800799024)(376014)(7416014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?clZRT25oVVYvRmRBcG05TVdhWlpzM0VMVUVYS0VrcFFDNDUwRHhPamxxWnNN?= =?utf-8?B?aFV2ZTFjU2lUTDM5Yko0aFBHVzIwWXdhQ0h6aXR5YTUyN3c4SUw3cGhGQ0xG?= =?utf-8?B?ZDVGdktLVEtpL3Y2OUxPdUpxbmU5dllNS3B1WmFseW9obW8zV1FUWVg0OTdm?= =?utf-8?B?Y21lRjB4K1lrNXdLNGhkS3V3OWxsVWpzZ1ZhbnFNYXFWQ3VSaWxpRGVDZ2VJ?= =?utf-8?B?ZjJnNFR6K0tLaXVwVDExMHIxVlU2YXhZUjg5UVRhQ05oNWEwakc3RXVuZit4?= =?utf-8?B?RDBMOWFsamtWTGJSanFKd1ZOSEJDM0d1ZnZkU0diZkxUUFY3dG5LcnlONjdn?= =?utf-8?B?dWwrT0NORi9nWExKVVBxYUVyUVdFSWxEbnFWYitDN3NEYzkwWkJjL1VaYm4z?= =?utf-8?B?Smo2cDVJVVRqUDF6RnhDVDJrdUhjVnNRcTFiOGs0T0VvRjAxWlcyNjRNNG5J?= =?utf-8?B?V1FOdDBnakkvWmNTcUwybjQreHIyVmtabk9YbEVtVU83Uy9EYTFNYzRzVytW?= =?utf-8?B?U2NTOGNBT1NpVlpWeC9rWWE2UDZvY1FDendmZUNNRzBMVUt0Ky9xUmJKWUc5?= =?utf-8?B?dHFuWmpMOXZUeUg2Tmt5UjRjZTNOb2RMbzQvbzNJUlc0azRqMTdIVFhmNXgw?= =?utf-8?B?eFY5ZUJLY01OMVBCaW9BcS9jSm4zNmI2MnZwSTBFT1VVUGllN3hJaUM3a08y?= =?utf-8?B?Q2JvazJ5a0RCVjRoTnRJQ1JuOC9uL2NRNGJvZWtVZytLdHlTSEpmM1N3clVk?= =?utf-8?B?MzZoZG8zcVZoQytySXJ4NGxidzVmZXRNSzBrUm9QenVvNUFWQ2RyN3ZWSFl4?= =?utf-8?B?eUlLREIyZGp0TytoVEVyTWJCSjhyYktEUzJnaHdxWUZjLzNIS2RwUHlpZ01r?= =?utf-8?B?MXR1YldoNmRpVlpCbjRqVnpZK0w0TDMycWEyTENHY2F3UDUwMG1CclV6bGJU?= =?utf-8?B?V1Z5cEdaQk81dTZObXJkTll4M21MQ25kV3Noc3BiYmtCcDROcTFvTythZzZB?= =?utf-8?B?QmJSR0p2eXdualVVV0lWSHN2aHdpZnNXOC9ncGRLSnBuRkJmVlV6dmdCUUJs?= =?utf-8?B?T1BQak5uV3dvVDFuZ2M1ays3Qk40cURqbEkzRDErUFJNQ2NFOEZYRWtGVHdK?= =?utf-8?B?a2tveVZqdXNCK25GVlhRb3RhcWtOMGttTk02Q2tnMFlqVGp4RUJ1dHlEWWFj?= =?utf-8?B?Z3F2NUM2K2dkYVNLa1dPMk1IeWw0cFUzMW4zcDlyU1U5akV2Mnd3WGt0Z3Fj?= =?utf-8?B?N3J2Ryt0WjVKOUdvdjcxSkZqWDVLZE9KNnN2SFpjWlhucnQyOERkQzdXaWRB?= =?utf-8?B?T0grMjZkbVVCVE1QNTVjRnE0K0M2dzZoRklqM2dwaEdpZnpPaUpEaStRTkJv?= =?utf-8?B?NXo4RjhMdUhXVEFLbXVnUnJHc3VYaFVoNWpodm9XOXVRZjdKUk1IWWxwUTJp?= =?utf-8?B?Nk9ja0R5RnlmSmxYZ3BjYjZmRmtjZlN1S3l6TWo5YkVtL3JCZkFNNHZBSlhE?= =?utf-8?B?T2xKYlZvZENUSjBHVjl2Nkp0MHpoNmg1MlZNdHhUSTRUbG5ycFhNemt2a0tw?= =?utf-8?B?bkR6MEh6RitkeEhCTTR4VXBPazVNdU5HMHJMZ3c1c3B4ZkhoZHJDODRNdkY4?= =?utf-8?B?YzAvMFRBNVM3ZUFhTUtNVnZhd3RYNEw1YVFzUjAzT0RwMEcxQ2o4eGtHSCsv?= =?utf-8?B?M0Y0YTdDVzV3U1o5TXRURk1jOUNXdm02c1BXSWNIV0Vwa2RCb09pM2IyeDRv?= =?utf-8?B?T2VUR2kvWEJacHkyK3hLVk1ndHphQUVsSkxxczhwQnlhbzdXYWpOL3M5TzFH?= =?utf-8?B?RndzNWtoRzE3VE1yeHlMVWpQZFNOT1RoK0RHQnpuTEZYYTliaXd1S1BxR0FR?= =?utf-8?B?OXdzWFE5RWMwTFpaRGpNb2ZSNG45cUhIN1VETmhqMWtETm50REVDMU9oTGVm?= =?utf-8?B?ZlJhVWo0MTJIRWl1YWN2VjNia1hhYWNpTjAyMlcxcHd4MnF2azc3R0VWcmVr?= =?utf-8?B?WE15bGxubFZ6ckJSMlNJUEd3d25JTjE0WWxCQjJDdnBPaWJVMVl6SGttR2JN?= =?utf-8?B?VlozLzhtMVFzVURWZzF3elRSTEx1TmM1ZWdDLzFoZCs1UDl5QzdhVUpQSFdP?= =?utf-8?B?elhJWTJXNzFRUy9oTmNtbVhkMytZbU5hZ0tqZmlCWFJyMlJHeVRqamZyRExz?= =?utf-8?Q?VuCE8nzlzNY+qfQSiDJKg0M=3D?= X-OriginatorOrg: cherry.de X-MS-Exchange-CrossTenant-Network-Message-Id: a85854e6-d1a6-45ba-a52f-08dd4753f9dc X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8897.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Feb 2025 08:46:55.7852 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 5e0e1b52-21b5-4e7b-83bb-514ec460677e X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: uYfD/uaK1W2/xUcigthW8AizuJ1nyaDQJPc4IPcetSUuwBocTEqhTFs9t6wXCXZ3d95mmiwuy44g+9dlgMrvXibwvyMaKFUhdM+Jo2kdUnw= X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9833 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250207_004704_639039_C225DFE4 X-CRM114-Status: GOOD ( 27.99 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Algea Cao , Heiko Stuebner , Geert Uytterhoeven , Kever Yang , dri-devel@lists.freedesktop.org, Niklas Cassel , Laurent Pinchart , Andrzej Hajda , kernel@collabora.com, David Airlie , Dragan Simic , Simona Vetter , Rob Herring , Robert Foss , Sebastian Reichel , Jernej Skrabec , Sugar Zhang , linux-rockchip@lists.infradead.org, Chen-Yu Tsai , FUKAUMI Naoki , devicetree@vger.kernel.org, Conor Dooley , Jonas Karlman , Maarten Lankhorst , Maxime Ripard , Dmitry Baryshkov , Johan Jonker , linux-arm-kernel@lists.infradead.org, Neil Armstrong , Jianfeng Liu , Alexey Charkov , Thomas Zimmermann , Krzysztof Kozlowski Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Hi Detlev, On 2/6/25 9:40 PM, Detlev Casanova wrote: > Hi Quentin, > > On Tuesday, 4 February 2025 04:59:12 EST Quentin Schulz wrote: >> Hi Detlev, >> >> Just some drive-by comment inline. >> >> On 2/3/25 6:16 PM, Detlev Casanova wrote: [...] >>> +static void dw_hdmi_qp_set_audio_interface(struct dw_hdmi_qp *hdmi, >>> + struct hdmi_codec_daifmt > *fmt, >>> + struct hdmi_codec_params > *hparms) >>> +{ >>> + u32 conf0 = 0; >>> + >>> + /* Reset the audio data path of the AVP */ >>> + dw_hdmi_qp_write(hdmi, AVP_DATAPATH_PACKET_AUDIO_SWINIT_P, >>> GLOBAL_SWRESET_REQUEST); + >>> + /* Disable AUDS, ACR, AUDI */ >>> + dw_hdmi_qp_mod(hdmi, 0, >>> + PKTSCHED_ACR_TX_EN | PKTSCHED_AUDS_TX_EN | > PKTSCHED_AUDI_TX_EN, >>> + PKTSCHED_PKT_EN); >>> + >>> + /* Clear the audio FIFO */ >>> + dw_hdmi_qp_write(hdmi, AUDIO_FIFO_CLR_P, AUDIO_INTERFACE_CONTROL0); >>> + >>> + /* Select I2S interface as the audio source */ >>> + dw_hdmi_qp_mod(hdmi, AUD_IF_I2S, AUD_IF_SEL_MSK, >>> AUDIO_INTERFACE_CONFIG0); + >>> + /* Enable the active i2s lanes */ >>> + switch (hparms->channels) { >>> + case 7 ... 8: >>> + conf0 |= I2S_LINES_EN(3); >>> + fallthrough; >>> + case 5 ... 6: >>> + conf0 |= I2S_LINES_EN(2); >>> + fallthrough; >>> + case 3 ... 4: >>> + conf0 |= I2S_LINES_EN(1); >>> + fallthrough; >>> + default: >>> + conf0 |= I2S_LINES_EN(0); >>> + break; >>> + } >>> + >>> + dw_hdmi_qp_mod(hdmi, conf0, I2S_LINES_EN_MSK, > AUDIO_INTERFACE_CONFIG0); >>> + >>> + /* >>> + * Enable bpcuv generated internally for L-PCM, or received >>> + * from stream for NLPCM/HBR. >>> + */ >>> + switch (fmt->bit_fmt) { >>> + case SNDRV_PCM_FORMAT_IEC958_SUBFRAME_LE: >>> + conf0 = (hparms->channels == 8) ? AUD_HBR : AUD_ASP; >>> + conf0 |= I2S_BPCUV_RCV_EN; >>> + break; >>> + default: >>> + conf0 = AUD_ASP | I2S_BPCUV_RCV_DIS; >>> + break; >>> + } >>> + >>> + dw_hdmi_qp_mod(hdmi, conf0, I2S_BPCUV_RCV_MSK | AUD_FORMAT_MSK, >>> + AUDIO_INTERFACE_CONFIG0); >>> + >>> + /* Enable audio FIFO auto clear when overflow */ >>> + dw_hdmi_qp_mod(hdmi, AUD_FIFO_INIT_ON_OVF_EN, > AUD_FIFO_INIT_ON_OVF_MSK, >>> + AUDIO_INTERFACE_CONFIG0); >> >> This is all very I2S-centric while the HDMI controllers on RK3588 do >> have the ability (according to the TRM) to use S/PDIF instead of I2S. I >> assume the driver should be able to know which format to use based on >> simple-audio-card,format property? Is that correct? Then current support >> which doesn't even check for I2S would be fine and not conflict with a >> later commit which would add support for S/PDIF? (Essentially asking if >> we need another DT property for the HDMI controller node or elsewhere to >> specify which mode to run in instead of expecting it to always be I2S). > > The hdmi_codec_daifmt::fmt field already has this information, based on the > simple-audio-card,format = "i2s"; field in the device tree. > > I could add a condition in dw_hdmi_qp_audio_prepare() to fail with -EINVAL if > the devicetree specifies anything else than "i2s" for now. > All I was asking is whether we can (easily) check this property is set to i2s the day we want to implement support for S/PDIF :) If that's the case, then I don't think we necessarily need to check it right now, but a -EINVAL would be fine too, up to you. > I'm not willing to implement support for the SPDIF path for now, mainly > because there's no need for that yet (I2S works well) and the downstream > kernel doesn't implemented it, meaning it hasn't been tested a lot anyway. > That's fine, just wanted to know if a "backward-compatible" implementation for S/PDIF would be possible in the future would we want to support it :) Cheers, Quentin _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip