lkml.org 
[lkml]   [2022]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2] dt-bindings: arm: mediatek: mmsys: refine power and gce properties
From
Date
Hi Krzysztof,

Thanks for the reviews.

On Fri, 2022-04-29 at 22:58 +0200, Krzysztof Kozlowski wrote:
> On 29/04/2022 10:30, Jason-JH.Lin wrote:
> > Power:
> > Refine description and add item number for power-domains
> > property.
> >
> > GCE:
> > Refine description and add item number for mboxes property and
> > mediatek,gce-client-reg property.
> >
> > Fixes: 1da90b8a7bae ("dt-bindings: arm: mediatek: mmsys: add power
> > and gce properties")
> > Signed-off-by: Jason-JH.Lin <jason-jh.lin@mediatek.com>

[snip]

> > mboxes:
> > + minItems: 1
> > + maxItems: 24
>
> Wait, no, I wrote it should be 16 for one device and 24 for other,
> not
> 24 for all...
>

Ok, I thought maxItems: 24 contained 16, but it's not clear enough.


I'm not sure how to describe different maxItems for different
platforms. Does it come like this?

1. Remove the maxItems in mboxes property.
2. Add the allOf condition before the line "additionalProperties:
false". E.g.

allOf:
- if:
properties:
compatible:
contains:
const: mediatek,mt8173-mmsys
then:
properties:
mboxes:
maxItems: 16
else:
properties:
mboxes:
maxItems: 24

additionalProperties: false
...


Regards,
Jason-JH.Lin

> Best regards,
> Krzysztof
--
Jason-JH Lin <jason-jh.lin@mediatek.com>

\
 
 \ /
  Last update: 2022-05-01 10:28    [W:0.041 / U:1.248 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site