Re: [PATCH v5 3/3] ARM: imx7d-remarkable2.dts: Initial device tree for reMarkable2
From: Marco Felsch
Date: Fri Mar 19 2021 - 09:07:00 EST
Hi Alistair,
the patch looks quite good only a few notes inline.
PS: It would be cool to have a log to previous patch versions.
On 21-03-15 22:27, Alistair Francis wrote:
...
> +&crypto {
> + status = "disabled";
> +};
Why do you disable the crypto subsystem?
> +&dma_apbh {
> + status = "disabled";
> +};
Why do you disable this dma controller?
> +
> +&sdma {
> + status = "okay";
> +};
This is the default state so you can drop this node.
Regards,
Marco