All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Cc: airlied@gmail.com, amergnat@baylibre.com, andrew+netdev@lunn.ch,
	andrew-ct.chen@mediatek.com,
	angelogioacchino.delregno@collabora.com, broonie@kernel.org,
	chunkuang.hu@kernel.org, ck.hu@mediatek.com, conor+dt@kernel.org,
	davem@davemloft.net, dmitry.torokhov@gmail.com,
	edumazet@google.com, flora.fu@mediatek.com,
	houlong.wei@mediatek.com, jeesw@melfas.com,
	jmassot@collabora.com, kernel@collabora.com, krzk+dt@kernel.org,
	kuba@kernel.org, kyrie.wu@mediatek.corp-partner.google.com,
	lgirdwood@gmail.com, linus.walleij@linaro.org,
	louisalexis.eyraud@collabora.com,
	maarten.lankhorst@linux.intel.com, matthias.bgg@gmail.com,
	mchehab@kernel.org, minghsiu.tsai@mediatek.com,
	mripard@kernel.org, p.zabel@pengutronix.de, pabeni@redhat.com,
	robh@kernel.org, sean.wang@kernel.org, simona@ffwll.ch,
	support.opensource@diasemi.com, tiffany.lin@mediatek.com,
	tzimmermann@suse.de, yunfei.dong@mediatek.com,
	devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org,
	linux-gpio@vger.kernel.org, linux-input@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
	linux-mediatek@lists.infradead.org, linux-sound@vger.kernel.org,
	netdev@vger.kernel.org
Subject: Re: [PATCH v1 03/14] dt-bindings: arm: mediatek: mmsys: Add assigned-clocks/rates properties
Date: Tue, 9 Sep 2025 08:29:21 +0200	[thread overview]
Message-ID: <898bf39e-1b34-40e9-bdfa-ec4eca1c3f7d@kernel.org> (raw)
In-Reply-To: <1cf0b296-adaa-4c80-864c-9b78f09cd3e3@collabora.com>

On 08/09/2025 21:19, Ariel D'Alessandro wrote:
> Krzysztof,
> 
> On 8/21/25 3:43 AM, Krzysztof Kozlowski wrote:
>> On Wed, Aug 20, 2025 at 02:12:51PM -0300, Ariel D'Alessandro wrote:
>>> Current, the DT bindings for MediaTek mmsys controller is missing the
>>> assigned-clocks and assigned-clocks-rates properties. Add these and
>>
>> No, they do not miss them. I don't understand why you are adding these.
> 
> The reason I added these is due to the following check error:
> 
> $ make -j$(nproc) CHECK_DTBS=y mediatek/mt8173-elm.dtb
>    DTC [C] arch/arm64/boot/dts/mediatek/mt8173-elm.dtb
> [...]
> arch/arm64/boot/dts/mediatek/mt8173-elm.dtb: syscon@14000000 
> (mediatek,mt8173-mmsys): 'assigned-clock-rates', 'assigned-clocks' do 
> not match any of the regexes: '^pinctrl-[0-9]+$'
> 	from schema $id: 
> http://devicetree.org/schemas/arm/mediatek/mediatek,mmsys.yaml#

This is looking like missing clocks or other unevaluated property by the
binding.


Best regards,
Krzysztof


  reply	other threads:[~2025-09-09  7:34 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-20 17:12 [PATCH v1 00/14] MediaTek dt-bindings sanitization (MT8173) Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 01/14] media: dt-bindings: Convert MediaTek mt8173-mdp bindings to YAML Ariel D'Alessandro
2025-08-21  6:46   ` Krzysztof Kozlowski
2025-09-08 17:52     ` Ariel D'Alessandro
2025-09-09  6:32       ` Krzysztof Kozlowski
2025-09-11 13:38         ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 02/14] media: dt-bindings: Convert MediaTek mt8173-vpu " Ariel D'Alessandro
2025-08-21  6:47   ` Krzysztof Kozlowski
2025-09-08 18:35     ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 03/14] dt-bindings: arm: mediatek: mmsys: Add assigned-clocks/rates properties Ariel D'Alessandro
2025-08-21  6:43   ` Krzysztof Kozlowski
2025-09-08 19:19     ` Ariel D'Alessandro
2025-09-09  6:29       ` Krzysztof Kozlowski [this message]
2025-09-11 13:40         ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 04/14] net: dt-bindings: Convert Marvell 8897/8997 bindings to YAML Ariel D'Alessandro
2025-08-21 13:24   ` Rob Herring (Arm)
2025-08-21 14:28   ` Rob Herring
2025-09-08 19:54     ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 05/14] sound: dt-bindings: Convert MediaTek RT5650 codecs " Ariel D'Alessandro
2025-08-22 15:14   ` Rob Herring
2025-09-08 20:04     ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 06/14] dt-bindings: display: mediatek,od: Add mediatek,gce-client-reg property Ariel D'Alessandro
2025-08-20 17:12   ` [PATCH v1 06/14] dt-bindings: display: mediatek, od: Add mediatek, gce-client-reg property Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 07/14] dt-bindings: display: mediatek,ufoe: Add mediatek,gce-client-reg property Ariel D'Alessandro
2025-08-20 17:12   ` [PATCH v1 07/14] dt-bindings: display: mediatek, ufoe: Add mediatek, gce-client-reg property Ariel D'Alessandro
2025-08-21  6:50   ` [PATCH v1 07/14] dt-bindings: display: mediatek,ufoe: Add mediatek,gce-client-reg property Krzysztof Kozlowski
2025-09-10 14:04     ` Ariel D'Alessandro
2025-09-10 14:21       ` Krzysztof Kozlowski
2025-09-10 15:45         ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 08/14] arm64: dts: mediatek: mt8173: Fix mt8173-pinctrl node names Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 09/14] dt-bindings: pinctrl: mediatek,mt65xx-pinctrl: Allow gpio-line-names Ariel D'Alessandro
2025-08-20 17:12   ` [PATCH v1 09/14] dt-bindings: pinctrl: mediatek, mt65xx-pinctrl: " Ariel D'Alessandro
2025-08-21 11:53   ` [PATCH v1 09/14] dt-bindings: pinctrl: mediatek,mt65xx-pinctrl: " Linus Walleij
2025-08-22 15:17   ` Rob Herring (Arm)
2025-08-20 17:12 ` [PATCH v1 10/14] regulator: dt-bindings: Convert Dialog Semiconductor DA9211 Regulators to YAML Ariel D'Alessandro
2025-08-21  6:53   ` Krzysztof Kozlowski
2025-09-10 14:53     ` Ariel D'Alessandro
2025-08-20 17:12 ` [PATCH v1 11/14] arm64: dts: mediatek: mt8173-elm: Drop unused bank supply Ariel D'Alessandro
2025-08-20 17:13 ` [PATCH v1 12/14] dt-bindings: soc: mediatek: pwrap: Add power-domains property Ariel D'Alessandro
2025-08-22 15:50   ` Rob Herring (Arm)
2025-08-20 17:13 ` [PATCH v1 13/14] dt-bindings: input/touchscreen: Convert MELFAS MIP4 Touchscreen to YAML Ariel D'Alessandro
2025-08-21 11:56   ` Linus Walleij
2025-09-05 10:02     ` Dmitry Torokhov
2025-09-05 11:33       ` Linus Walleij
2025-09-09  6:56         ` Krzysztof Kozlowski
2025-09-10 16:01           ` Ariel D'Alessandro
2025-08-22 15:52   ` Rob Herring
2025-09-10 15:51     ` Ariel D'Alessandro
2025-08-20 17:13 ` [PATCH v1 14/14] dt-bindings: media: mediatek,jpeg: Fix jpeg encoder/decoder ranges Ariel D'Alessandro
2025-08-20 17:13   ` [PATCH v1 14/14] dt-bindings: media: mediatek, jpeg: " Ariel D'Alessandro
2025-08-20 18:55   ` [PATCH v1 14/14] dt-bindings: media: mediatek,jpeg: " Rob Herring
2025-09-10 15:02     ` Ariel D'Alessandro
2025-08-20 17:19 ` [PATCH v1 00/14] MediaTek dt-bindings sanitization (MT8173) Mark Brown
2025-09-11 14:29   ` Ariel D'Alessandro
2025-08-20 18:57 ` Rob Herring (Arm)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=898bf39e-1b34-40e9-bdfa-ec4eca1c3f7d@kernel.org \
    --to=krzk@kernel.org \
    --cc=airlied@gmail.com \
    --cc=amergnat@baylibre.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=andrew-ct.chen@mediatek.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=ariel.dalessandro@collabora.com \
    --cc=broonie@kernel.org \
    --cc=chunkuang.hu@kernel.org \
    --cc=ck.hu@mediatek.com \
    --cc=conor+dt@kernel.org \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=edumazet@google.com \
    --cc=flora.fu@mediatek.com \
    --cc=houlong.wei@mediatek.com \
    --cc=jeesw@melfas.com \
    --cc=jmassot@collabora.com \
    --cc=kernel@collabora.com \
    --cc=krzk+dt@kernel.org \
    --cc=kuba@kernel.org \
    --cc=kyrie.wu@mediatek.corp-partner.google.com \
    --cc=lgirdwood@gmail.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=louisalexis.eyraud@collabora.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mchehab@kernel.org \
    --cc=minghsiu.tsai@mediatek.com \
    --cc=mripard@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=pabeni@redhat.com \
    --cc=robh@kernel.org \
    --cc=sean.wang@kernel.org \
    --cc=simona@ffwll.ch \
    --cc=support.opensource@diasemi.com \
    --cc=tiffany.lin@mediatek.com \
    --cc=tzimmermann@suse.de \
    --cc=yunfei.dong@mediatek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.