[PATCH] arm64: dts: imx8mm: Add SNVS LPGPR on MX8Menlo board

[Shawn Guo]

On Sat, May 21, 2022 at 05:07:50PM +0200, Marek Vasut wrote:
> Add SNVS LPGPR bindings on this system, the LPGPR is used to store
> boot counter.
> 
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Fabio Estevam <festevam at denx.de>
> Cc: Marcel Ziswiler <marcel.ziswiler at toradex.com>
> Cc: Peng Fan <peng.fan at nxp.com>
> Cc: Rob Herring <robh+dt at kernel.org>
> Cc: Shawn Guo <shawnguo at kernel.org>
> Cc: NXP Linux Team <linux-imx at nxp.com>
> Cc: devicetree at vger.kernel.org
> To: linux-arm-kernel at lists.infradead.org
> ---
> NOTE: Depends on
>       https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/commit/?h=for-next&id=fee6de80bdd3df976a43f3092a165cb43c072f20
> ---
>  arch/arm64/boot/dts/freescale/imx8mm-mx8menlo.dts | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-mx8menlo.dts b/arch/arm64/boot/dts/freescale/imx8mm-mx8menlo.dts
> index 92eaf4ef45638..6956c9bb992be 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-mx8menlo.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-mx8menlo.dts
> @@ -303,6 +303,12 @@ &sai2 {
>  	status = "disabled";
>  };
>  
> +&snvs {
> +	snvs-lpgpr {
> +		compatible = "fsl,imx7d-snvs-lpgpr";

Should we encode imx8mm specific compatible as well, while you added it
in the bindings?

Also this is a SoC rather than board device, so we may want to add it in
soc.dtsi instead?

Shawn

> +	};
> +};
> +
>  &uart1 {
>  	uart-has-rtscts;
>  	status = "okay";
> -- 
> 2.35.1
>