From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout02.posteo.de (mout02.posteo.de [185.67.36.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E4DC02AF00 for ; Mon, 28 Apr 2025 07:26:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.67.36.66 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745825215; cv=none; b=qCBhRJtc9gzQzjzRm3NHPcTC7Crs4VU6hPmiP8n4RkGpCUIIC2ByK0PDZd0oHaJ0BbnaZyAM4wm4oclNomG2ZvH8D8j1PivUhIP0cylb1E4qfcsXfxnMJCYxopo363fHUXJhOrZ94z6h2dXAIH9gQwXCzc6fE1S1d6H3anJcafI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745825215; c=relaxed/simple; bh=nDBkBJAdWu/ACY86EyWr8Sx3/0ngI6TPK6V9GBIUBiU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=D6jBtnPwNaqBWNcNJx3lEEGODvevawDPcwQ2gQGpicTXAJAcrAAH9DXr+Tn7+Qi/xGm/OkieeA8g8jSUBqD01C50ovrNiM9k/Tz2XJbSQSvtq3oypTfP2WbdsvyAjUueF1m+U+4ZGmBG6nFg9Uhs2ux1KUXImZgkpAeKNiLnGNM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net; spf=pass smtp.mailfrom=posteo.net; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b=BOcE2IqS; arc=none smtp.client-ip=185.67.36.66 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=posteo.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=posteo.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=posteo.net header.i=@posteo.net header.b="BOcE2IqS" Received: from submission (posteo.de [185.67.36.169]) by mout02.posteo.de (Postfix) with ESMTPS id 70832240104 for ; Mon, 28 Apr 2025 09:26:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1745825205; bh=nDBkBJAdWu/ACY86EyWr8Sx3/0ngI6TPK6V9GBIUBiU=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition:Content-Transfer-Encoding:From; b=BOcE2IqSE0aj4vSs00NcAdlFm8ATmFEhh9teUyMmf0JirIaIWhgwB1vb2PndV51Xs DeOwScmMKSPH4wtviibnNMzL1gBjlrmeWAo7YWmOy9GH59FrZwhqj3/u548KECG1Fn UTrRW8YU2WWYkc+rrFqL12G1f+FJiDHnBnzAhVh20btLU4XT6l84J1O8rHrL8xsyxE NNKOO1THUDZahjCzlAucJlX4qtQdrgWTXJuPevlaS8hmOqzM8Pg3PYZOwHbPdi/+xU kzTbPBhNVrcUdj7xjTBlu1pA7oiSdaqZK8YGTapplJdQeeNKFx+6u9JjzqT50Yoke5 5tNdOFlnxVVrg== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4ZmFPZ1fG3z9rxG; Mon, 28 Apr 2025 09:26:42 +0200 (CEST) Date: Mon, 28 Apr 2025 07:26:41 +0000 From: =?utf-8?Q?J=2E_Neusch=C3=A4fer?= To: Chen-Yu Tsai Cc: j.ne@posteo.net, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jernej Skrabec , Samuel Holland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ARM: dts: allwinner: orangepi-zero: Enable audio codec Message-ID: References: <20250418-opz-audio-v1-1-4e86bb5bc734@posteo.net> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Sun, Apr 27, 2025 at 01:28:58PM +0800, Chen-Yu Tsai wrote: > On Fri, Apr 18, 2025 at 7:32 AM J. Neuschäfer via B4 Relay > wrote: > > > > From: "J. Neuschäfer" > > > > Line out playback and microphone capture work, after enabling the > > corresponding ALSA controls. Tested with the Orange Pi Zero interface > > board, which is a (mostly) passive adapter from the 13-pin header to > > standard connectors (2x USB A, 1x Audio/Video output, 1x built-in > > microphone). > > > > https://orangepi.com/index.php?route=product/product&product_id=871 > > What about the USB ports? Good point, I could test these. I have not mentioned them because they are not related to audio. > In any case, we don't enable peripherals on "headers" by default. That's > up to the end user. The description for the whole adapter board could be > an overlay that users can then apply directly. For features such as pinmuxed UARTs (already present and disabled in the dts), I fully agree, because they could as well be used as GPIO, and that's for the user to decide. For the audio pins, there's nothing else one can do with them, as far as I understand it. It is possible to use at the least the Line Out pins with very minimal setup, e.g. just connecting the pins to headphones or something else. I'll clarify the following points in a comment in the next version: - The audio signals are exposed on the 1x13 header described in https://linux-sunxi.org/Xunlong_Orange_Pi_Zero - The pins can't be used for anything else according to section 3.1. Pin Characteristics of the Allwinner H3 Datasheet. Although I would prefer to enable the audio pins by default (because they are present on the Orange Pi Zero board), I would also be ok with adding them as status = "disabled", as is the case with uart1 and uart2. Please let me know your opinion. I will test the USB ports and write a devicetree overlay at arch/arm/boot/dts/allwinner/orangepi-zero-interface-board.dtso Best regards, J. Neuschäfer