All of lore.kernel.org
 help / color / mirror / Atom feed
* imx6ul.dtsi missing ipg clock and pgc node
@ 2023-02-11 13:35 Stefan Wahren
  2023-02-23 16:12 ` Stefan Wahren
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Wahren @ 2023-02-11 13:35 UTC (permalink / raw)
  To: Anson Huang, linux-imx
  Cc: Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, Linux ARM, kernel

Hi,

i ran make dtbs_check for a imx6ull board and noticed that there are 
some warnings:

gpc@20dc000: 'clocks' is a required property
     From schema: 
/home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml
gpc@20dc000: 'clock-names' is a required property
     From schema: 
/home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml
gpc@20dc000: 'pgc' is a required property
     From schema: 
/home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml

According to the i.MX6ULL reference manual the clock and the power gate 
control IP seems to exists. Adding the ipg clock should be trivial, but 
i don't have a clue which power domains are available to pgc. I also 
didn't find anything in the Linux 4.9 vendor tree.

Can anyone help here?

Best regards


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: imx6ul.dtsi missing ipg clock and pgc node
  2023-02-11 13:35 imx6ul.dtsi missing ipg clock and pgc node Stefan Wahren
@ 2023-02-23 16:12 ` Stefan Wahren
  2023-02-24  1:23   ` Jacky Bai
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Wahren @ 2023-02-23 16:12 UTC (permalink / raw)
  To: Peng Fan, Dong Aisheng
  Cc: Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, Linux ARM,
	linux-imx, kernel

Hi Dong,
hi Peng,

Am 11.02.23 um 14:35 schrieb Stefan Wahren:
> Hi,
>
> i ran make dtbs_check for a imx6ull board and noticed that there are 
> some warnings:
>
> gpc@20dc000: 'clocks' is a required property
>     From schema: 
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml
> gpc@20dc000: 'clock-names' is a required property
>     From schema: 
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml
> gpc@20dc000: 'pgc' is a required property
>     From schema: 
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/fsl,imx-gpc.yaml
>
> According to the i.MX6ULL reference manual the clock and the power 
> gate control IP seems to exists. Adding the ipg clock should be 
> trivial, but i don't have a clue which power domains are available to 
> pgc. I also didn't find anything in the Linux 4.9 vendor tree.
>
> Can anyone help here?

originally i send this also to Anson Huang, but his email address was 
bouncing. Maybe you can help me here.

Best regards

>
> Best regards
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: imx6ul.dtsi missing ipg clock and pgc node
  2023-02-23 16:12 ` Stefan Wahren
@ 2023-02-24  1:23   ` Jacky Bai
  2023-02-24  9:38     ` Stefan Wahren
  0 siblings, 1 reply; 4+ messages in thread
From: Jacky Bai @ 2023-02-24  1:23 UTC (permalink / raw)
  To: Stefan Wahren, Peng Fan, Aisheng Dong
  Cc: Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, Linux ARM,
	dl-linux-imx, kernel

Hi Stefan,

> Subject: Re: imx6ul.dtsi missing ipg clock and pgc node
> 
> Hi Dong,
> hi Peng,
> 
> Am 11.02.23 um 14:35 schrieb Stefan Wahren:
> > Hi,
> >
> > i ran make dtbs_check for a imx6ull board and noticed that there are
> > some warnings:
> >
> > gpc@20dc000: 'clocks' is a required property
> >     From schema:
> >
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
> > fsl,imx-gpc.yaml
> > gpc@20dc000: 'clock-names' is a required property
> >     From schema:
> >
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
> > fsl,imx-gpc.yaml
> > gpc@20dc000: 'pgc' is a required property
> >     From schema:
> >
> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
> > fsl,imx-gpc.yaml
> >
> > According to the i.MX6ULL reference manual the clock and the power
> > gate control IP seems to exists. Adding the ipg clock should be
> > trivial, but i don't have a clue which power domains are available to
> > pgc. I also didn't find anything in the Linux 4.9 vendor tree.
> >
> > Can anyone help here?

ipg clock is the IMX6UL_CLK_IPG, for PGC, the arm PGC is available and can also be added to fix the dtbs_check issue.

pgc {
        #address-cells = <1>;
        #size-cells = <0>;

        power-domain@0 {
                reg = <0>;
                #power-domain-cells = <0>;
        };
};

You will submit a fix or need NXP to do that?

BR
> 
> originally i send this also to Anson Huang, but his email address was bouncing.
> Maybe you can help me here.
> 
> Best regards
> 
> >
> > Best regards
> >
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: imx6ul.dtsi missing ipg clock and pgc node
  2023-02-24  1:23   ` Jacky Bai
@ 2023-02-24  9:38     ` Stefan Wahren
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Wahren @ 2023-02-24  9:38 UTC (permalink / raw)
  To: Jacky Bai, Peng Fan, Aisheng Dong
  Cc: Krzysztof Kozlowski, Shawn Guo, Fabio Estevam, Linux ARM,
	dl-linux-imx, kernel

Hi Jacky,

Am 24.02.23 um 02:23 schrieb Jacky Bai:
> Hi Stefan,
>
>> Subject: Re: imx6ul.dtsi missing ipg clock and pgc node
>>
>> Hi Dong,
>> hi Peng,
>>
>> Am 11.02.23 um 14:35 schrieb Stefan Wahren:
>>> Hi,
>>>
>>> i ran make dtbs_check for a imx6ull board and noticed that there are
>>> some warnings:
>>>
>>> gpc@20dc000: 'clocks' is a required property
>>>      From schema:
>>>
>> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
>>> fsl,imx-gpc.yaml
>>> gpc@20dc000: 'clock-names' is a required property
>>>      From schema:
>>>
>> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
>>> fsl,imx-gpc.yaml
>>> gpc@20dc000: 'pgc' is a required property
>>>      From schema:
>>>
>> /home/stefanw/linux-next-real/Documentation/devicetree/bindings/power/
>>> fsl,imx-gpc.yaml
>>>
>>> According to the i.MX6ULL reference manual the clock and the power
>>> gate control IP seems to exists. Adding the ipg clock should be
>>> trivial, but i don't have a clue which power domains are available to
>>> pgc. I also didn't find anything in the Linux 4.9 vendor tree.
>>>
>>> Can anyone help here?
> ipg clock is the IMX6UL_CLK_IPG, for PGC, the arm PGC is available and can also be added to fix the dtbs_check issue.
>
> pgc {
>          #address-cells = <1>;
>          #size-cells = <0>;
>
>          power-domain@0 {
>                  reg = <0>;
>                  #power-domain-cells = <0>;
>          };
> };
>
> You will submit a fix or need NXP to do that?

i plan to submit related imx6ul DTS fixes, so i can integrate this one.

Thanks

>
> BR
>> originally i send this also to Anson Huang, but his email address was bouncing.
>> Maybe you can help me here.
>>
>> Best regards
>>
>>> Best regards
>>>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-02-24  9:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-11 13:35 imx6ul.dtsi missing ipg clock and pgc node Stefan Wahren
2023-02-23 16:12 ` Stefan Wahren
2023-02-24  1:23   ` Jacky Bai
2023-02-24  9:38     ` Stefan Wahren

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.