Re: [PATCH v2 0/8] Common Clock Framework support for Samsung S3C64xx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/07/13 07:06, Tomasz Figa wrote:
On Tuesday 06 of August 2013 12:47:51 Mike Turquette wrote:
Quoting Tomasz Figa (2013-08-05 16:42:16)

On Monday 05 of August 2013 12:02:16 Mike Turquette wrote:
Quoting Kukjin Kim (2013-08-05 11:13:55)

On 08/06/13 03:06, Mike Turquette wrote:
Quoting Kukjin Kim (2013-08-05 10:01:36)

On 07/23/13 08:49, Tomasz Figa wrote:
This series is an attempt to move clock support on Samsung
S3C64xx
SoCs
to Common Clock Framework.

First, support for PLL types present on S3C64xx SoCs is added
to
Samsung Common Clock Framework driver. Then the main clock
driver
for mentioned SoCs is introduced. Further patches contain
fixes
for drivers to make them compliant with CCF semantics,
migration
of platform code to use the new clock driver and removal of
old
clock management code.

Depends on:
    - [PATCH v4 00/20] Samsung PWM support cleanup

      http://thread.gmane.org/gmane.linux.kernel.samsung-soc/20
      856

On S3C6410-based Tiny6410 board (Mini6410-compatible):

Tested-by: Tomasz Figa<tomasz.figa@xxxxxxxxx>

For v1:

Acked-by: Mike Turquette<mturquette@xxxxxxxxxx>

Changes since v1:
    - added patch for read-only muxes,
    - exported configurable muxes and dividers,
    - defined mout_syncmux as read-only mux,
    - in DT-enabled case fixed-clock binding is used to define
    external clocks.>  >>>

Tomasz Figa (8):
     clk: mux: Add support for read-only muxes.
     clk: samsung: pll: Add support for PLL6552 and PLL6553
     clk: samsung: Add clock driver for S3C64xx SoCs
     ARM: SAMSUNG: Add soc_is_s3c6400/s3c6410 macros
     ARM: s3c64xx: dma: Use
     clk_prepare_enable/clk_disable_unprepare
     usb: host: ohci-s3c2410 Use
     clk_prepare_enable/clk_disable_unprepare
     ARM: s3c64xx: Migrate clock handling to Common Clock
     Framework
     ARM: s3c64xx: Remove old clock management code

    .../bindings/clock/samsung,s3c64xx-clock.txt       |   77
    ++
    arch/arm/Kconfig                                   |    2
    +-
    arch/arm/mach-s3c64xx/Makefile                     |    2
    +-
    arch/arm/mach-s3c64xx/clock.c                      | 1007
    -------------------- arch/arm/mach-s3c64xx/common.c

             |   21 +-

    arch/arm/mach-s3c64xx/common.h                     |   12
    +-
    arch/arm/mach-s3c64xx/dma.c                        |    4
    +-
    arch/arm/mach-s3c64xx/include/mach/regs-clock.h    |  132
    +--
    arch/arm/mach-s3c64xx/mach-anw6410.c               |    2
    +-
    arch/arm/mach-s3c64xx/mach-crag6410.c              |    2
    +-
    arch/arm/mach-s3c64xx/mach-hmt.c                   |    2
    +-
    arch/arm/mach-s3c64xx/mach-mini6410.c              |    2
    +-
    arch/arm/mach-s3c64xx/mach-ncp.c                   |    2
    +-
    arch/arm/mach-s3c64xx/mach-smartq.c                |   11
    +-
    arch/arm/mach-s3c64xx/mach-smdk6400.c              |    2
    +-
    arch/arm/mach-s3c64xx/mach-smdk6410.c              |    2
    +-
    arch/arm/mach-s3c64xx/pm.c                         |   21 -
    arch/arm/mach-s3c64xx/s3c6400.c                    |    6 -
    arch/arm/mach-s3c64xx/s3c6410.c                    |    7 -
    arch/arm/plat-samsung/include/plat/cpu.h           |    4 +
    drivers/clk/clk-mux.c                              |   10
    +-
    drivers/clk/samsung/Makefile                       |    1 +
    drivers/clk/samsung/clk-pll.c                      |  160
    ++++
    drivers/clk/samsung/clk-pll.h                      |    4 +
    drivers/clk/samsung/clk-s3c64xx.c                  |  465
    +++++++++
    drivers/usb/host/ohci-s3c2410.c                    |    8
    +-
    include/dt-bindings/clock/samsung,s3c64xx-clock.h  |  178
    ++++
    include/linux/clk-provider.h                       |    2 +
    28 files changed, 943 insertions(+), 1205 deletions(-)
    create mode 100644
    Documentation/devicetree/bindings/clock/samsung,s3c64xx-clo
    ck.
    txt delete mode 100644 arch/arm/mach-s3c64xx/clock.c
    create mode 100644 drivers/clk/samsung/clk-s3c64xx.c
    create mode 100644
    include/dt-bindings/clock/samsung,s3c64xx-clock.h

Basically, this series looks good to me, but I'm not sure how
this
should be handled because of dependency with PWM cleanup and
clk
stuff
in clk tree now...

Patches 1-3 can go into the clk tree. 4-6 should go through
their
respective trees.

If you want I can take 7&   8 through the clk tree.

Alternatively I can provide patches 1-3 in a separate stable
topic
branch for you to pull in as a dependency. We'll both merge that
stable
topic branch into our trees and you can make a note of it for
the
arm-soc folks.

Thanks for your quick response. Would be helpful to samsung tree
if
you
could provide a separate stable topic branch what you suggested
for
clk
stuff in this series.

git://git.linaro.org/people/mturquette/linux.git clk-next-s3c64xx

Topic branch contains patches 1-3 of this series on top of
v3.11-rc2. I
have already merged the same into the clk-next branch.

Mike, thanks. I've merged the branch 'clk-next-s3c64xx' into the samsung tree :)

What about the dependency on PWM cleanup series? Patches 7-8 must be
merged after them, otherwise there will be a regression, because the
old PWM clocksource driver will be broken and so s3c64xx won't boot
until moved to new one, which is done by mentioned series.

Is this a question for me? I did not take patches 7 or 8 into clk-next.
Sounds like those patches should go through Samsung tree into arm-soc,
and perhaps be a late-applied branch to get the ordering right.

It's just continuation of the topic. I guess the question is mostly for
Kukjin. Anyway, your suggestion is most likely the right thing, thanks.

Tomasz, I can handle. Thanks.

- Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux