From: kgene.kim@samsung•com (Kukjin Kim)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v2 0/8] Common Clock Framework support for Samsung S3C64xx
Date: Wed, 07 Aug 2013 07:11:40 +0900 [thread overview]
Message-ID: <5201749C.70307@samsung.com> (raw)
In-Reply-To: <2308912.pxOVkGYgip@flatron>
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@gmail•com>
>>>>>>>>
>>>>>>>> For v1:
>>>>>>>>
>>>>>>>> Acked-by: Mike Turquette<mturquette@linaro•org>
>>>>>>>>
>>>>>>>> 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
next prev parent reply other threads:[~2013-08-06 22:11 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-22 23:49 [PATCH v2 0/8] Common Clock Framework support for Samsung S3C64xx Tomasz Figa
2013-07-22 23:49 ` [PATCH v2 1/8] clk: mux: Add support for read-only muxes Tomasz Figa
2013-07-23 11:22 ` Sergei Shtylyov
2013-07-23 11:25 ` Tomasz Figa
2013-07-27 12:41 ` Tomasz Figa
2013-08-02 21:46 ` Mike Turquette
2013-07-22 23:49 ` [PATCH v2 2/8] clk: samsung: pll: Add support for PLL6552 and PLL6553 Tomasz Figa
2013-07-23 23:52 ` [PATCH v3 " Tomasz Figa
2013-07-28 12:30 ` Mark Brown
2013-07-28 12:38 ` Tomasz Figa
2013-08-02 22:53 ` Mike Turquette
2013-07-22 23:49 ` [PATCH v2 3/8] clk: samsung: Add clock driver for S3C64xx SoCs Tomasz Figa
2013-07-23 23:55 ` [PATCH v3 " Tomasz Figa
2013-07-22 23:49 ` [PATCH v2 4/8] ARM: SAMSUNG: Add soc_is_s3c6400/s3c6410 macros Tomasz Figa
2013-07-22 23:49 ` [PATCH v2 5/8] ARM: s3c64xx: dma: Use clk_prepare_enable/clk_disable_unprepare Tomasz Figa
2013-07-28 12:32 ` Mark Brown
2013-07-22 23:49 ` [PATCH v2 6/8] usb: host: ohci-s3c2410 " Tomasz Figa
2013-07-23 0:15 ` Fabio Estevam
2013-07-23 0:44 ` Tomasz Figa
2013-07-31 18:58 ` Tomasz Figa
2013-07-31 20:44 ` Alan Stern
2013-08-01 7:45 ` Greg KH
2013-07-22 23:49 ` [PATCH v2 7/8] ARM: s3c64xx: Migrate clock handling to Common Clock Framework Tomasz Figa
2013-07-22 23:49 ` [PATCH v2 8/8] ARM: s3c64xx: Remove old clock management code Tomasz Figa
2013-07-24 12:20 ` [PATCH v2 0/8] Common Clock Framework support for Samsung S3C64xx Kukjin Kim
2013-08-05 17:01 ` Kukjin Kim
2013-08-05 18:06 ` Mike Turquette
2013-08-05 18:13 ` Kukjin Kim
2013-08-05 19:02 ` Mike Turquette
2013-08-05 23:42 ` Tomasz Figa
2013-08-06 19:47 ` Mike Turquette
2013-08-06 22:06 ` Tomasz Figa
2013-08-06 22:11 ` Kukjin Kim [this message]
2013-08-06 22:13 ` Tomasz Figa
2013-08-16 10:44 ` Tomasz Figa
2013-08-16 21:02 ` Mike Turquette
2013-08-16 21:15 ` Tomasz Figa
2013-08-17 10:30 ` Kukjin Kim
2013-08-20 0:22 ` Mike Turquette
2013-08-20 7:09 ` Kukjin Kim
2013-08-25 17:19 ` Kukjin Kim
2013-08-21 0:33 ` [PATCH] clk: samsung: pll: Use new registration method for PLL6552 and PLL6553 Tomasz Figa
2013-08-27 1:14 ` Mike Turquette
2013-08-27 17:16 ` Kukjin Kim
2013-08-27 23:45 ` Mike Turquette
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5201749C.70307@samsung.com \
--to=kgene.kim@samsung$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox