2022-12-31 16:52:15

by Samuel Holland

[permalink] [raw]
Subject: [PATCH 2/4] media: dt-bindings: cedrus: Make allwinner,sram optional

Allwinner SoCs can remap some bits of peripheral SRAM to a contiguous
range of addresses for use by early boot software. Usually the video
engine's SRAM is used for this purpose, so its mapping must be switched
back before the hardware can be used.

However, the D1 and related SoCs use the DSP SRAM for this purpose. As a
result, the video engine SRAM is not switchable, and there is no need
for an allwinner,sram reference in the devicetree.

Signed-off-by: Samuel Holland <[email protected]>
---

.../bindings/media/allwinner,sun4i-a10-video-engine.yaml | 1 -
1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
index d5be7f604e8c..74363b242fff 100644
--- a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
+++ b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
@@ -73,7 +73,6 @@ required:
- clocks
- clock-names
- resets
- - allwinner,sram

additionalProperties: false

--
2.37.4


2022-12-31 19:30:36

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 2/4] media: dt-bindings: cedrus: Make allwinner,sram optional

On 31/12/2022 17:46, Samuel Holland wrote:
> Allwinner SoCs can remap some bits of peripheral SRAM to a contiguous
> range of addresses for use by early boot software. Usually the video
> engine's SRAM is used for this purpose, so its mapping must be switched
> back before the hardware can be used.
>


Acked-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2023-01-05 10:20:53

by Paul Kocialkowski

[permalink] [raw]
Subject: Re: [PATCH 2/4] media: dt-bindings: cedrus: Make allwinner,sram optional

Hi Samuel,

On Sat 31 Dec 22, 10:46, Samuel Holland wrote:
> Allwinner SoCs can remap some bits of peripheral SRAM to a contiguous
> range of addresses for use by early boot software. Usually the video
> engine's SRAM is used for this purpose, so its mapping must be switched
> back before the hardware can be used.
>
> However, the D1 and related SoCs use the DSP SRAM for this purpose. As a
> result, the video engine SRAM is not switchable, and there is no need
> for an allwinner,sram reference in the devicetree.
>
> Signed-off-by: Samuel Holland <[email protected]>

Reviewed-by: Paul Kocialkowski <[email protected]>

Cheers,

Paul

> ---
>
> .../bindings/media/allwinner,sun4i-a10-video-engine.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
> index d5be7f604e8c..74363b242fff 100644
> --- a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
> +++ b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml
> @@ -73,7 +73,6 @@ required:
> - clocks
> - clock-names
> - resets
> - - allwinner,sram
>
> additionalProperties: false
>
> --
> 2.37.4
>

--
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com


Attachments:
(No filename) (1.42 kB)
signature.asc (499.00 B)
Download all attachments