From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.manjaro.org (mail.manjaro.org [116.203.91.91]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E4EF41CDFBD; Tue, 12 Nov 2024 23:25:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=116.203.91.91 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731453933; cv=none; b=tSae7eLP5kTOOzPzi4NJ9gUOGjc8lpEr+pMhz7Ar7FE8ig0a/uVpJrOREdmMDf2sfA5tnLqOaWW2+lHlAW5l9uctaHtGBIyh9WM3vRkcpSbxSbqZsbGaVfn17Q0Ht/lX3Z1ZyNW7iw0+xYYGhj2uoxH9dJPphnEK/xf4Z65wcZc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1731453933; c=relaxed/simple; bh=u5+sBODmjLDV3d7dxSv9XLwRzvuWtgWa8Hyh/CmRwnE=; h=MIME-Version:Date:From:To:Cc:Subject:In-Reply-To:References: Message-ID:Content-Type; b=I2row6rwT2xV72srgMxYw6xjgdglJal4Xujb+XhGqWH3JVMA2X2U69S/rGpFTkt+8f9GPkkXVMMgp/sjLWiPkz6HpyjspDgLFUH6r2aG2NvsnbBbbJc4Yc5oOw4WovM1b9I2ug3MsW9RpmCQQ84E19cqaXs/stVKgrhfb3Oalkk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org; spf=pass smtp.mailfrom=manjaro.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b=sZivjMBQ; arc=none smtp.client-ip=116.203.91.91 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=manjaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b="sZivjMBQ" Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1731453929; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jfGfhX/60wKOjIGQemt3Pw+kuswpotS/o+x+NvMi1fw=; b=sZivjMBQomCGF+mskHkPDQfwxXTIBR89ZtoMiCh4EHKlRR1+DgLwnLx+w0EpZ15f+Q5Qc5 klc3ywdOgC6vmMAUIliO1ONW3C+WYt515DFfSZ0lPNuyXsZDq/Slm9w6lnxT93JrdVQcDM eU13jNq4Va+AoUcu0klIUARwUEAmy7H93xPZZU+Lu8/0u8F4u/V7mHncCgPbgs6kQs8z4C pk0azoAb59HhToAAvFJBlt/3xb9ApT2go8i2F4h8NoOfIVGa72+ja66WCKyRosTB/V2039 Nb8D7D6d60xJ7VTPhZ1Si0yn5BNSeoDuAQgEfl2WnzJQfT8OfJZEKGUiijxgTw== Date: Wed, 13 Nov 2024 00:25:28 +0100 From: Dragan Simic To: =?UTF-8?Q?Tam=C3=A1s_Sz=C5=B1cs?= Cc: Jonas Karlman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , FUKAUMI Naoki , Chukun Pan , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] arm64: dts: rockchip: Enable UART8 on rock-3b In-Reply-To: References: <20241111181807.13211-1-tszucs@linux.com> <20241111181807.13211-4-tszucs@linux.com> <4ba81dfa-f276-4e05-b46b-92f50dbcfcc4@kwiboo.se> <9330ebb370780c001fd2aaee49aec9e8@manjaro.org> Message-ID: <0eb19e4daf2cdf3d4a04935876c3d3b0@manjaro.org> X-Sender: dsimic@manjaro.org Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org Hello Tamas, On 2024-11-12 22:04, Tamás Szűcs wrote: > On Tue, Nov 12, 2024 at 4:07 PM Dragan Simic > wrote: >> Please correct me if I'm wrong, but isn't this UART supposed to be >> used for the Bluetooth part of an SDIO WiFi + Bluetooth module, in >> form of a non-standard M.2 module that Radxa sells? > > UART8 is supposed to be used for any radio module connected to the M2E > connector. > It will typically be responsible for Bluetooth or BLE but it could be > 802.15.4 or whatever. In any case, all wanting to use it will need the > uart8 node enabled. I see, but I'm still guessing what's the actual use of enabling the UART8 when it will remain pretty much useless without the additional DT configuration, such as in the WiFi+Bluetooth DT overlay that Jonas sent a bit earlier? I think that the UART8 should be enabled together with something that actually makes use of it, which in this case unfortunately cannot be automatically detected and configured, so it belongs to a DT overlay. I'll get back to this in my next response. >> With that in mind, I see very little sense in just enabling the UART, >> without defining the entire Bluetooth interface, which AFAIK produces > > Defining a bluetooth node would hardwire idiosyncrasies of a given > radio module's Bluetooth core. Sure you could add a sleep clock, all > kind of sideband signals for wakeups, reset, power down, etc. But hey, > some will use them, some won't. I think it's undesirable and > unnecessary. You can hciattach from here and most will work just like > that. Tighter integration or anything special, module specific on top > should be handled individially, on a case-by-case basis. This is a dev > board after all. I say trick of all trades. > >> nasty looking error messages in the kernel log when there's actually >> nothing connected to the UART. > > My dmesg is clean as a whistle > root@rock-3b:~# dmesg | grep -E 'fe6c0000|ttyS0' > [ 0.344818] fe6c0000.serial: ttyS0 at MMIO 0xfe6c0000 (irq = 26, > base_baud = 1500000) is a 16550A > What kind of nasty errors do you recall? Those would be the kernel error messages produced with the Bluetooth DT configuration in place, but with no SDIO module installed.