mbox series

[v7,00/11] clk: imx8: add new clock binding for better pm support

Message ID 1596009618-25516-1-git-send-email-aisheng.dong@nxp.com (mailing list archive)
Headers show
Series clk: imx8: add new clock binding for better pm support | expand

Message

Aisheng Dong July 29, 2020, 8 a.m. UTC
This patch series is a preparation for the MX8 Architecture improvement.
As for IMX SCU based platforms like MX8QM and MX8QXP, they are comprised
of a couple of SS(Subsystems) while most of them within the same SS
can be shared. e.g. Clocks, Devices and etc.

However, current clock binding is using SW IDs for device tree to use
which can cause troubles in writing the common <soc>-ss-xx.dtsi file for
different SoCs.

This patch series aims to introduce a new binding which is more close to
hardware and platform independent and can makes us write a more general
drivers for different SCU based SoCs.

Another important thing is that on MX8, each Clock resource is associated
with a power domain. So we have to attach that clock device to the power
domain in order to make it work properly. Further more, the clock state
will be lost when its power domain is completely off during suspend/resume,
so we also introduce the clock state save&restore mechanism.

It's based on latest shanw/for-next branch.

The top commit is:
3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next

ChangeLog:
v6->v7:
 * addressed all comments from Stephen
 * rebased to latest shawn/for-next
v5->v6:
 * add scu clk unregister if add provider failed
v4->v5:
 * Address all comments from Stephen
v3->v4:
 * use clk-indices for LPCG to fetch each clks offset from dt
v2->v3:
 * change scu clk into two cells binding
 * add clk pm patches to ease the understand of the changes
v1->v2:
 * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
   Documentation/devicetree/bindings/arm/arm,scpi.txt
 * Add required power domain property
 * Dropped PATCH 3&4 first, will send the updated version accordingly
   after the binding is finally determined,

Dong Aisheng (11):
  dt-bindings: firmware: imx-scu: new binding to parse clocks from
    device tree
  dt-bindings: clock: imx-lpcg: add support to parse clocks from device
    tree
  clk: imx: scu: add two cells binding support
  clk: imx: scu: bypass cpu power domains
  clk: imx: scu: allow scu clk to take device pointer
  clk: imx: scu: add runtime pm support
  clk: imx: scu: add suspend/resume support
  clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
  clk: imx: lpcg: allow lpcg clk to take device pointer
  clk: imx: clk-imx8qxp-lpcg: add runtime pm support
  clk: imx: lpcg: add suspend/resume support

 .../bindings/arm/freescale/fsl,scu.txt        |  12 +-
 .../bindings/clock/imx8qxp-lpcg.yaml          |  79 ++++--
 drivers/clk/imx/clk-imx8qxp-lpcg.c            | 139 +++++++++++
 drivers/clk/imx/clk-imx8qxp.c                 | 136 +++++-----
 drivers/clk/imx/clk-lpcg-scu.c                |  53 +++-
 drivers/clk/imx/clk-scu.c                     | 234 +++++++++++++++++-
 drivers/clk/imx/clk-scu.h                     |  56 ++++-
 include/dt-bindings/clock/imx8-lpcg.h         |  14 ++
 8 files changed, 620 insertions(+), 103 deletions(-)
 create mode 100644 include/dt-bindings/clock/imx8-lpcg.h

Comments

Dong Aisheng July 29, 2020, 8:04 a.m. UTC | #1
Hi Stephen,

This patchset addressed all your new comments.
Right now only Patch 2/3 still haven't got any R-b or A-b tags. So
still need your help to review.
All other patches already got the R-b tags.

BTW, for [PATCH v7 02/11] dt-bindings: clock: imx-lpcg: add support to parse
 clocks from device tree.
It was reviewed by Rob/Shawn and you before, however, due to the legacy binding
file has been removed and changed to json format.
So this patch also updated to json format accordingly based on the
former reviewed patch
and need a re-review.

Regards
Aisheng

On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <aisheng.dong@nxp.com> wrote:
>
> This patch series is a preparation for the MX8 Architecture improvement.
> As for IMX SCU based platforms like MX8QM and MX8QXP, they are comprised
> of a couple of SS(Subsystems) while most of them within the same SS
> can be shared. e.g. Clocks, Devices and etc.
>
> However, current clock binding is using SW IDs for device tree to use
> which can cause troubles in writing the common <soc>-ss-xx.dtsi file for
> different SoCs.
>
> This patch series aims to introduce a new binding which is more close to
> hardware and platform independent and can makes us write a more general
> drivers for different SCU based SoCs.
>
> Another important thing is that on MX8, each Clock resource is associated
> with a power domain. So we have to attach that clock device to the power
> domain in order to make it work properly. Further more, the clock state
> will be lost when its power domain is completely off during suspend/resume,
> so we also introduce the clock state save&restore mechanism.
>
> It's based on latest shanw/for-next branch.
>
> The top commit is:
> 3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next
>
> ChangeLog:
> v6->v7:
>  * addressed all comments from Stephen
>  * rebased to latest shawn/for-next
> v5->v6:
>  * add scu clk unregister if add provider failed
> v4->v5:
>  * Address all comments from Stephen
> v3->v4:
>  * use clk-indices for LPCG to fetch each clks offset from dt
> v2->v3:
>  * change scu clk into two cells binding
>  * add clk pm patches to ease the understand of the changes
> v1->v2:
>  * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
>    Documentation/devicetree/bindings/arm/arm,scpi.txt
>  * Add required power domain property
>  * Dropped PATCH 3&4 first, will send the updated version accordingly
>    after the binding is finally determined,
>
> Dong Aisheng (11):
>   dt-bindings: firmware: imx-scu: new binding to parse clocks from
>     device tree
>   dt-bindings: clock: imx-lpcg: add support to parse clocks from device
>     tree
>   clk: imx: scu: add two cells binding support
>   clk: imx: scu: bypass cpu power domains
>   clk: imx: scu: allow scu clk to take device pointer
>   clk: imx: scu: add runtime pm support
>   clk: imx: scu: add suspend/resume support
>   clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
>   clk: imx: lpcg: allow lpcg clk to take device pointer
>   clk: imx: clk-imx8qxp-lpcg: add runtime pm support
>   clk: imx: lpcg: add suspend/resume support
>
>  .../bindings/arm/freescale/fsl,scu.txt        |  12 +-
>  .../bindings/clock/imx8qxp-lpcg.yaml          |  79 ++++--
>  drivers/clk/imx/clk-imx8qxp-lpcg.c            | 139 +++++++++++
>  drivers/clk/imx/clk-imx8qxp.c                 | 136 +++++-----
>  drivers/clk/imx/clk-lpcg-scu.c                |  53 +++-
>  drivers/clk/imx/clk-scu.c                     | 234 +++++++++++++++++-
>  drivers/clk/imx/clk-scu.h                     |  56 ++++-
>  include/dt-bindings/clock/imx8-lpcg.h         |  14 ++
>  8 files changed, 620 insertions(+), 103 deletions(-)
>  create mode 100644 include/dt-bindings/clock/imx8-lpcg.h
>
> --
> 2.23.0
>
Dong Aisheng Aug. 17, 2020, 7:42 a.m. UTC | #2
Gently ping...

On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <dongas86@gmail.com> wrote:
>
> Hi Stephen,
>
> This patchset addressed all your new comments.
> Right now only Patch 2/3 still haven't got any R-b or A-b tags. So
> still need your help to review.
> All other patches already got the R-b tags.
>
> BTW, for [PATCH v7 02/11] dt-bindings: clock: imx-lpcg: add support to parse
>  clocks from device tree.
> It was reviewed by Rob/Shawn and you before, however, due to the legacy binding
> file has been removed and changed to json format.
> So this patch also updated to json format accordingly based on the
> former reviewed patch
> and need a re-review.
>
> Regards
> Aisheng
>
> On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <aisheng.dong@nxp.com> wrote:
> >
> > This patch series is a preparation for the MX8 Architecture improvement.
> > As for IMX SCU based platforms like MX8QM and MX8QXP, they are comprised
> > of a couple of SS(Subsystems) while most of them within the same SS
> > can be shared. e.g. Clocks, Devices and etc.
> >
> > However, current clock binding is using SW IDs for device tree to use
> > which can cause troubles in writing the common <soc>-ss-xx.dtsi file for
> > different SoCs.
> >
> > This patch series aims to introduce a new binding which is more close to
> > hardware and platform independent and can makes us write a more general
> > drivers for different SCU based SoCs.
> >
> > Another important thing is that on MX8, each Clock resource is associated
> > with a power domain. So we have to attach that clock device to the power
> > domain in order to make it work properly. Further more, the clock state
> > will be lost when its power domain is completely off during suspend/resume,
> > so we also introduce the clock state save&restore mechanism.
> >
> > It's based on latest shanw/for-next branch.
> >
> > The top commit is:
> > 3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next
> >
> > ChangeLog:
> > v6->v7:
> >  * addressed all comments from Stephen
> >  * rebased to latest shawn/for-next
> > v5->v6:
> >  * add scu clk unregister if add provider failed
> > v4->v5:
> >  * Address all comments from Stephen
> > v3->v4:
> >  * use clk-indices for LPCG to fetch each clks offset from dt
> > v2->v3:
> >  * change scu clk into two cells binding
> >  * add clk pm patches to ease the understand of the changes
> > v1->v2:
> >  * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
> >    Documentation/devicetree/bindings/arm/arm,scpi.txt
> >  * Add required power domain property
> >  * Dropped PATCH 3&4 first, will send the updated version accordingly
> >    after the binding is finally determined,
> >
> > Dong Aisheng (11):
> >   dt-bindings: firmware: imx-scu: new binding to parse clocks from
> >     device tree
> >   dt-bindings: clock: imx-lpcg: add support to parse clocks from device
> >     tree
> >   clk: imx: scu: add two cells binding support
> >   clk: imx: scu: bypass cpu power domains
> >   clk: imx: scu: allow scu clk to take device pointer
> >   clk: imx: scu: add runtime pm support
> >   clk: imx: scu: add suspend/resume support
> >   clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
> >   clk: imx: lpcg: allow lpcg clk to take device pointer
> >   clk: imx: clk-imx8qxp-lpcg: add runtime pm support
> >   clk: imx: lpcg: add suspend/resume support
> >
> >  .../bindings/arm/freescale/fsl,scu.txt        |  12 +-
> >  .../bindings/clock/imx8qxp-lpcg.yaml          |  79 ++++--
> >  drivers/clk/imx/clk-imx8qxp-lpcg.c            | 139 +++++++++++
> >  drivers/clk/imx/clk-imx8qxp.c                 | 136 +++++-----
> >  drivers/clk/imx/clk-lpcg-scu.c                |  53 +++-
> >  drivers/clk/imx/clk-scu.c                     | 234 +++++++++++++++++-
> >  drivers/clk/imx/clk-scu.h                     |  56 ++++-
> >  include/dt-bindings/clock/imx8-lpcg.h         |  14 ++
> >  8 files changed, 620 insertions(+), 103 deletions(-)
> >  create mode 100644 include/dt-bindings/clock/imx8-lpcg.h
> >
> > --
> > 2.23.0
> >
Dong Aisheng Sept. 2, 2020, 6:35 a.m. UTC | #3
Hi Stephen,

Would you help look at this?
Now only one patch left that still does not have a R-b tag and needs
your review.
[PATCH v7 03/11] clk: imx: scu: add two cells binding support
All other patches already get R-b tags.

Regards
Aisheng

On Mon, Aug 17, 2020 at 3:42 PM Dong Aisheng <dongas86@gmail.com> wrote:
>
> Gently ping...
>
> On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <dongas86@gmail.com> wrote:
> >
> > Hi Stephen,
> >
> > This patchset addressed all your new comments.
> > Right now only Patch 2/3 still haven't got any R-b or A-b tags. So
> > still need your help to review.
> > All other patches already got the R-b tags.
> >
> > BTW, for [PATCH v7 02/11] dt-bindings: clock: imx-lpcg: add support to parse
> >  clocks from device tree.
> > It was reviewed by Rob/Shawn and you before, however, due to the legacy binding
> > file has been removed and changed to json format.
> > So this patch also updated to json format accordingly based on the
> > former reviewed patch
> > and need a re-review.
> >
> > Regards
> > Aisheng
> >
> > On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <aisheng.dong@nxp.com> wrote:
> > >
> > > This patch series is a preparation for the MX8 Architecture improvement.
> > > As for IMX SCU based platforms like MX8QM and MX8QXP, they are comprised
> > > of a couple of SS(Subsystems) while most of them within the same SS
> > > can be shared. e.g. Clocks, Devices and etc.
> > >
> > > However, current clock binding is using SW IDs for device tree to use
> > > which can cause troubles in writing the common <soc>-ss-xx.dtsi file for
> > > different SoCs.
> > >
> > > This patch series aims to introduce a new binding which is more close to
> > > hardware and platform independent and can makes us write a more general
> > > drivers for different SCU based SoCs.
> > >
> > > Another important thing is that on MX8, each Clock resource is associated
> > > with a power domain. So we have to attach that clock device to the power
> > > domain in order to make it work properly. Further more, the clock state
> > > will be lost when its power domain is completely off during suspend/resume,
> > > so we also introduce the clock state save&restore mechanism.
> > >
> > > It's based on latest shanw/for-next branch.
> > >
> > > The top commit is:
> > > 3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next
> > >
> > > ChangeLog:
> > > v6->v7:
> > >  * addressed all comments from Stephen
> > >  * rebased to latest shawn/for-next
> > > v5->v6:
> > >  * add scu clk unregister if add provider failed
> > > v4->v5:
> > >  * Address all comments from Stephen
> > > v3->v4:
> > >  * use clk-indices for LPCG to fetch each clks offset from dt
> > > v2->v3:
> > >  * change scu clk into two cells binding
> > >  * add clk pm patches to ease the understand of the changes
> > > v1->v2:
> > >  * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
> > >    Documentation/devicetree/bindings/arm/arm,scpi.txt
> > >  * Add required power domain property
> > >  * Dropped PATCH 3&4 first, will send the updated version accordingly
> > >    after the binding is finally determined,
> > >
> > > Dong Aisheng (11):
> > >   dt-bindings: firmware: imx-scu: new binding to parse clocks from
> > >     device tree
> > >   dt-bindings: clock: imx-lpcg: add support to parse clocks from device
> > >     tree
> > >   clk: imx: scu: add two cells binding support
> > >   clk: imx: scu: bypass cpu power domains
> > >   clk: imx: scu: allow scu clk to take device pointer
> > >   clk: imx: scu: add runtime pm support
> > >   clk: imx: scu: add suspend/resume support
> > >   clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
> > >   clk: imx: lpcg: allow lpcg clk to take device pointer
> > >   clk: imx: clk-imx8qxp-lpcg: add runtime pm support
> > >   clk: imx: lpcg: add suspend/resume support
> > >
> > >  .../bindings/arm/freescale/fsl,scu.txt        |  12 +-
> > >  .../bindings/clock/imx8qxp-lpcg.yaml          |  79 ++++--
> > >  drivers/clk/imx/clk-imx8qxp-lpcg.c            | 139 +++++++++++
> > >  drivers/clk/imx/clk-imx8qxp.c                 | 136 +++++-----
> > >  drivers/clk/imx/clk-lpcg-scu.c                |  53 +++-
> > >  drivers/clk/imx/clk-scu.c                     | 234 +++++++++++++++++-
> > >  drivers/clk/imx/clk-scu.h                     |  56 ++++-
> > >  include/dt-bindings/clock/imx8-lpcg.h         |  14 ++
> > >  8 files changed, 620 insertions(+), 103 deletions(-)
> > >  create mode 100644 include/dt-bindings/clock/imx8-lpcg.h
> > >
> > > --
> > > 2.23.0
> > >
Aisheng Dong Sept. 27, 2020, 4:13 a.m. UTC | #4
Gently ping again...

@Stephen Boyd
Could you kindly provide some feedback how we can proceed this?

Regards
Aisheng

> From: Dong Aisheng <dongas86@gmail.com>
> Sent: Wednesday, September 2, 2020 2:36 PM
> 
> Hi Stephen,
> 
> Would you help look at this?
> Now only one patch left that still does not have a R-b tag and needs your review.
> [PATCH v7 03/11] clk: imx: scu: add two cells binding support All other patches
> already get R-b tags.
> 
> Regards
> Aisheng
> 
> On Mon, Aug 17, 2020 at 3:42 PM Dong Aisheng <dongas86@gmail.com>
> wrote:
> >
> > Gently ping...
> >
> > On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <dongas86@gmail.com>
> wrote:
> > >
> > > Hi Stephen,
> > >
> > > This patchset addressed all your new comments.
> > > Right now only Patch 2/3 still haven't got any R-b or A-b tags. So
> > > still need your help to review.
> > > All other patches already got the R-b tags.
> > >
> > > BTW, for [PATCH v7 02/11] dt-bindings: clock: imx-lpcg: add support
> > > to parse  clocks from device tree.
> > > It was reviewed by Rob/Shawn and you before, however, due to the
> > > legacy binding file has been removed and changed to json format.
> > > So this patch also updated to json format accordingly based on the
> > > former reviewed patch and need a re-review.
> > >
> > > Regards
> > > Aisheng
> > >
> > > On Wed, Jul 29, 2020 at 4:04 PM Dong Aisheng <aisheng.dong@nxp.com>
> wrote:
> > > >
> > > > This patch series is a preparation for the MX8 Architecture improvement.
> > > > As for IMX SCU based platforms like MX8QM and MX8QXP, they are
> > > > comprised of a couple of SS(Subsystems) while most of them within
> > > > the same SS can be shared. e.g. Clocks, Devices and etc.
> > > >
> > > > However, current clock binding is using SW IDs for device tree to
> > > > use which can cause troubles in writing the common
> > > > <soc>-ss-xx.dtsi file for different SoCs.
> > > >
> > > > This patch series aims to introduce a new binding which is more
> > > > close to hardware and platform independent and can makes us write
> > > > a more general drivers for different SCU based SoCs.
> > > >
> > > > Another important thing is that on MX8, each Clock resource is
> > > > associated with a power domain. So we have to attach that clock
> > > > device to the power domain in order to make it work properly.
> > > > Further more, the clock state will be lost when its power domain
> > > > is completely off during suspend/resume, so we also introduce the clock
> state save&restore mechanism.
> > > >
> > > > It's based on latest shanw/for-next branch.
> > > >
> > > > The top commit is:
> > > > 3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next
> > > >
> > > > ChangeLog:
> > > > v6->v7:
> > > >  * addressed all comments from Stephen
> > > >  * rebased to latest shawn/for-next
> > > > v5->v6:
> > > >  * add scu clk unregister if add provider failed
> > > > v4->v5:
> > > >  * Address all comments from Stephen
> > > > v3->v4:
> > > >  * use clk-indices for LPCG to fetch each clks offset from dt
> > > > v2->v3:
> > > >  * change scu clk into two cells binding
> > > >  * add clk pm patches to ease the understand of the changes
> > > > v1->v2:
> > > >  * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
> > > >    Documentation/devicetree/bindings/arm/arm,scpi.txt
> > > >  * Add required power domain property
> > > >  * Dropped PATCH 3&4 first, will send the updated version accordingly
> > > >    after the binding is finally determined,
> > > >
> > > > Dong Aisheng (11):
> > > >   dt-bindings: firmware: imx-scu: new binding to parse clocks from
> > > >     device tree
> > > >   dt-bindings: clock: imx-lpcg: add support to parse clocks from device
> > > >     tree
> > > >   clk: imx: scu: add two cells binding support
> > > >   clk: imx: scu: bypass cpu power domains
> > > >   clk: imx: scu: allow scu clk to take device pointer
> > > >   clk: imx: scu: add runtime pm support
> > > >   clk: imx: scu: add suspend/resume support
> > > >   clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
> > > >   clk: imx: lpcg: allow lpcg clk to take device pointer
> > > >   clk: imx: clk-imx8qxp-lpcg: add runtime pm support
> > > >   clk: imx: lpcg: add suspend/resume support
> > > >
> > > >  .../bindings/arm/freescale/fsl,scu.txt        |  12 +-
> > > >  .../bindings/clock/imx8qxp-lpcg.yaml          |  79 ++++--
> > > >  drivers/clk/imx/clk-imx8qxp-lpcg.c            | 139 +++++++++++
> > > >  drivers/clk/imx/clk-imx8qxp.c                 | 136 +++++-----
> > > >  drivers/clk/imx/clk-lpcg-scu.c                |  53 +++-
> > > >  drivers/clk/imx/clk-scu.c                     | 234
> +++++++++++++++++-
> > > >  drivers/clk/imx/clk-scu.h                     |  56 ++++-
> > > >  include/dt-bindings/clock/imx8-lpcg.h         |  14 ++
> > > >  8 files changed, 620 insertions(+), 103 deletions(-)  create mode
> > > > 100644 include/dt-bindings/clock/imx8-lpcg.h
> > > >
> > > > --
> > > > 2.23.0
> > > >
Stephen Boyd Oct. 12, 2020, 7:28 p.m. UTC | #5
Quoting Aisheng Dong (2020-09-26 21:13:44)
> Gently ping again...
> 
> @Stephen Boyd
> Could you kindly provide some feedback how we can proceed this?
> 

Hmm I thought Shawn was going to apply this a while ago. Did that not
happen? I guess I'll have to pile it in now.
Dong Aisheng Oct. 13, 2020, 9:27 a.m. UTC | #6
Hi Stephen,

On Tue, Oct 13, 2020 at 3:28 AM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Quoting Aisheng Dong (2020-09-26 21:13:44)
> > Gently ping again...
> >
> > @Stephen Boyd
> > Could you kindly provide some feedback how we can proceed this?
> >
>
> Hmm I thought Shawn was going to apply this a while ago. Did that not
> happen? I guess I'll have to pile it in now.

Now there's still one patch that does not have your R-b tags.
I guess Shawn is waiting for this.
Once you give the R-b tags, this series could go to Shawn's tree
as another DT patch set also depends on this.

Would you help check this left one?
[PATCH v7 03/11] clk: imx: scu: add two cells binding support
All other patches already get yours R-b tags.

Shawn,
are you fine with this patch series to go through your tree?

Regards
Aisheng
Stephen Boyd Oct. 14, 2020, 12:41 a.m. UTC | #7
Quoting Dong Aisheng (2020-10-13 02:27:37)
> On Tue, Oct 13, 2020 at 3:28 AM Stephen Boyd <sboyd@kernel.org> wrote:
> >
> > Quoting Aisheng Dong (2020-09-26 21:13:44)
> > > Gently ping again...
> > >
> > > @Stephen Boyd
> > > Could you kindly provide some feedback how we can proceed this?
> > >
> >
> > Hmm I thought Shawn was going to apply this a while ago. Did that not
> > happen? I guess I'll have to pile it in now.
> 
> Now there's still one patch that does not have your R-b tags.
> I guess Shawn is waiting for this.
> Once you give the R-b tags, this series could go to Shawn's tree
> as another DT patch set also depends on this.
> 
> Would you help check this left one?
> [PATCH v7 03/11] clk: imx: scu: add two cells binding support
> All other patches already get yours R-b tags.

Ok I see. Sent the tag now.
Dong Aisheng Oct. 15, 2020, 6:51 a.m. UTC | #8
Hi Shawn,

On Wed, Oct 14, 2020 at 8:41 AM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Quoting Dong Aisheng (2020-10-13 02:27:37)
> > On Tue, Oct 13, 2020 at 3:28 AM Stephen Boyd <sboyd@kernel.org> wrote:
> > >
> > > Quoting Aisheng Dong (2020-09-26 21:13:44)
> > > > Gently ping again...
> > > >
> > > > @Stephen Boyd
> > > > Could you kindly provide some feedback how we can proceed this?
> > > >
> > >
> > > Hmm I thought Shawn was going to apply this a while ago. Did that not
> > > happen? I guess I'll have to pile it in now.
> >
> > Now there's still one patch that does not have your R-b tags.
> > I guess Shawn is waiting for this.
> > Once you give the R-b tags, this series could go to Shawn's tree
> > as another DT patch set also depends on this.
> >
> > Would you help check this left one?
> > [PATCH v7 03/11] clk: imx: scu: add two cells binding support
> > All other patches already get yours R-b tags.
>
> Ok I see. Sent the tag now.

Thanks.

Shawn,
Would you apply this series?
Kindly let me know if you want a resend.

Regards
Aisheng
Shawn Guo Oct. 26, 2020, 2:55 a.m. UTC | #9
On Wed, Jul 29, 2020 at 04:00:07PM +0800, Dong Aisheng wrote:
> This patch series is a preparation for the MX8 Architecture improvement.
> As for IMX SCU based platforms like MX8QM and MX8QXP, they are comprised
> of a couple of SS(Subsystems) while most of them within the same SS
> can be shared. e.g. Clocks, Devices and etc.
> 
> However, current clock binding is using SW IDs for device tree to use
> which can cause troubles in writing the common <soc>-ss-xx.dtsi file for
> different SoCs.
> 
> This patch series aims to introduce a new binding which is more close to
> hardware and platform independent and can makes us write a more general
> drivers for different SCU based SoCs.
> 
> Another important thing is that on MX8, each Clock resource is associated
> with a power domain. So we have to attach that clock device to the power
> domain in order to make it work properly. Further more, the clock state
> will be lost when its power domain is completely off during suspend/resume,
> so we also introduce the clock state save&restore mechanism.
> 
> It's based on latest shanw/for-next branch.
> 
> The top commit is:
> 3c1a41dab7b8 Merge branch 'imx/defconfig' into for-next
> 
> ChangeLog:
> v6->v7:
>  * addressed all comments from Stephen
>  * rebased to latest shawn/for-next
> v5->v6:
>  * add scu clk unregister if add provider failed
> v4->v5:
>  * Address all comments from Stephen
> v3->v4:
>  * use clk-indices for LPCG to fetch each clks offset from dt
> v2->v3:
>  * change scu clk into two cells binding
>  * add clk pm patches to ease the understand of the changes
> v1->v2:
>  * SCU clock changed to one cell clock binding inspired by arm,scpi.txt
>    Documentation/devicetree/bindings/arm/arm,scpi.txt
>  * Add required power domain property
>  * Dropped PATCH 3&4 first, will send the updated version accordingly
>    after the binding is finally determined,
> 
> Dong Aisheng (11):
>   dt-bindings: firmware: imx-scu: new binding to parse clocks from
>     device tree
>   dt-bindings: clock: imx-lpcg: add support to parse clocks from device
>     tree
>   clk: imx: scu: add two cells binding support
>   clk: imx: scu: bypass cpu power domains
>   clk: imx: scu: allow scu clk to take device pointer
>   clk: imx: scu: add runtime pm support
>   clk: imx: scu: add suspend/resume support
>   clk: imx: imx8qxp-lpcg: add parsing clocks from device tree
>   clk: imx: lpcg: allow lpcg clk to take device pointer
>   clk: imx: clk-imx8qxp-lpcg: add runtime pm support
>   clk: imx: lpcg: add suspend/resume support

Applied all, thanks.