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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2010C61DA4 for ; Tue, 14 Feb 2023 20:02:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231347AbjBNUCv (ORCPT ); Tue, 14 Feb 2023 15:02:51 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56498 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231658AbjBNUCq (ORCPT ); Tue, 14 Feb 2023 15:02:46 -0500 Received: from hutie.ust.cz (unknown [IPv6:2a03:3b40:fe:f0::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8DDE52E804 for ; Tue, 14 Feb 2023 12:02:41 -0800 (PST) Content-Type: text/plain; charset=utf-8 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cutebit.org; s=mail; t=1676404954; bh=Shdv6/ZcQzIt7yhST3CHJufWnCYjpSB75fBRBOSxd+0=; h=Subject:From:In-Reply-To:Date:Cc:References:To; b=iWYGC7YUqubuGS2vAtT84GYtuxST7LYKjC3Az57RqBIi1+tqmEmdQx0eaWmdnTz+2 plOGtN3pscV8wqiw1KthB2s5389jaArJzD4cxbYpT9s/GAnin82Dxfdi2vooLNhWIw mM1HyaZJo3ycDOoArRMPTIzut8X2CV0p5qtq+OAo= Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3696.120.41.1.1\)) Subject: Re: [RFC PATCH] dt-bindings: dma: apple,sio: Add schema From: =?utf-8?Q?Martin_Povi=C5=A1er?= In-Reply-To: <20230214192544.GB27025-robh@kernel.org> Date: Tue, 14 Feb 2023 21:02:33 +0100 Cc: Vinod Koul , Krzysztof Kozlowski , Hector Martin , Sven Peter , Janne Grunau , dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev Content-Transfer-Encoding: quoted-printable Message-Id: <9938921C-29ED-4253-9F9D-4BA7432448F3@cutebit.org> References: <20230214141053.92731-1-povik+lin@cutebit.org> <20230214192544.GB27025-robh@kernel.org> To: Rob Herring Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Thanks for the review! > On 14. 2. 2023, at 20:25, Rob Herring wrote: >=20 > On Tue, Feb 14, 2023 at 03:10:53PM +0100, Martin Povi=C5=A1er wrote: >> Describe the SIO coprocessor which serves as pretend DMA controller = on >> recent Apple platforms. >>=20 >> Signed-off-by: Martin Povi=C5=A1er >> --- >>=20 >> Since the schema mentions a loader preparing the binding = appropriately, >> here's a PR with the relevant (WIP) loader code, if anyone wants to = look: >> https://github.com/AsahiLinux/m1n1/pull/286 >>=20 >> .../devicetree/bindings/dma/apple,sio.yaml | 129 = ++++++++++++++++++ >> 1 file changed, 129 insertions(+) >> create mode 100644 = Documentation/devicetree/bindings/dma/apple,sio.yaml >>=20 >> diff --git a/Documentation/devicetree/bindings/dma/apple,sio.yaml = b/Documentation/devicetree/bindings/dma/apple,sio.yaml >> new file mode 100644 >> index 000000000000..a76cc8265e76 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/dma/apple,sio.yaml (...) >> + memory-region: >> + description: >> + A number of references to reserved memory regions among which = are the DATA/TEXT >> + sections of coprocessor executable firmware and also auxiliary = firmware data >> + describing the available DMA-enabled peripherals >=20 > Can you define some range of number of entries. I think I can provide a reliable lower bound (at least one segment with DATA/TEXT and one segment with auxiliary data), and a conservative upper = bound (separate DATA/TEXT and no more than 6 segments of auxiliary data if = each kind of data has a segment of its own). I will work it in into the next = version. Martin