[RESEND,v2,00/18] clk: imx: Switch the imx6 and imx7 to clk_hw based API

Message

Abel Vesa May 29, 2019, 12:26 p.m. UTC

Resend for the following:

https://lkml.org/lkml/2019/5/2/170

Abel Vesa (18):
  clk: imx: Add imx_obtain_fixed_clock clk_hw based variant
  clk: imx6sx: Do not reparent to unregistered IMX6SX_CLK_AXI
  clk: imx6q: Do not reparent uninitialized IMX6QDL_CLK_PERIPH2 clock
  clk: imx: clk-busy: Switch to clk_hw based API
  clk: imx: clk-cpu: Switch to clk_hw based API
  clk: imx: clk-gate2: Switch to clk_hw based API
  clk: imx: clk-pllv3: Switch to clk_hw based API
  clk: imx: clk-pfd: Switch to clk_hw based API
  clk: imx: clk-gate-exclusive: Switch to clk_hw based API
  clk: imx: clk-fixup-div: Switch to clk_hw based API
  clk: imx: clk-fixup-mux: Switch to clk_hw based API
  clk: imx: Switch wrappers to clk_hw based API
  clk: imx6q: Switch to clk_hw based API
  clk: imx6sl: Switch to clk_hw based API
  clk: imx6sx: Switch to clk_hw based API
  clk: imx6ul: Switch to clk_hw based API
  clk: imx7d: Switch to clk_hw based API
  clk: imx6sll: Switch to clk_hw based API

 drivers/clk/imx/clk-busy.c           |  30 +-
 drivers/clk/imx/clk-cpu.c            |  14 +-
 drivers/clk/imx/clk-fixup-div.c      |  15 +-
 drivers/clk/imx/clk-fixup-mux.c      |  15 +-
 drivers/clk/imx/clk-gate-exclusive.c |  17 +-
 drivers/clk/imx/clk-gate2.c          |  14 +-
 drivers/clk/imx/clk-imx6q.c          | 767 ++++++++++++++-------------
 drivers/clk/imx/clk-imx6sl.c         | 404 +++++++-------
 drivers/clk/imx/clk-imx6sll.c        | 430 +++++++--------
 drivers/clk/imx/clk-imx6sx.c         | 656 +++++++++++------------
 drivers/clk/imx/clk-imx6ul.c         | 574 ++++++++++----------
 drivers/clk/imx/clk-imx7d.c          | 983 ++++++++++++++++++-----------------
 drivers/clk/imx/clk-pfd.c            |  14 +-
 drivers/clk/imx/clk-pllv3.c          |  14 +-
 drivers/clk/imx/clk.c                |  11 +
 drivers/clk/imx/clk.h                | 142 +++--
 16 files changed, 2151 insertions(+), 1949 deletions(-)

Comments

Stephen Boyd June 6, 2019, 6:29 p.m. UTC | #1

Quoting Abel Vesa (2019-05-29 05:26:38)
> Resend for the following:
> 
> https://lkml.org/lkml/2019/5/2/170

What's left after this series to convert over to clk_hw based APIs? I'm
happy to see this merge as long as we eventually delete the clk based
versions of the code in the imx driver so that we can complete the task.

I took a look over everything and nothing stuck out, so:

Reviewed-by: Stephen Boyd <[e-mail]>

for the series.

Shawn Guo June 7, 2019, 12:41 a.m. UTC | #2

On Thu, Jun 06, 2019 at 11:29:40AM -0700, Stephen Boyd wrote:
> Quoting Abel Vesa (2019-05-29 05:26:38)
> > Resend for the following:
> > 
> > https://lkml.org/lkml/2019/5/2/170
> 
> What's left after this series to convert over to clk_hw based APIs? I'm
> happy to see this merge as long as we eventually delete the clk based
> versions of the code in the imx driver so that we can complete the task.
> 
> I took a look over everything and nothing stuck out, so:
> 
> Reviewed-by: Stephen Boyd <[e-mail]>
> 
> for the series.

I just applied the series to my for-next branch, so that it can be
pulled into linux-next for testing.

Shawn

Abel Vesa June 7, 2019, 8:01 a.m. UTC | #3

On 19-06-06 11:29:40, Stephen Boyd wrote:
> Quoting Abel Vesa (2019-05-29 05:26:38)
> > Resend for the following:
> > 
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flkml.org%2Flkml%2F2019%2F5%2F2%2F170&data=02%7C01%7Cabel.vesa%40nxp.com%7Ccd18ca91df204bd6f99408d6eaacf258%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636954425864592743&sdata=12O60IgeI2yxb2hJ5NmYyYxkE4o7VwH7rh1Zye%2BkT8E%3D&reserved=0
> 
> What's left after this series to convert over to clk_hw based APIs? I'm
> happy to see this merge as long as we eventually delete the clk based
> versions of the code in the imx driver so that we can complete the task.
> 

The clk based defines will go away once there is no clk provider that needs it.
The left clk providers are: all imx1 through imx5 and imx8.

I will take care of this cleanup myself.

> I took a look over everything and nothing stuck out, so:
> 
> Reviewed-by: Stephen Boyd <[e-mail]>
> 
> for the series.
>