[PATCH] ARM: dts: i.MX8MP: disable NPU node if hardware is missing

[PATCH] ARM: dts: i.MX8MP: disable NPU node if hardware is missing

[Holger Assmann]

Besides its power domain, the NPU node itself also needs to be disabled
in the kernel device tree if no respective hardware is present at the
SoC.

The power domain has already been dealt with by upstream commit
924101c81d6e ("ARM: i.MX8MP: add feature controller support for Plus").
This patch adds the necessary entry for the NPU node.

Signed-off-by: Holger Assmann <h.assmann@pengutronix.de>
---
 arch/arm/dts/imx8mp.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/dts/imx8mp.dtsi b/arch/arm/dts/imx8mp.dtsi
index 4d1f1bf588..1604fa13a5 100644
--- a/arch/arm/dts/imx8mp.dtsi
+++ b/arch/arm/dts/imx8mp.dtsi
@@ -99,6 +99,10 @@ &vpumix_blk_ctrl {
 	barebox,feature-gates = <&feat IMX8M_FEAT_VPU>;
 };
 
+&npu {
+	barebox,feature-gates = <&feat IMX8M_FEAT_NPU>;
+};
+
 &pgc_mlmix {
 	barebox,feature-gates = <&feat IMX8M_FEAT_NPU>;
 };
-- 
2.39.5

Re: [PATCH] ARM: dts: i.MX8MP: disable NPU node if hardware is missing

[Ahmad Fatoum]

Hello Holger,

Thanks for your fix.

On 08.11.24 11:57, Holger Assmann wrote:
> Besides its power domain, the NPU node itself also needs to be disabled
> in the kernel device tree if no respective hardware is present at the
> SoC.
> 
> The power domain has already been dealt with by upstream commit
> 924101c81d6e ("ARM: i.MX8MP: add feature controller support for Plus").
> This patch adds the necessary entry for the NPU node.
> > Signed-off-by: Holger Assmann <h.assmann@pengutronix.de>

Reviewed-by: Ahmad Fatoum <a.fatoum@pengutronix.de

Cheers,
Ahmad

> ---
>  arch/arm/dts/imx8mp.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/dts/imx8mp.dtsi b/arch/arm/dts/imx8mp.dtsi
> index 4d1f1bf588..1604fa13a5 100644
> --- a/arch/arm/dts/imx8mp.dtsi
> +++ b/arch/arm/dts/imx8mp.dtsi
> @@ -99,6 +99,10 @@ &vpumix_blk_ctrl {
>  	barebox,feature-gates = <&feat IMX8M_FEAT_VPU>;
>  };
>  
> +&npu {
> +	barebox,feature-gates = <&feat IMX8M_FEAT_NPU>;
> +};
> +
>  &pgc_mlmix {
>  	barebox,feature-gates = <&feat IMX8M_FEAT_NPU>;
>  };


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Re: [PATCH] ARM: dts: i.MX8MP: disable NPU node if hardware is missing

[Sascha Hauer]

On Fri, 08 Nov 2024 11:57:51 +0100, Holger Assmann wrote:
> Besides its power domain, the NPU node itself also needs to be disabled
> in the kernel device tree if no respective hardware is present at the
> SoC.
> 
> The power domain has already been dealt with by upstream commit
> 924101c81d6e ("ARM: i.MX8MP: add feature controller support for Plus").
> This patch adds the necessary entry for the NPU node.
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: i.MX8MP: disable NPU node if hardware is missing
      https://git.pengutronix.de/cgit/barebox/commit/?id=a93c184fb15c (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>