[<prev] [next>] [day] [month] [year] [list]
Message-ID: <d3d6392f-98ae-f6da-914c-943962d94a74@gmail.com>
Date: Fri, 31 Mar 2023 16:42:49 +0200
From: Matthias Brugger <matthias.bgg@...il.com>
To: Alexandre Mergnat <amergnat@...libre.com>,
Daniel Vetter <daniel@...ll.ch>, CK Hu <ck.hu@...iatek.com>,
Jitao Shi <jitao.shi@...iatek.com>,
Thierry Reding <thierry.reding@...il.com>,
Philipp Zabel <p.zabel@...gutronix.de>,
Sam Ravnborg <sam@...nborg.org>,
Rob Herring <robh+dt@...nel.org>,
Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
Chun-Kuang Hu <chunkuang.hu@...nel.org>,
David Airlie <airlied@...il.com>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Xinlei Lee <xinlei.lee@...iatek.com>
Cc: Guillaume La Roque <glaroque@...libre.com>,
linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org,
dri-devel@...ts.freedesktop.org,
Fabien Parent <fparent@...libre.com>,
Neil Armstrong <neil.armstrong@...aro.org>,
linux-pwm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 14/21] dt-bindings: soc: mediatek: specify which
compatible requires clocks property
On 09/03/2023 15:23, Alexandre Mergnat wrote:
> According to the mtk-mutex.c driver and the SoC DTS, the clock isn't
> required to work properly for some of MTK SoC. Improve the clock
> requirement by adding a condition which is function to the compatible.
>
> Signed-off-by: Alexandre Mergnat <amergnat@...libre.com>
Applied, thanks.
Now I think we can get rid of the no_clk variable in struct mtk_mutex_data, as
this should be mandated by the device-tree.
Regards,
Matthias
> ---
> .../bindings/soc/mediatek/mediatek,mutex.yaml | 20 +++++++++++++++++++-
> 1 file changed, 19 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
> index 9241e5fc7cff..ca0ca549257d 100644
> --- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
> +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
> @@ -69,12 +69,30 @@ properties:
> 4 arguments defined in this property. Each GCE subsys id is mapping to
> a client defined in the header include/dt-bindings/gce/<chip>-gce.h.
>
> +allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - mediatek,mt2701-disp-mutex
> + - mediatek,mt2712-disp-mutex
> + - mediatek,mt6795-disp-mutex
> + - mediatek,mt8173-disp-mutex
> + - mediatek,mt8186-disp-mutex
> + - mediatek,mt8186-mdp3-mutex
> + - mediatek,mt8192-disp-mutex
> + - mediatek,mt8195-disp-mutex
> + then:
> + required:
> + - clocks
> +
> +
> required:
> - compatible
> - reg
> - interrupts
> - power-domains
> - - clocks
>
> additionalProperties: false
>
>
Powered by blists - more mailing lists