lkml.org 
[lkml]   [2022]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4 1/2] dt-bindings: apple,aic: Document A7-A11 compatibles
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 <konrad.dybcio@somainline.org>
>> ---
>> 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
>>
>>

\
 
 \ /
  Last update: 2022-10-10 16:24    [W:0.062 / U:0.208 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site