Received: by 2002:a05:6358:1087:b0:cb:c9d3:cd90 with SMTP id j7csp817598rwi; Mon, 10 Oct 2022 07:35:44 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4UP9YIBAzf07NqRkzWQdLIowJU6Rtoj6lBrmhyac1Q8gHZxcn6l8Kh64E2Ap7iDpbjMUMA X-Received: by 2002:a17:906:cc18:b0:78d:8f26:706c with SMTP id ml24-20020a170906cc1800b0078d8f26706cmr10524625ejb.424.1665412544469; Mon, 10 Oct 2022 07:35:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1665412544; cv=none; d=google.com; s=arc-20160816; b=JCeWF2QtKvdlW4HTDR1KQJ1mJIn24ceD/n2L84lWBg4rRwbCdIDerd57a0H3tY/KAy 79071AeCbv1K1RyQ0UrtfzLYX6QuOMyWRbBmyPsLp3LXw2mO2Ijd20Aqhx9+Er0Dzm98 oah4xgGtI+76SyLQd21B5S/QC6uq26jLCAXNb6i78ojpQcto0pj6y17RNRBbAVOAVbYD rfrbX0lSMycDORWelyf/j8v8/0TXgqdGNCl+0Q7S++SqwMs16Xu46r4i5OvGWd/tjf3z 8Xq+731bpgRRKYzQvpnw8BIE5bMu2H/UjZFWI//3calHinTmT80gZnfjzNsFmxc57Zeq 6u1Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:message-id:user-agent :references:in-reply-to:subject:cc:to:from:date:mime-version; bh=XUby94mE0lN3yFMyaxu7YP7wQXC7PpAfXKljtJMROTo=; b=e0ObdG0iy1eTuBd9R36hmmOWec+iLNivXmmmWzYjuOa34/SpEcho5Ufs6Kvw4r/wrh tXhqUJViyfXL7KmaQbGqLP3RoBmLbsBLZC7vQIrD/yRpuy0WH3Dv1AAcmWndGxtQYZoF ekOqyEEiFluH9zhVSCvQJSRtaVjj72NvmwV1qw4CxmvhZDCupJPAknpJpl3pldJvAhm0 kWaIyWbNIMEoEeS7dq/RBmY5Ib3kOskh7aiNqUvgY5nIUBVJVxcr4kJGsK4022OGMMpT cpGqHPyz/b8xGXE9D5sFZ7kvwtdSkXiv3VZka2V1bztmnLk/zEuKhYPxaWbZ1zfnP95F FU7g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id ay7-20020a056402202700b0045bc92ef2c0si6465594edb.195.2022.10.10.07.35.13; Mon, 10 Oct 2022 07:35:44 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229658AbiJJOYE (ORCPT + 99 others); Mon, 10 Oct 2022 10:24:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35814 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229544AbiJJOYD (ORCPT ); Mon, 10 Oct 2022 10:24:03 -0400 Received: from relay03.th.seeweb.it (relay03.th.seeweb.it [5.144.164.164]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 383626527B for ; Mon, 10 Oct 2022 07:24:02 -0700 (PDT) Received: from cp.tophost.it (vm1054.cs12.seeweb.it [217.64.195.253]) by m-r1.th.seeweb.it (Postfix) with ESMTPA id A809A1F660; Mon, 10 Oct 2022 16:23:59 +0200 (CEST) MIME-Version: 1.0 Date: Mon, 10 Oct 2022 16:10:42 +0200 From: konrad.dybcio@somainline.org To: Rob Herring Cc: Konrad Dybcio , ~postmarketos/upstreaming@lists.sr.ht, asahi@lists.linux.dev, Hector Martin , Sven Peter , Alyssa Rosenzweig , martin.botka@somainline.org, angelogioacchino.delregno@somainline.org, marijn.suijten@somainline.org, jamipkettunen@somainline.org, Thomas Gleixner , Marc Zyngier , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v4 1/2] dt-bindings: apple,aic: Document A7-A11 compatibles In-Reply-To: <20221010122727.GA432201-robh@kernel.org> References: <20221007200022.22844-1-konrad.dybcio@somainline.org> <20221007200022.22844-2-konrad.dybcio@somainline.org> <20221010122727.GA432201-robh@kernel.org> User-Agent: Roundcube Webmail/1.4.6 Message-ID: <0ff2cede3ac644141ba66f54244eb069@pop.somainline.org> X-Sender: konrad.dybcio@somainline.org Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2022-10-10 14:27, Rob Herring wrote: > On Fri, Oct 07, 2022 at 10:00:21PM +0200, Konrad Dybcio wrote: >> Document the compatibles for Apple A7-A11 SoCs. >> >> Signed-off-by: Konrad Dybcio >> --- >> Changes since v3: >> - Bindings: A7-A10+M1 only get SoC compat and generic fallback, >> A11 gets M1 fallback and generic fallback (removed A11 compatible) > > I have no idea if that makes sense or not. Do I have to go read the > rationale in the prior version? > > The question to ask on fallbacks, is would s/w that only understands > the > fallback compatible work as-is? Yep, A7-A10 share the same feature level ("apple,aic"), but I think having a separate, per-SoC compatible in the DT from the get-go makes a lot of sense, since we're not sure what's gonna happen when we, for example, add SMP support for these older chipsets, maybe they'll need some special handling. Similar story with A11 and M1. They can both work with "apple,aic", but they also support things that are enabled with the "apple,t8103-aic" compat string. I do not plan on adding specific compatibles for A7-A11 SoCs in the driver, unless we uncover some bugs / deviations. Konrad > >> - Drop Rob's A-b as this is essentially yet another rewrite.. >> >> .../bindings/interrupt-controller/apple,aic.yaml | 16 >> +++++++++++++--- >> 1 file changed, 13 insertions(+), 3 deletions(-) >> >> diff --git >> a/Documentation/devicetree/bindings/interrupt-controller/apple,aic.yaml >> b/Documentation/devicetree/bindings/interrupt-controller/apple,aic.yaml >> index e18107eafe7c..1427ea9b04a2 100644 >> --- >> a/Documentation/devicetree/bindings/interrupt-controller/apple,aic.yaml >> +++ >> b/Documentation/devicetree/bindings/interrupt-controller/apple,aic.yaml >> @@ -36,9 +36,19 @@ allOf: >> >> properties: >> compatible: >> - items: >> - - const: apple,t8103-aic >> - - const: apple,aic >> + oneOf: >> + - items: >> + - enum: >> + - apple,s5l8960x-aic >> + - apple,s8000-aic >> + - apple,t7000-aic >> + - apple,t8010-aic >> + - apple,t8103-aic >> + - const: apple,aic >> + - items: >> + - const: apple,t8015-aic >> + - const: apple,t8103-aic >> + - const: apple,aic >> >> interrupt-controller: true >> >> -- >> 2.37.3 >> >>