Re: [PATCH 3/4] arm64: dts: imx8mn: add GPC node and power domains

From: Adam Ford
Date: Fri Oct 23 2020 - 06:56:37 EST


On Fri, Oct 23, 2020 at 4:52 AM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
>
> On Thu, Oct 22, 2020 at 10:08:06AM -0500, Adam Ford wrote:
> > This adds the DT nodes to describe the power domains available on the
> > i.MX8MN. There are four power domains, but the displaymix and mipi
> > power domains need a separate clock block controller which is also
> > pending for 8MP and 8MM. Once the path for those is clear, Nano will
> > need something similar, but the registers for Nano differ. For now,
> > the dispmix and mipi are placeholders.
> >
> > Signed-off-by: Adam Ford <aford173@xxxxxxxxx>
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
> > index 9b4baf7bdfb1..27733fbe87e9 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
> > @@ -596,6 +596,55 @@ src: reset-controller@30390000 {
> > interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
> > #reset-cells = <1>;
> > };
> > +
> > + gpc: gpc@303a0000 {
> > + compatible = "fsl,imx8mn-gpc";
> > + reg = <0x303a0000 0x10000>;
> > + interrupt-parent = <&gic>;
> > + interrupt-controller;
> > + #interrupt-cells = <3>;
>
> Missing interrupts.
Oops. I'll go back and review this.

>
> > +
> > + pgc {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + pgc_hsiomix: power-domain@0 {
> > + #power-domain-cells = <0>;
> > + reg = <IMX8MN_POWER_DOMAIN_HSIOMIX>;
> > + clocks = <&clk IMX8MN_CLK_USB_BUS>;
> > + };
> > +
> > + pgc_otg1: power-domain@1 {
> > + #power-domain-cells = <0>;
> > + reg = <IMX8MN_POWER_DOMAIN_OTG1>;
> > + power-domains = <&pgc_hsiomix>;
> > + };
> > +
> > + pgc_gpumix: power-domain@2 {
> > + #power-domain-cells = <0>;
> > + reg = <IMX8MN_POWER_DOMAIN_GPUMIX>;
> > + clocks = <&clk IMX8MN_CLK_GPU_CORE_ROOT>,
> > + <&clk IMX8MN_CLK_GPU_SHADER_DIV>,
> > + <&clk IMX8MN_CLK_GPU_BUS_ROOT>,
> > + <&clk IMX8MN_CLK_GPU_AHB>;
> > + resets = <&src IMX8MQ_RESET_GPU_RESET>;
>
> Does it compile without include? Did the include come via dependencies
> of this patch?

Oops. I cherry picked the git commits, but I forgot to include the
includes which is why it compiles for me. I'll fix in a V2.

Thanks for all the reviews.

adam
>
> Best regards,
> Krzysztof
>