[PATCH 3/3] arm64: dts: mediatek: mt8188: gce: add missing 'clock-names'

Julien Massot posted 3 patches 7 months, 1 week ago
[PATCH 3/3] arm64: dts: mediatek: mt8188: gce: add missing 'clock-names'
Posted by Julien Massot 7 months, 1 week ago
The 'clock-names' property is required by the dt binding.

Fix the following dtb-check error:
arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dtb: mailbox@10320000: 'clock-names' is a required property
from schema $id: http://devicetree.org/schemas/mailbox/mediatek,gce-mailbox.yaml#

Fixes: 45682a4fffdd ("arm64: dts: mediatek: mt8188: Add Global Command Engine mailboxes")
Signed-off-by: Julien Massot <julien.massot@collabora.com>
---
 arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
index dec6ce3e94e92c8e1e2c3680cb3584394d9058bd..d5892ca35bc29f830d7894ba592aca3f58dc3bdc 100644
--- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
@@ -1372,6 +1372,7 @@ gce0: mailbox@10320000 {
 			interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH 0>;
 			#mbox-cells = <2>;
 			clocks = <&infracfg_ao CLK_INFRA_AO_GCE>;
+			clock-names = "gce";
 		};
 
 		gce1: mailbox@10330000 {
@@ -1380,6 +1381,7 @@ gce1: mailbox@10330000 {
 			interrupts = <GIC_SPI 228 IRQ_TYPE_LEVEL_HIGH 0>;
 			#mbox-cells = <2>;
 			clocks = <&infracfg_ao CLK_INFRA_AO_GCE2>;
+			clock-names = "gce";
 		};
 
 		scp_cluster: scp@10720000 {

-- 
2.49.0
Re: [PATCH 3/3] arm64: dts: mediatek: mt8188: gce: add missing 'clock-names'
Posted by AngeloGioacchino Del Regno 7 months, 1 week ago
Il 15/05/25 15:31, Julien Massot ha scritto:
> The 'clock-names' property is required by the dt binding.
> 

I just remembered something....


...care to respin this patch instead?
https://lore.kernel.org/all/20240911104327.123602-1-angelogioacchino.delregno@collabora.com/

Cheers,
Angelo
Re: [PATCH 3/3] arm64: dts: mediatek: mt8188: gce: add missing 'clock-names'
Posted by Julien Massot 7 months, 1 week ago
On Thu, 2025-05-15 at 16:52 +0200, AngeloGioacchino Del Regno wrote:
> Il 15/05/25 15:31, Julien Massot ha scritto:
> > The 'clock-names' property is required by the dt binding.
> > 
> 
> I just remembered something....
> 
> 
> ...care to respin this patch instead?
> https://lore.kernel.org/all/20240911104327.123602-1-angelogioacchino.delregno@collabora.com/
> 

Ok, we will resend this patch instead, I dropped the patch 3/3 in the v2.

Regards,
Julien