* [PATCH v2 00/10] sh73a0 common clock framework implementation
@ 2014-09-02 9:12 Ulrich Hecht
2014-09-03 0:09 ` Simon Horman
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Ulrich Hecht @ 2014-09-02 9:12 UTC (permalink / raw)
To: linux-sh
Hi!
Here's an updated sh73a0 CCF implementation. It adds the missing clock
assignments, CMT1, and various minor fixes. It also splits off the parent
selection for the DIV6 clocks so it can be merged without requiring the DIV6
driver patch.
CU
Uli
Changes since v1:
- div6 patch replaced with v4
- split off div6 extended clocks
- added cmt1 DT description
- added MSTP clock assignments
- reordered according to dependencies
- removed dummy frequencies in extal2, extcki
- removed enable_on_init on zb1 and flctlck
- added missing tpu4 in mstp3_clks
- use c->parent in cpg driver
- fix some style issues
Ulrich Hecht (10):
clk: shmobile: sh73a0 common clock framework implementation
ARM: shmobile: sh73a0: Common clock framework DT description
ARM: shmobile: sh73a0: common clock framework board support
ARM: shmobile: sh73a0: add MSTP clock assignments to DT
ARM: shmobile: sh73a0: Add CMT1 device to DT
ARM: shmobile: kzm9g-reference: Enable CMT1 in device tree
ARM: shmobile: kzm9g: require ZB1 clock for Ethernet controller
clk: shmobile: Add r8a7740, sh73a0 SoCs to MSTP bindings
clk: shmobile: div6: support selectable-input clocks
ARM: shmobile: sh73a0: add selectable sources to DIV6 clocks
.../bindings/clock/renesas,cpg-div6-clocks.txt | 12 +-
.../bindings/clock/renesas,cpg-mstp-clocks.txt | 2 +
.../bindings/clock/renesas,sh73a0-cpg-clocks.txt | 31 ++
arch/arm/boot/dts/sh73a0-kzm9g-reference.dts | 9 +
arch/arm/boot/dts/sh73a0.dtsi | 440 +++++++++++++++++++++
arch/arm/mach-shmobile/board-kzm9g-reference.c | 25 ++
arch/arm/mach-shmobile/setup-sh73a0.c | 16 +-
drivers/clk/shmobile/Makefile | 1 +
drivers/clk/shmobile/clk-div6.c | 32 +-
drivers/clk/shmobile/clk-sh73a0.c | 202 ++++++++++
include/dt-bindings/clock/sh73a0-clock.h | 79 ++++
11 files changed, 832 insertions(+), 17 deletions(-)
create mode 100644 Documentation/devicetree/bindings/clock/renesas,sh73a0-cpg-clocks.txt
create mode 100644 drivers/clk/shmobile/clk-sh73a0.c
create mode 100644 include/dt-bindings/clock/sh73a0-clock.h
--
1.8.4.5
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2 00/10] sh73a0 common clock framework implementation
2014-09-02 9:12 [PATCH v2 00/10] sh73a0 common clock framework implementation Ulrich Hecht
@ 2014-09-03 0:09 ` Simon Horman
2014-09-03 8:05 ` Ulrich Hecht
2014-09-04 7:09 ` Simon Horman
2 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2014-09-03 0:09 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 02, 2014 at 11:12:56AM +0200, Ulrich Hecht wrote:
> Hi!
>
> Here's an updated sh73a0 CCF implementation. It adds the missing clock
> assignments, CMT1, and various minor fixes. It also splits off the parent
> selection for the DIV6 clocks so it can be merged without requiring the DIV6
> driver patch.
>
> CU
> Uli
>
>
> Changes since v1:
> - div6 patch replaced with v4
> - split off div6 extended clocks
> - added cmt1 DT description
> - added MSTP clock assignments
> - reordered according to dependencies
> - removed dummy frequencies in extal2, extcki
> - removed enable_on_init on zb1 and flctlck
> - added missing tpu4 in mstp3_clks
> - use c->parent in cpg driver
> - fix some style issues
>
>
> Ulrich Hecht (10):
> clk: shmobile: sh73a0 common clock framework implementation
> ARM: shmobile: sh73a0: Common clock framework DT description
> ARM: shmobile: sh73a0: common clock framework board support
> ARM: shmobile: sh73a0: add MSTP clock assignments to DT
> ARM: shmobile: sh73a0: Add CMT1 device to DT
> ARM: shmobile: kzm9g-reference: Enable CMT1 in device tree
> ARM: shmobile: kzm9g: require ZB1 clock for Ethernet controller
> clk: shmobile: Add r8a7740, sh73a0 SoCs to MSTP bindings
> clk: shmobile: div6: support selectable-input clocks
> ARM: shmobile: sh73a0: add selectable sources to DIV6 clocks
Hi Ulrich,
in general this seems nice to me. I have a few observations:
* It looks like some of the "ARM: shmobile:" patches,
especially patches 2 - 7, may break bisection. If
so could you re-arrange the patches to avoid this?
* It looks like you are enabling CMT1. Thanks for that,
for some reason I thought that I had already handled that: it seems not.
* Would it be useful if I made a series on top of your series
to enable serial via DT. Again, I thought I had already handled
that too but it seems I haven't.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 00/10] sh73a0 common clock framework implementation
2014-09-02 9:12 [PATCH v2 00/10] sh73a0 common clock framework implementation Ulrich Hecht
2014-09-03 0:09 ` Simon Horman
@ 2014-09-03 8:05 ` Ulrich Hecht
2014-09-04 7:09 ` Simon Horman
2 siblings, 0 replies; 4+ messages in thread
From: Ulrich Hecht @ 2014-09-03 8:05 UTC (permalink / raw)
To: linux-sh
On Wed, Sep 3, 2014 at 2:09 AM, Simon Horman <horms@verge.net.au> wrote:
> * It looks like some of the "ARM: shmobile:" patches,
> especially patches 2 - 7, may break bisection. If
> so could you re-arrange the patches to avoid this?
I don't really see how.
> * It looks like you are enabling CMT1. Thanks for that,
> for some reason I thought that I had already handled that: it seems not.
Well...
> * Would it be useful if I made a series on top of your series
> to enable serial via DT. Again, I thought I had already handled
> that too but it seems I haven't.
...I thought I had handled that already. :) Please go ahead.
CU
Uli
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 00/10] sh73a0 common clock framework implementation
2014-09-02 9:12 [PATCH v2 00/10] sh73a0 common clock framework implementation Ulrich Hecht
2014-09-03 0:09 ` Simon Horman
2014-09-03 8:05 ` Ulrich Hecht
@ 2014-09-04 7:09 ` Simon Horman
2 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2014-09-04 7:09 UTC (permalink / raw)
To: linux-sh
On Wed, Sep 03, 2014 at 10:05:46AM +0200, Ulrich Hecht wrote:
> On Wed, Sep 3, 2014 at 2:09 AM, Simon Horman <horms@verge.net.au> wrote:
> > * It looks like some of the "ARM: shmobile:" patches,
> > especially patches 2 - 7, may break bisection. If
> > so could you re-arrange the patches to avoid this?
>
> I don't really see how.
Thanks, nothing jumps out now that I look at it for a second time.
> > * It looks like you are enabling CMT1. Thanks for that,
> > for some reason I thought that I had already handled that: it seems not.
>
> Well...
>
> > * Would it be useful if I made a series on top of your series
> > to enable serial via DT. Again, I thought I had already handled
> > that too but it seems I haven't.
>
> ...I thought I had handled that already. :) Please go ahead.
I must have been looking at the wrong branch yesterday,
it seems to be there.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-09-04 7:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-02 9:12 [PATCH v2 00/10] sh73a0 common clock framework implementation Ulrich Hecht
2014-09-03 0:09 ` Simon Horman
2014-09-03 8:05 ` Ulrich Hecht
2014-09-04 7:09 ` Simon Horman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).