mbox series

[0/2] clk: meson: expose clk ids

Message ID 20170731113832.26490-1-jbrunet@baylibre.com
Headers show
Series clk: meson: expose clk ids | expand

Message

Jerome Brunet July 31, 2017, 11:38 a.m. UTC
In previous cycles, concerns have been raised about the way clocks are
exposed for the meson clock controller, only when validated and required
in DT.

Instead, this patchset expose all the known clocks which may be needed in
DT later on. The clocks left out are internal clocks which should never be
used directly in DT. These are mostly internal muxes and dividers of
composite clocks.

While this should reduce the patch flow on the related header files, It
won't stop it completely. There are still many clocks we don't know about
yet. We tend discover a few in every cycle.

This patchset depends on these fixes [0] to avoid issues when adding new
clk ids.

[0]: https://lkml.kernel.org/r/20170727161755.10393-1-jbrunet@baylibre.com

Jerome Brunet (2):
  clk: meson8b: expose every clock in the bindings
  clk: meson-gxbb: expose almost every clock in the bindings

 drivers/clk/meson/gxbb.h                 | 117 ++-----------------------------
 drivers/clk/meson/meson8b.h              | 103 ++-------------------------
 include/dt-bindings/clock/gxbb-clkc.h    |  60 ++++++++++++++++
 include/dt-bindings/clock/meson8b-clkc.h |  70 ++++++++++++++++++
 4 files changed, 141 insertions(+), 209 deletions(-)

-- 
2.9.4

Comments

Neil Armstrong July 31, 2017, noon UTC | #1
On 07/31/2017 01:38 PM, Jerome Brunet wrote:
> In previous cycles, concerns have been raised about the way clocks are

> exposed for the meson clock controller, only when validated and required

> in DT.

> 

> Instead, this patchset expose all the known clocks which may be needed in

> DT later on. The clocks left out are internal clocks which should never be

> used directly in DT. These are mostly internal muxes and dividers of

> composite clocks.

> 

> While this should reduce the patch flow on the related header files, It

> won't stop it completely. There are still many clocks we don't know about

> yet. We tend discover a few in every cycle.

> 

> This patchset depends on these fixes [0] to avoid issues when adding new

> clk ids.

> 

> [0]: https://lkml.kernel.org/r/20170727161755.10393-1-jbrunet@baylibre.com

> 

> Jerome Brunet (2):

>   clk: meson8b: expose every clock in the bindings

>   clk: meson-gxbb: expose almost every clock in the bindings

> 

>  drivers/clk/meson/gxbb.h                 | 117 ++-----------------------------

>  drivers/clk/meson/meson8b.h              | 103 ++-------------------------

>  include/dt-bindings/clock/gxbb-clkc.h    |  60 ++++++++++++++++

>  include/dt-bindings/clock/meson8b-clkc.h |  70 ++++++++++++++++++

>  4 files changed, 141 insertions(+), 209 deletions(-)

> 


Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Arnd Bergmann July 31, 2017, 2:05 p.m. UTC | #2
On Mon, Jul 31, 2017 at 1:38 PM, Jerome Brunet <jbrunet@baylibre.com> wrote:
> In previous cycles, concerns have been raised about the way clocks are

> exposed for the meson clock controller, only when validated and required

> in DT.

>

> Instead, this patchset expose all the known clocks which may be needed in

> DT later on. The clocks left out are internal clocks which should never be

> used directly in DT. These are mostly internal muxes and dividers of

> composite clocks.

>

> While this should reduce the patch flow on the related header files, It

> won't stop it completely. There are still many clocks we don't know about

> yet. We tend discover a few in every cycle.

>

> This patchset depends on these fixes [0] to avoid issues when adding new

> clk ids.

>

> [0]: https://lkml.kernel.org/r/20170727161755.10393-1-jbrunet@baylibre.com


Thanks a lot!

 Acked-by: Arnd Bergmann <arnd@arndb.de>