[PATCH v3 1/2] dt-bindings: clock: mediatek: Add SMI LARBs reset for MT8188
Conor Dooley
conor at kernel.org
Tue Jan 21 09:30:23 PST 2025
On Tue, Jan 21, 2025 at 02:50:40PM +0800, Friday Yang wrote:
> SMI LARBs require reset functions when applying clamp operations.
> Add '#reset-cells' for the clock controller located in image, camera
> and IPE subsystems.
A new required property is an abi break, please explain why this is
required. What are "SMI LARBs"? Why did things previously work without
acting as a reset controller?
>
> Signed-off-by: Friday Yang <friday.yang at mediatek.com>
> ---
> .../bindings/clock/mediatek,mt8188-clock.yaml | 21 +++++++++++++++++++
> 1 file changed, 21 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> index 860570320545..2985c8c717d7 100644
> --- a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> @@ -57,6 +57,27 @@ required:
> - reg
> - '#clock-cells'
>
> +allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - mediatek,mt8188-camsys-rawa
> + - mediatek,mt8188-camsys-rawb
> + - mediatek,mt8188-camsys-yuva
> + - mediatek,mt8188-camsys-yuvb
> + - mediatek,mt8188-imgsys-wpe1
> + - mediatek,mt8188-imgsys-wpe2
> + - mediatek,mt8188-imgsys-wpe3
> + - mediatek,mt8188-imgsys1-dip-nr
> + - mediatek,mt8188-imgsys1-dip-top
> + - mediatek,mt8188-ipesys
> +
> + then:
> + required:
> + - '#reset-cells'
> +
> additionalProperties: false
>
> examples:
> --
> 2.46.0
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mediatek/attachments/20250121/6c39a343/attachment.sig>
More information about the Linux-mediatek
mailing list