From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AC4971990C4; Fri, 9 Aug 2024 16:18:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723220290; cv=none; b=Kn06bA5Z4qnxyeydrr0t1NK054LZbZjEV8iU85hAUKURnU5pIDOKx5WeVPVZ7yt3TYkDnA1MkRzHHDCvvr8TUkgfpI8M9g328IbXlLjSWiMHy4Rh7w0hTKEmvz3U0zVmnGg+BYdrbaTaN/3FiQVI2saxtnOM0JYWM7wZdRN13Hg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723220290; c=relaxed/simple; bh=ek/Kg6OmXUi6D5cWxC7C10DYiBHbw6OL9TD8XPV7y4Y=; h=Date:Content-Type:MIME-Version:From:To:Cc:In-Reply-To:References: Message-Id:Subject; b=UFxtZ+KIVft/rHNSA65l1sh1Pc0UXCbZ8nCGhK7IGZ+ClImxMLyXDULhPay81OtNnHg9yiPueuL7A3GZI+NK/uB8eP+fsTy2t/w2PB0q6NXllc7bV0BHow6eHqZ4Ka/p539NHf0m573gOe49SyYBrDGVhkhJCgP5I9z9P9I2z5g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=L9p0E7Ee; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="L9p0E7Ee" Received: by smtp.kernel.org (Postfix) with ESMTPSA id EA725C32782; Fri, 9 Aug 2024 16:18:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1723220290; bh=ek/Kg6OmXUi6D5cWxC7C10DYiBHbw6OL9TD8XPV7y4Y=; h=Date:From:To:Cc:In-Reply-To:References:Subject:From; b=L9p0E7EeQlzvhjRAqoGpPdcihwHW1MjQbllDSkwN06TxsM8qdXsU6oSBQB9tXP/cJ yADOr0bk3A/w5CS4ytEl0pVafURAAnGy6zOvlnWf8hPhBHiuJYleEaHytbChUqrC9B smY3y3ezB64BiTbn8lBcvsjxyjgez2K8ANOqBorTJAS4uoVBXKsNQbje2c176iUbT6 wxjvkme+t/94n448CAveKNbDTTIPJ04CYafRy5mU8vc0mD07nhhG+8SZf0IgPHbfpZ Ra2eBlsO5e959W0mhdWJGXYF5ZMTqAC0vktQbwOFxeyiFqIHmmeh9jU9rqdy79t52Y 6bOG5kGvno5TA== Date: Fri, 09 Aug 2024 10:18:08 -0600 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: "Rob Herring (Arm)" To: Yannick Fertre Cc: Alexandre Torgue , Conor Dooley , Maxime Coquelin , linux-arm-kernel@lists.infradead.org, Krzysztof Kozlowski , linux-stm32@st-md-mailman.stormreply.com, Maarten Lankhorst , David Airlie , Maxime Ripard , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Daniel Vetter , Raphael Gallais-Pou , dri-devel@lists.freedesktop.org, Philippe Cornu , Thomas Zimmermann In-Reply-To: <20240809151314.221746-1-yannick.fertre@foss.st.com> References: <20240809151314.221746-1-yannick.fertre@foss.st.com> Message-Id: <172322028851.588157.5328364169708651531.robh@kernel.org> Subject: Re: [PATCH v2] dt-bindings: display: st,stm32-ltdc: Document stm32mp25 compatible On Fri, 09 Aug 2024 17:13:14 +0200, Yannick Fertre wrote: > Add "st,stm32mp25-ltdc" compatible for SOC MP25. This new SOC introduce > new clocks (bus, ref & lvds). Bus clock was separated from lcd clock. > New sources are possible for lcd clock (lvds / ref). > > Signed-off-by: Yannick Fertre > --- > Changes in v2: Rework clock property. > .../bindings/display/st,stm32-ltdc.yaml | 51 +++++++++++++++---- > 1 file changed, 41 insertions(+), 10 deletions(-) > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml: allOf:0:else:properties:clocks: {'maxItems': 1, 'items': [{'description': 'Lcd Clock'}]} should not be valid under {'required': ['maxItems']} hint: "maxItems" is not needed with an "items" list from schema $id: http://devicetree.org/meta-schemas/items.yaml# /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml: allOf:0:then:properties:clocks: {'maxItems': 4, 'items': [{'description': 'Lcd Clock'}, {'description': 'Bus Clock'}, {'description': 'Reference Clock'}, {'description': 'Lvds Clock'}]} should not be valid under {'required': ['maxItems']} hint: "maxItems" is not needed with an "items" list from schema $id: http://devicetree.org/meta-schemas/items.yaml# /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/st,stm32-ltdc.example.dtb: display-controller@40016800: 'clock-names', 'clocks' do not match any of the regexes: 'pinctrl-[0-9]+' from schema $id: http://devicetree.org/schemas/display/st,stm32-ltdc.yaml# doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240809151314.221746-1-yannick.fertre@foss.st.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.