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 B037ACA5FFF for ; Tue, 6 Oct 2026 18:50:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc: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=tXLSbpyUG31gTT+jrU2zP+FXGyzeEv4VmzVE/LaXxjY=; b=K2BnP9fIB0wslSus/gD7gxIJr1 LNPZs2QcekW8n8VBZodYAEBj6Jg4XdLvLWzGJadRIS1dmC0rnPTfq/D5HTURBZDzfD0Rg8DnLtJCD lHhFAD7zV+3/ymWWReSEJf91bRAt/h1RTfAxuf8PiYvOgZhWxMzbbdtAJTjkGKutgC1ReqRn2weSN jOFcVzUAD/pdRMt1JenljDO+v3jGYwsNd7FGh/91YzjSM3Wz/R2tB5dI59mJn1xBFA79RKXd9sEjI m/IHJunDEO+kph9bFAb99ARE3j1ofCQqR+XriWKW5XP0F5dJXXapBz9aZNIbX9U4EVGeVKl4hB/9/ VZeFcxVw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xEAFO-00000001Hq7-0wYE; Tue, 06 Oct 2026 18:50:42 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xEAFN-00000001Hq1-1y6X for linux-arm-kernel@lists.infradead.org; Tue, 06 Oct 2026 18:50:41 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 6E1A941A6E; Tue, 6 Oct 2026 18:50:40 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 258821F0089B; Tue, 6 Oct 2026 18:50:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791312640; bh=tXLSbpyUG31gTT+jrU2zP+FXGyzeEv4VmzVE/LaXxjY=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=nqVtH0PDsADZHitSsB9NWX6JfSdnbWpCkPis74u/0eQ63jKclvt5x9BouQeAL7kPr 8ZusnzhuwxM4EYc3SKaoCz3/Qn3ZujK5jHZtdB3Smo+GYoy+pVIXJtUzesjicm7FC2 MEmscX1QmXDyp9yR/DlINPPGfK6bwhEccUyN68rL4c+Nm9IBGK4MA69p/6wLDZGTdF nQXottDid1ozi9tFP6Jli6w//JVcmnFjNThZ5LaTmMuv+kwLLrjQW+bFTIZBRLgMWK WjclFiJq1gqZGf13rs3vSt7mU51/uVUG4sBUKYEnJEsmJfKiJf7KA2EerfDEQjyXOs ZRZJ0xUdNzSdA== Date: Tue, 6 Oct 2026 13:50:39 -0500 From: Rob Herring To: Yureka Lilian Cc: Michael Reeves , Sven Peter , Janne Grunau , Neal Gompa , Jassi Brar , Krzysztof Kozlowski , Conor Dooley , Hector Martin , "Joerg Roedel (AMD)" , Will Deacon , Robin Murphy , Dmitry Torokhov , Jiri Kosina , Benjamin Tissoires , asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, iommu@lists.linux.dev, linux-input@vger.kernel.org, Julian Braha , Sasha Finkelstein Subject: Re: [PATCH v4 04/10] dt-bindings: input: apple: Add DockChannel HID transport Message-ID: <20261006185039.GE2194299-robh@kernel.org> References: <20260925-apple-mtp-keyboard-final-v4-0-304c267518f4@gmail.com> <20260925-apple-mtp-keyboard-final-v4-4-304c267518f4@gmail.com> <20261006160049.GA2406717-robh@kernel.org> <2687bc3e-b73a-415c-8c68-8249e1fe4e6b@cyberchaos.dev> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2687bc3e-b73a-415c-8c68-8249e1fe4e6b@cyberchaos.dev> X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Oct 06, 2026 at 08:09:47PM +0200, Yureka Lilian wrote: > > On 10/6/26 18:00, Rob Herring wrote: > > On Fri, Sep 25, 2026 at 10:09:31PM +1000, Michael Reeves wrote: > > [...] > > > + > > > + mboxes: > > > + items: > > > + - description: ASC mailbox used for RTKit control > > > + - description: DockChannel FIFO mailbox used for HID packets > > > + > > > + mbox-names: > > > + items: > > > + - const: asc > > > + - const: dockchannel > > > + > > > + iommus: > > > + maxItems: 1 > > > + > > > + stm: > > > + type: object > > > + description: > > > + STM interface providing the vendor, product and version identifiers > > > + and serial number shared by the HID devices. When present, the host > > > + must query this interface before registering the keyboard. > > > + additionalProperties: false > > An empty node is unusual. Why can't you just query the STM interface and > > treat it not existing or having those properties the same as no 'stm' > > node. > > The issue is there is no indicator for the stm *not* being present: We boot > the MTP coprocessor, and then receive a message when the stm is available. > In that case we know it's ready and we can proceed with obtaining the serial > numbers and registering the hid devices. But if there is no stm, we simply > do not get the stm ready message, and the hid devices are never registered > (this is the case in the original downstream Asahi dockchannel-hid). To make > the stm optional *without* taking the information from the device tree, this > would require some sort of timeout for waiting for the stm ready message. I > outlined the available options in this thread[1] and argued the empty stm > subnode makes sense, since this is describing a peripheral which may or may > not be present (and this information is useful to initialize the device > properly). > > Link[1]: https://lore.kernel.org/asahi/bbaca769-312c-4a24-9524-16cb2b4f277f@cyberchaos.dev/ Okay, but you don't need a node here. Just an 'apple,has-stm' or similar property will suffice. Rob