Date:   Thu, 11 Nov 2021 22:23:10 -0600
From:   Rob Herring <robh@kernel.org>
To:     Geert Uytterhoeven <geert+renesas@glider.be>
Cc:     Stanimir Varbanov <stanimir.varbanov@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        linux-arm-msm@vger.kernel.org,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Andy Gross <agross@kernel.org>, linux-media@vger.kernel.org,
        Dikshita Agarwal <dikshita@codeaurora.org>
Subject: Re: [PATCH v2] bindings: media: venus: Drop redundant maxItems for
 power-domain-names
Message-ID: <YY3sLvysqWlB4f9Q@robh.at.kernel.org>
References: <d94924e1bd00f396f2106f04d4a2bb839cf5f071.1636453406.git.geert+renesas@glider.be>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <d94924e1bd00f396f2106f04d4a2bb839cf5f071.1636453406.git.geert+renesas@glider.be>
Precedence: bulk

On Tue, 09 Nov 2021 11:29:41 +0100, Geert Uytterhoeven wrote:
> make dt_binding_check:
> 
>     Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: ignoring, error in schema: properties: power-domain-names
>     warning: no schema found in file: Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
>     Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: properties:power-domain-names: {'required': ['maxItems']} is not allowed for {'minItems': 2, 'maxItems': 3, 'items': [{'const': 'venus'}, {'const': 'vcodec0'}, {'const': 'cx'}]}
> 	   hint: "maxItems" is not needed with an "items" list
> 	   from schema $id: http://devicetree.org/meta-schemas/items.yaml#
> 
> Fixes: e48b839b6699c226 ("media: dt-bindings: media: venus: Add sc7280 dt schema")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
> v2:
>   - Add Acked-by,
>   - s/bogus/redundant/,
>   - Include full error message.
> ---
>  Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Applied, thanks!