Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 5 Jan 2023 10:57:27 +0100
From:   Paul Kocialkowski <paul.kocialkowski@bootlin.com>
To:     Samuel Holland <samuel@sholland.org>
Cc:     Chen-Yu Tsai <wens@csie.org>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Conor Dooley <conor@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Maxime Ripard <mripard@kernel.org>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
        linux-riscv@lists.infradead.org, linux-staging@lists.linux.dev,
        linux-sunxi@lists.linux.dev
Subject: Re: [PATCH 2/4] media: dt-bindings: cedrus: Make allwinner,sram
 optional
Message-ID: <Y7afB98vPc56bDhm@aptenodytes>
References: <20221231164628.19688-1-samuel@sholland.org>
 <20221231164628.19688-3-samuel@sholland.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="jljho90DVYkk8si4"
Content-Disposition: inline
In-Reply-To: <20221231164628.19688-3-samuel@sholland.org>
Precedence: bulk


--jljho90DVYkk8si4
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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.
>=20
> 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.
>=20
> Signed-off-by: Samuel Holland <samuel@sholland.org>

Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>

Cheers,

Paul

> ---
>=20
>  .../bindings/media/allwinner,sun4i-a10-video-engine.yaml         | 1 -
>  1 file changed, 1 deletion(-)
>=20
> 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-e=
ngine.yaml
> +++ b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-e=
ngine.yaml
> @@ -73,7 +73,6 @@ required:
>    - clocks
>    - clock-names
>    - resets
> -  - allwinner,sram
> =20
>  additionalProperties: false
> =20
> --=20
> 2.37.4
>=20

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

--jljho90DVYkk8si4
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEEJZpWjZeIetVBefti3cLmz3+fv9EFAmO2nwcACgkQ3cLmz3+f
v9Ggugf9HSS2KhNLgkevts5teZksFM4RMyoOoDlpVlPoHJ9dDgeTfdMcoN7vPA9r
YRKtDEyWUVX96YV/RmzVDWGazmbd46/khL5p8HhGHUjphTNVKm6/Ou5V91xBzgmN
lDL7m8hE2LE/LPIa6EOiIAdOwmI0UVcvp2ltKhZibVKZ3gDtovUQmEftYuQmNAF6
Z+iMSZEu2b0N+p1llPTxPwjHjruZknkYhRy3GEZcQEFTPUMIb5u39WjqA8TY1C7o
54aAxAgd0v/T/iELJYHlal4LodbLczu4/OJdHIj1bludFs/cZQwHV+51prwnTlCD
dHwTRJ6b3eLdlvq+km+n8zehF1FLTw==
=4H5j
-----END PGP SIGNATURE-----

--jljho90DVYkk8si4--